Comment 5 for bug 225898

Revision history for this message
Josh Kelley (joshkel) wrote :

As far as I can tell, this is NOT a duplicate of #816763.

That bug states that CommonDefinitions.h uses the wrong log facility for messages created by the dbglog macro in prnt/hpcups/*.cpp.

The error messages in this bug are created by the BUG macro in io/hpmud/musb.c. One error message is created by the
BUG("bulk_write failed buf=%p size=%d len=%d: %m\n") call in the musb_write function, and the other error message is created by the BUG("unable to write data %s: %m\n") call in the musb_raw_channel_write function.

From what I can tell from looking at the code, the only way that musb_raw_channel_write will log its error message is if musb_write had just failed and logged its error message, so I believe that the simplest fix is to remove that BUG call from musb_raw_channel_write and let syslog use its "last message repeated N times" handling to prevent musb_write's BUG call from filling up the disk.

I've run into this problem in a Debian 5 installation running a backported hplip 3.9.10, but as far as I can tell, Ubuntu's version of hplip and the latest upstream hplip have the same problem.