Attachment 310831 Details for Bug 314941 – cups-fatalerrors.patch

[patch] cups-fatalerrors.patch

cups-fatalerrors.patch (text/plain), 10.03 KB, created by Tim Waugh on 2008-07-02 16:49:25 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tim Waugh

Created: 2008-07-02 16:49:25 UTC

Size: 10.03 KB

patch

obsolete

>diff -up cups-1.3.7/config.h.in.fatalerrors cups-1.3.7/config.h.in
>--- cups-1.3.7/config.h.in.fatalerrors	2008-01-07 18:26:57.000000000 +0000
>+++ cups-1.3.7/config.h.in	2008-07-02 17:46:27.000000000 +0100
>@@ -84,6 +84,13 @@
> 
> 
> /*
>+ * Errors to treat as fatal...
>+ */
>+
>+#define CUPS_DEFAULT_FATAL_ERRORS	"none"
>+
>+
>+/*
>  * Where are files stored?
>  *
>  * Note: These are defaults, which can be overridden by environment
>diff -up cups-1.3.7/scheduler/conf.c.fatalerrors cups-1.3.7/scheduler/conf.c
>--- cups-1.3.7/scheduler/conf.c.fatalerrors	2008-07-02 17:46:27.000000000 +0100
>+++ cups-1.3.7/scheduler/conf.c	2008-07-02 17:46:58.000000000 +0100
>@@ -23,6 +23,7 @@
>  *                              access control lines.
>  *   parse_groups()           - Parse system group names in a string.
>  *   parse_protocols()        - Parse browse protocols in a string.
>+ *   parse_fatalerrors()      - Parse fatal errors in a string.
>  *   read_configuration()     - Read a configuration file.
>  *   read_location()          - Read a <Location path> definition.
>  *   read_policy()            - Read a <Policy name> definition.
>@@ -194,6 +195,7 @@ static int		parse_aaa(cupsd_location_t *
> 			          char *value, int linenum);
> static int		parse_groups(const char *s);
> static int		parse_protocols(const char *s);
>+static int		parse_fatalerrors(const char *s);
> static int		read_configuration(cups_file_t *fp);
> static int		read_location(cups_file_t *fp, char *name, int linenum);
> static int		read_policy(cups_file_t *fp, char *name, int linenum);
>@@ -549,6 +551,7 @@ cupsdReadConfiguration(void)
>   BrowseTimeout         = DEFAULT_TIMEOUT;
>   Browsing              = CUPS_DEFAULT_BROWSING;
>   DefaultShared         = CUPS_DEFAULT_DEFAULT_SHARED;
>+  FatalErrors		= parse_fatalerrors(CUPS_DEFAULT_FATAL_ERRORS);
> 
>   cupsdClearString(&BrowseLocalOptions);
>   cupsdClearString(&BrowseRemoteOptions);
>@@ -2053,6 +2056,68 @@ parse_protocols(const char *s)		/* I - S
> 
> 
> /*
>+ * 'parse_fatalerrors()' - Parse fatal errors in a string.
>+ */
>+
>+static int				/* O - Fatal error bits */
>+parse_fatalerrors(const char *s)	/* I - Space-delimited errors */
>+{
>+  int	errors;				/* Fatal error bits */
>+  char	value[1024],			/* Value string */
>+	*valstart,			/* Pointer into value */
>+	*valend;			/* End of value */
>+
>+
>+ /*
>+  * Empty protocol line yields NULL pointer...
>+  */
>+
>+  if (!s)
>+    return (0);
>+
>+ /*
>+  * Loop through the value string,...
>+  */
>+
>+  strlcpy(value, s, sizeof(value));
>+
>+  errors = 0;
>+
>+  for (valstart = value; *valstart;)
>+  {
>+   /*
>+    * Get the current space/comma-delimited error name...
>+    */
>+
>+    for (valend = valstart; *valend; valend ++)
>+      if (isspace(*valend & 255) || *valend == ',')
>+	break;
>+
>+    if (*valend)
>+      *valend++ = '\0';
>+
>+   /*
>+    * Add the error to the bitmask...
>+    */
>+
>+    if (!strcasecmp(valstart, "browse"))
>+      errors |= FATAL_ERROR_BROWSE;
>+    else if (!strcasecmp(valstart, "all"))
>+      errors |= FATAL_ERROR_ALL;
>+    else if (strcasecmp(valstart, "none"))
>+      cupsdLogMessage(CUPSD_LOG_ERROR,
>+                      "Unknown error \"%s\" ignored!", valstart);
>+
>+    for (valstart = valend; *valstart; valstart ++)
>+      if (!isspace(*valstart & 255) || *valstart != ',')
>+	break;
>+  }
>+
>+  return (errors);
>+}
>+
>+
>+/*
>  * 'read_configuration()' - Read a configuration file.
>  */
> 
>@@ -3025,6 +3090,24 @@ read_configuration(cups_file_t *fp)	/* I
> 	                "Missing value for SetEnv directive on line %d.",
> 	                linenum);
>     }
>+    else if (!strcasecmp(line, "FatalErrors"))
>+    {
>+     /*
>+      * "FatalErrors name [... name]"
>+      */
>+
>+      int errors = parse_fatalerrors(value);
>+
>+      if (errors < 0)
>+      {
>+	cupsdLogMessage(CUPSD_LOG_ERROR,
>+	                "Unknown error type \"%s\" on line %d.",
>+	                value, linenum);
>+        break;
>+      }
>+
>+      FatalErrors = errors;
>+    }
>     else
>     {
>      /*
>diff -up cups-1.3.7/scheduler/cupsd.h.fatalerrors cups-1.3.7/scheduler/cupsd.h
>--- cups-1.3.7/scheduler/cupsd.h.fatalerrors	2008-02-15 23:26:51.000000000 +0000
>+++ cups-1.3.7/scheduler/cupsd.h	2008-07-02 17:46:27.000000000 +0100
>@@ -178,6 +178,9 @@ VAR PSQUpdateQuotaProcPtr PSQUpdateQuota
> 					/* Apple PrintService quota function */
> #endif /* __APPLE__ && HAVE_DLFCN_H */
> 
>+#define FATAL_ERROR_BROWSE	1	/* Bind to a browse socket */
>+#define FATAL_ERROR_ALL		(FATAL_ERROR_BROWSE)
>+VAR int			FatalErrors	VALUE(FATAL_ERROR_ALL);
> 
> 
> 
>diff -up cups-1.3.7/scheduler/dirsvc.c.fatalerrors cups-1.3.7/scheduler/dirsvc.c
>--- cups-1.3.7/scheduler/dirsvc.c.fatalerrors	2008-01-21 22:56:06.000000000 +0000
>+++ cups-1.3.7/scheduler/dirsvc.c	2008-07-02 17:46:27.000000000 +0100
>@@ -927,18 +927,19 @@ cupsdSendBrowseList(void)
>  * 'cupsdStartBrowsing()' - Start sending and receiving broadcast information.
>  */
> 
>-void
>+int
> cupsdStartBrowsing(void)
> {
>   int			val;		/* Socket option value */
>   struct sockaddr_in	addr;		/* Broadcast address */
>   cupsd_printer_t	*p;		/* Current printer */
>+  int			fail = 0;	/* Whether anything failed */
> 
> 
>   BrowseNext = NULL;
> 
>   if (!Browsing || !(BrowseLocalProtocols | BrowseRemoteProtocols))
>-    return;
>+    return (0);
> 
>   if ((BrowseLocalProtocols | BrowseRemoteProtocols) & BROWSE_CUPS)
>   {
>@@ -955,7 +956,7 @@ cupsdStartBrowsing(void)
> 			strerror(errno));
> 	BrowseLocalProtocols &= ~BROWSE_CUPS;
> 	BrowseRemoteProtocols &= ~BROWSE_CUPS;
>-	return;
>+	return (1);
>       }
> 
>      /*
>@@ -982,7 +983,7 @@ cupsdStartBrowsing(void)
> 	BrowseSocket = -1;
> 	BrowseLocalProtocols &= ~BROWSE_CUPS;
> 	BrowseRemoteProtocols &= ~BROWSE_CUPS;
>-	return;
>+	return (1);
>       }
>     }
> 
>@@ -1005,7 +1006,7 @@ cupsdStartBrowsing(void)
>       BrowseSocket = -1;
>       BrowseLocalProtocols &= ~BROWSE_CUPS;
>       BrowseRemoteProtocols &= ~BROWSE_CUPS;
>-      return;
>+      return (1);
>     }
> 
>    /*
>@@ -1044,6 +1045,7 @@ cupsdStartBrowsing(void)
>                       "Unable to open an SLP handle; disabling SLP browsing!");
>       BrowseLocalProtocols &= ~BROWSE_SLP;
>       BrowseRemoteProtocols &= ~BROWSE_SLP;
>+      fail = 1;
>     }
> 
>     BrowseSLPRefresh = 0;
>@@ -1061,6 +1063,7 @@ cupsdStartBrowsing(void)
>                       "Need to set BrowseLDAPDN to use LDAP browsing!");
>       BrowseLocalProtocols &= ~BROWSE_LDAP;
>       BrowseRemoteProtocols &= ~BROWSE_LDAP;
>+      fail = 1;
>     }
>     else
>     {
>@@ -1086,9 +1089,12 @@ cupsdStartBrowsing(void)
>         if ((rc = ldap_set_option(NULL, LDAP_OPT_X_TLS_CACERTFILE,
> 	                          (void *)BrowseLDAPCACertFile))
> 	        != LDAP_SUCCESS)
>+	{
>           cupsdLogMessage(CUPSD_LOG_ERROR,
>                           "Unable to set CA certificate file for LDAP "
> 			  "connections: %d - %s", rc, ldap_err2string(rc));
>+	  fail = 1;
>+	}
>       }
> 
>      /*
>@@ -1106,6 +1112,7 @@ cupsdStartBrowsing(void)
> 	                "Unable to initialize LDAP; disabling LDAP browsing!");
> 	BrowseLocalProtocols &= ~BROWSE_LDAP;
> 	BrowseRemoteProtocols &= ~BROWSE_LDAP;
>+	fail = 1;
>       }
>       else if (ldap_set_option(BrowseLDAPHandle, LDAP_OPT_PROTOCOL_VERSION,
>                                (const void *)&version) != LDAP_SUCCESS)
>@@ -1117,6 +1124,7 @@ cupsdStartBrowsing(void)
> 			"disabling LDAP browsing!");
> 	BrowseLocalProtocols &= ~BROWSE_LDAP;
> 	BrowseRemoteProtocols &= ~BROWSE_LDAP;
>+	fail = 1;
>       }
>       else
>       {
>@@ -1135,6 +1143,7 @@ cupsdStartBrowsing(void)
> 	  ldap_unbind_ext(BrowseLDAPHandle, NULL, NULL);
> 	  BrowseLocalProtocols &= ~BROWSE_LDAP;
> 	  BrowseRemoteProtocols &= ~BROWSE_LDAP;
>+	  fail = 1;
> 	}
>       }
>     }
>@@ -1152,6 +1161,8 @@ cupsdStartBrowsing(void)
>        p = (cupsd_printer_t *)cupsArrayNext(Printers))
>     if (!(p->type & (CUPS_PRINTER_REMOTE | CUPS_PRINTER_IMPLICIT)))
>       cupsdRegisterPrinter(p);
>+
>+  return (fail);
> }
> 
> 
>@@ -1159,7 +1170,7 @@ cupsdStartBrowsing(void)
>  * 'cupsdStartPolling()' - Start polling servers as needed.
>  */
> 
>-void
>+int
> cupsdStartPolling(void)
> {
>   int			i;		/* Looping var */
>@@ -1171,6 +1182,7 @@ cupsdStartPolling(void)
>   int			statusfds[2];	/* Status pipe */
>   char			*argv[6];	/* Arguments */
>   char			*envp[100];	/* Environment */
>+  int			fail = 0;	/* Whether anything failed */
> 
> 
>  /*
>@@ -1181,7 +1193,7 @@ cupsdStartPolling(void)
>   {
>     PollPipe         = -1;
>     PollStatusBuffer = NULL;
>-    return;
>+    return (0);
>   }
> 
>  /*
>@@ -1217,7 +1229,7 @@ cupsdStartPolling(void)
> 	            strerror(errno));
>     PollPipe         = -1;
>     PollStatusBuffer = NULL;
>-    return;
>+    return (1);
>   }
> 
>   PollPipe         = statusfds[0];
>@@ -1240,6 +1252,7 @@ cupsdStartPolling(void)
>                       "cupsdStartPolling: Unable to fork polling daemon - %s",
>                       strerror(errno));
>       pollp->pid = 0;
>+      fail = 1;
>       break;
>     }
>     else
>@@ -1255,6 +1268,7 @@ cupsdStartPolling(void)
>   */
> 
>   cupsdAddSelect(PollPipe, (cupsd_selfunc_t)update_polling, NULL, NULL);
>+  return (fail);
> }
> 
> 
>diff -up cups-1.3.7/scheduler/dirsvc.h.fatalerrors cups-1.3.7/scheduler/dirsvc.h
>--- cups-1.3.7/scheduler/dirsvc.h.fatalerrors	2007-07-11 22:46:42.000000000 +0100
>+++ cups-1.3.7/scheduler/dirsvc.h	2008-07-02 17:46:27.000000000 +0100
>@@ -162,8 +162,8 @@ extern void	cupsdRegisterPrinter(cupsd_p
> extern void	cupsdRestartPolling(void);
> extern void	cupsdSaveRemoteCache(void);
> extern void	cupsdSendBrowseList(void);
>-extern void	cupsdStartBrowsing(void);
>-extern void	cupsdStartPolling(void);
>+extern int	cupsdStartBrowsing(void);
>+extern int	cupsdStartPolling(void);
> extern void	cupsdStopBrowsing(void);
> extern void	cupsdStopPolling(void);
> #ifdef HAVE_DNSSD
>diff -up cups-1.3.7/scheduler/server.c.fatalerrors cups-1.3.7/scheduler/server.c
>--- cups-1.3.7/scheduler/server.c.fatalerrors	2007-07-11 22:46:42.000000000 +0100
>+++ cups-1.3.7/scheduler/server.c	2008-07-02 17:46:27.000000000 +0100
>@@ -84,8 +84,12 @@ cupsdStartServer(void)
>   */
> 
>   cupsdStartListening();
>-  cupsdStartBrowsing();
>-  cupsdStartPolling();
>+  if (cupsdStartBrowsing() != 0 &&
>+      (FatalErrors & FATAL_ERROR_BROWSE))
>+    exit (EXIT_FAILURE);
>+  if (cupsdStartPolling() != 0 &&
>+      (FatalErrors & FATAL_ERROR_BROWSE))
>+    exit (EXIT_FAILURE);
> 
>  /*
>   * Create a pipe for CGI processes...

Actions: View | Diff

Attachments on bug 314941: 310824 | 310831