Comment 10 for bug 335116

There are 2 important defect patches for 3.9.2 that should be applied for
9.04. The patch to hpdio fixes an issue that causes 100% CPU utilization
when the user logs out or switches to another user. The issue with
settingsdialog is a traceback in the UI code due to a syntax error in the
code. Both of these fixes have been tested by HP.

Index: hpdio.py
===================================================================
--- hpdio.py (revision 10469)
+++ hpdio.py (working copy)
@@ -85,12 +85,20 @@
                 r, w, e = select.select([r2], [], [r2], 1.0)
             except KeyboardInterrupt:
                 break
+ except select.error, e:
+ if e[0] == errno.EINTR:
+ continue
+ else:
+ break

             if not r: continue
             if e: break

             m = ''.join([m, os.read(r2, fmt_size)])

+ if not m:
+ break
+
             while len(m) >= fmt_size:
                 response.clear()
                 event = device.Event(*struct.unpack(fmt, m[:fmt_size]))
Index: ui4/settingsdialog.py
===================================================================
--- ui4/settingsdialog.py (revision 10469)
+++ ui4/settingsdialog.py (working copy)
@@ -55,7 +55,7 @@
         if self.user_settings.auto_refresh_type == 1:
             self.RefreshCurrentRadioButton.setChecked(True)
         else:
- RefreshAllRadioButton.setChecked(True)
+ self.RefreshAllRadioButton.setChecked(True)

         self.ScanCommandLineEdit.setText(self.user_settings.cmd_scan)
         self.SystemTraySettings.systray_visible =
self.user_settings.systray_visible

On 3/9/09, Till Kamppeter <email address hidden> wrote:
>
> Tested also faxing on the HP LaserJet 3390 as it is the most bug-prone.
> Works.
>
> Uploaded 3.9.2 to Ubuntu.
>
>
> ** Changed in: hplip (Ubuntu)
> Status: Confirmed => Fix Released
>
>
> --
> FFE Request: HPLIP 3.9.2 released one day after FF
> https://bugs.launchpad.net/bugs/335116
> You received this bug notification because you are a direct subscriber
> of the bug.
>