cups crashes on SIGHUP if printer classes are defined

Bug #857663 reported by Ray Link
36
This bug affects 6 people
Affects Status Importance Assigned to Milestone
cups (Ubuntu)
Fix Released
High
Till Kamppeter
Lucid
Fix Released
High
Till Kamppeter

Bug Description

Affects: cups 1.4.3-1ubuntu1.5 on Lucid.

Ubuntu releases with a cups derived from 1.4.4 or later are unaffected.

Known bug in CUPS 1.4.3, fixed in 1.4.4. When cups receives a SIGHUP (for log rotation, restart, etc), the cleanup routines free pointers to printer policy info, but then later try to dereference them.

See attached patch directly from CUPS's svn repository. This patch contains the fix exactly as it made it into mainline cups 1.4.4. Patch has been tested against cups_1.4.3-1ubuntu1.5 and works.

Upstream SVN commit info here: http://www.cups.org/str.php?L3524

Revision history for this message
Ray Link (rlink) wrote :
Ray Link (rlink)
description: updated
description: updated
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "str3524.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Till, you are the one with the most knowledge on cups, can you have a look?

Changed in cups (Ubuntu):
assignee: nobody → Till Kamppeter (till-kamppeter)
Revision history for this message
Till Kamppeter (till-kamppeter) wrote :

Uploaded patched CUPS package to -proposed. As soon as it gets approved it will get available for testing. We will then post a comment with instructions how to test. Please test and report here, so that we can make the fixed CUPS an official update for Lucid.

A debdiff of the changes is attached.

Changed in cups (Ubuntu):
status: New → Fix Committed
importance: Undecided → High
milestone: none → lucid-updates
tags: added: patch-accepted-upstream
removed: patch
Revision history for this message
Chris Halse Rogers (raof) wrote : Please test proposed package

Hello Ray, or anyone else affected,

Accepted cups into lucid-proposed. The package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Revision history for this message
Ray Link (rlink) wrote :

Tested 1.4.3-1ubuntu1.6 from lucid-proposed.

Works fine.

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Till, is this fixed in precise? Its not clear from the bug status.

Changed in cups (Ubuntu Lucid):
status: New → Fix Committed
importance: Undecided → High
assignee: nobody → Till Kamppeter (till-kamppeter)
Changed in cups (Ubuntu):
status: Fix Committed → Incomplete
tags: added: verification-done
removed: verification-needed
Changed in cups (Ubuntu Lucid):
milestone: none → lucid-updates
Revision history for this message
Jeffrey Hutzelman (jhutz) wrote :

This was never in precise; it was fixed upstream in 1.4.4, so doesn't affect anything newer than lucid.

Revision history for this message
Till Kamppeter (till-kamppeter) wrote :

Clint, the patch is a backport from upstream, therefore in Precise it is already included.

Changed in cups (Ubuntu):
status: Incomplete → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package cups - 1.4.3-1ubuntu1.6

---------------
cups (1.4.3-1ubuntu1.6) lucid-proposed; urgency=low

  * debian/patches/cupsd-fix-crash-on-sighup.dpatch: Fixed crash of the
    scheduler (CUPS daemon) on the SIGHUP signal (sent on log rotation,
    restart, ..., LP: #857663).
 -- Till Kamppeter <email address hidden> Wed, 28 Mar 2012 14:36:54 +0200

Changed in cups (Ubuntu Lucid):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.