Bug 145335

Summary: Infinite recursion in printconf_conf.py
Product: [Fedora] Fedora Reporter: Jan Arne Petersen <janarne>
Component: system-config-printerAssignee: Tim Waugh <twaugh>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: rawhide   
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: 0.6.120-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-01-17 18:02:11 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
/usr/share/printconf/util/printconf_conf.py none

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.