Bug 145335 - Infinite recursion in printconf_conf.py
Summary: Infinite recursion in printconf_conf.py
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: system-config-printer
Version: rawhide
Hardware: i386
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tim Waugh
QA Contact:
URL:
Whiteboard:
Keywords:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2005-01-17 15:58 UTC by Jan Arne Petersen
Modified: 2007-11-30 22:10 UTC (History)
0 users

(edit)
Clone Of:
(edit)
Last Closed: 2005-01-17 18:02:11 UTC

Attachments (Terms of Use)
/usr/share/printconf/util/printconf_conf.py (57.56 KB, text/plain)
2005-01-17 16:16 UTC, Tim Waugh 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Description Jan Arne Petersen 2005-01-17 15:58:34 UTC 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.5)
Gecko/20041228 Firefox/1.0 Fedora/1.0-8

Description of problem:
When I start system-config-printer and try to edit a HP LaserJet 5MP
system-config-printer freezes with the output below

Version-Release number of selected component (if applicable):
system-config-printer-0.6.119-1

How reproducible:
Always

Steps to Reproduce:
1. start system-config-printer
2. select HP laserprinter
3. click edit
    

Additional info:

...

    p = foomatic_match_printer (alias[a], model)
  File "/usr/share/printconf/util/printconf_conf.py", line 1871, in
foomatic_match_printer
    p = foomatic_match_printer (alias[a], model)
  File "/usr/share/printconf/util/printconf_conf.py", line 1871, in
foomatic_match_printer
    p = foomatic_match_printer (alias[a], model)
  File "/usr/share/printconf/util/printconf_conf.py", line 1870, in
foomatic_match_printer
    if make.lower () == a.lower ():
RuntimeError: maximum recursion depth exceeded in cmp
Comment 1 Tim Waugh 2005-01-17 16:16:13 UTC 
Created attachment 109866 [details]
/usr/share/printconf/util/printconf_conf.py

Thanks for the report.	Please try this replacement printconf_conf.py.
Comment 2 Jan Arne Petersen 2005-01-17 17:32:40 UTC 
Fixed the problem.

Thanks.
Comment 3 Tim Waugh 2005-01-17 17:43:55 UTC 
Great.  Fixed in CVS.  Thanks for testing.
Note You need to log in before you can comment on or make changes to this bug.