SIGHUP causes segmentation fault #811

Closed
michaelrsweet opened this Issue Jul 13, 2004 · 3 comments

Comments

Projects
None yet
1 participant
Collaborator

michaelrsweet commented Jul 13, 2004

Version: 1.1-current
CUPS.org User: kmuto.debian

I found current 1.1CVS went crash when it got SIGHUP.
Crash point is scheduler/dirsvc.c:1967,
if (p->type & (CUPS_PRINTER_REMOTE | CUPS_PRINTER_IMPLICIT))

In my test and observation, this problem is from dirsvc.c 1.132 change.

Collaborator

michaelrsweet commented Jul 17, 2004

CUPS.org User: mike

Try the attached patch. BTW, the line number you reported is the second-to-last line in the file...

Collaborator

michaelrsweet commented Jul 17, 2004

CUPS.org User: mike

Fixed in CVS - the anonymous CVS repository will be updated at midnight EST.

Collaborator

michaelrsweet commented Aug 11, 2004

"str811.patch":

Index: dirsvc.c

RCS file: /development/cvs/cups/scheduler/dirsvc.c,v
retrieving revision 1.133
diff -u -r1.133 dirsvc.c
--- dirsvc.c 29 Jun 2004 03:27:35 -0000 1.133
+++ dirsvc.c 17 Jul 2004 01:57:58 -0000
@@ -563,7 +563,9 @@
if (!p)
p = Printers;

  •  if (p->type & (CUPS_PRINTER_REMOTE | CUPS_PRINTER_IMPLICIT))
    
  •  if (!p)
    
  •    break;
    
  •  else if (p->type & (CUPS_PRINTER_REMOTE | CUPS_PRINTER_IMPLICIT))
     continue;
    
    else if (p->browse_time < ut)
    {

michaelrsweet added this to the Stable milestone Mar 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment