Fixed traceback on error condition in device.py (bug #628125).

hplip-raw_deviceID-traceback.patch

@@ -0,0 +1,24 @@
+diff -up hplip-3.10.9/base/device.py.raw_deviceID-traceback hplip-3.10.9/base/device.py
+--- hplip-3.10.9/base/device.py.raw_deviceID-traceback	2010-10-18 17:22:55.138868192 +0100
++++ hplip-3.10.9/base/device.py	2010-10-18 17:24:02.965869220 +0100
+@@ -1368,6 +1368,8 @@ class Device(object):
+ 
+     def getDeviceID(self):
+         needs_close = False
++        self.raw_deviceID = ''
++        self.deviceID = {}
+         if self.io_state != IO_STATE_HP_OPEN:
+            try:
+                self.open()
+@@ -1377,10 +1379,7 @@ class Device(object):
+ 
+         result_code, data = hpmudext.get_device_id(self.device_id)
+ 
+-        if result_code != hpmudext.HPMUD_R_OK:
+-            self.raw_deviceID = ''
+-            self.deviceID = {}
+-        else:
++        if result_code == hpmudext.HPMUD_R_OK:
+             self.raw_deviceID = data
+             self.deviceID = parseDeviceID(data)
+ 

hplip.spec

@@ -38,6 +38,7 @@ Patch21: hplip-openPPD.patch
 Patch22: hplip-ppd-ImageableArea.patch
 Patch23: hplip-network-timeout.patch
 Patch24: hplip-addgroup.patch
+Patch25: hplip-raw_deviceID-traceback.patch
 
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
@@ -238,6 +239,9 @@ done
 # utils.addgroup() was returning string instead of array (bug #642771).
 %patch24 -p1 -b .addgroup
 
+# Fixed traceback on error condition in device.py (bug #628125).
+%patch25 -p1 -b .raw_deviceID-traceback
+
 sed -i.duplex-constraints \
     -e 's,\(UIConstraints.* \*Duplex\),//\1,' \
     prnt/drv/hpcups.drv.in
@@ -465,6 +469,7 @@ fi
 
 %changelog
 * Mon Oct 18 2010 Tim Waugh <twaugh@redhat.com> - 3.10.9-3
+- Fixed traceback on error condition in device.py (bug #628125).
 - Fixed bogus low ink warnings from hpijs driver (bug #643643).
 
 * Thu Oct 14 2010 Jiri Popelka <jpopelka@redhat.com> - 3.10.9-2