After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 738165 - trying to add a printer crashes system settings
trying to add a printer crashes system settings
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Printers
3.14.x
Other Linux
: Normal normal
: ---
Assigned To: Marek Kašík
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-10-08 17:25 UTC by Jakub Steiner
Modified: 2014-11-13 11:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
backtrace (9.99 KB, text/plain)
2014-10-08 17:25 UTC, Jakub Steiner
  Details
1 gigabyte download later, BT with debuginfo (13.59 KB, text/plain)
2014-10-09 19:03 UTC, Jakub Steiner
  Details
Don't crash for URIs with leading "//" (1.01 KB, patch)
2014-10-10 10:01 UTC, Marek Kašík
committed Details | Review

Description Jakub Steiner 2014-10-08 17:25:10 UTC
Created attachment 288062 [details]
backtrace

No local network printers get detected, upon typing in the URI, settings crashed with the following backtrace.
Comment 1 Marek Kašík 2014-10-09 11:34:46 UTC
Hi Jakub,

I can not reproduce the problem. What URI have you entered there? Maybe it is related to its format. Could you try to crash it again with debuginfo installed?
Comment 2 Jakub Steiner 2014-10-09 18:37:44 UTC
//zeus
Comment 3 Jakub Steiner 2014-10-09 18:43:40 UTC
I have updated F21 testing in the meantime and the printer is actually autodetected now. Still crashing though. Will attach the backtrace with debuginfo installed.
Comment 4 Jakub Steiner 2014-10-09 19:03:17 UTC
Created attachment 288169 [details]
1 gigabyte download later, BT with debuginfo
Comment 5 Jakub Steiner 2014-10-09 19:10:04 UTC
Btw setting up the autodetected printer fails unless you explicitly install the splix package: http://youtu.be/HthEZL_ew6U
Comment 6 Marek Kašík 2014-10-10 10:01:36 UTC
Created attachment 288197 [details] [review]
Don't crash for URIs with leading "//"

(In reply to comment #2)
> //zeus

Thank you for the URI. It helped me to reproduce the problem. The crash was caused by an uninitialized value in function which searches for LPD printers.
The attached patch initialize it early so that it doesn't crash now.
Comment 7 Rui Matos 2014-10-10 10:08:01 UTC
Review of attachment 288197 [details] [review]:

Looks fine
Comment 8 Marek Kašík 2014-10-10 10:42:04 UTC
Comment on attachment 288197 [details] [review]
Don't crash for URIs with leading "//"

(In reply to comment #7)
> Review of attachment 288197 [details] [review]:
> 
> Looks fine

Thank you for the review. I've pushed the patch to master.
Comment 9 Marek Kašík 2014-10-10 10:59:57 UTC
I'll look at the problem from comment #5 later.
Comment 10 Marek Kašík 2014-11-13 11:24:31 UTC
(In reply to comment #9)
> I'll look at the problem from comment #5 later.

The problem is a bug in foomatic. I've filled a bug for it here: https://bugzilla.redhat.com/show_bug.cgi?id=1163731.
I'm closing this one then.