]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
Move standard printcap parsing to print_standard.c
authorSimo Sorce <ssorce@redhat.com>
Thu, 13 May 2010 18:23:23 +0000 (14:23 -0400)
committerGünther Deschner <gd@samba.org>
Fri, 14 May 2010 12:14:55 +0000 (14:14 +0200)
Signed-off-by: Günther Deschner <gd@samba.org>
source3/Makefile.in
source3/include/proto.h
source3/printing/pcap.c
source3/printing/print_standard.c [new file with mode: 0644]

index d661b94bfe57953d1b2b6789300dc3dd45df3e26..968e5ba42a9ce379ae63513b392faec53d6a1be9 100644 (file)
@@ -831,7 +831,7 @@ SMBD_OBJ_BASE = $(PARAM_WITHOUT_REG_OBJ) $(SMBD_OBJ_SRV) $(LIBSMB_OBJ) \
 PRINTING_OBJ = printing/pcap.o printing/print_svid.o printing/print_aix.o \
                printing/print_cups.o printing/print_generic.o \
                printing/lpq_parse.o printing/load.o \
-               printing/print_iprint.o
+               printing/print_iprint.o printing/print_standard.o
 
 PRINTBASE_OBJ = printing/notify.o printing/printing_db.o
 PRINTBACKEND_OBJ = printing/printing.o printing/nt_printing.o $(PRINTBASE_OBJ)
index 0d0864e8f6a1466f9c98d0b9b2e23cf8beeadcfd..75267f820fc66166b1e40b3a11cd82d44fb4851d 100644 (file)
@@ -4928,6 +4928,10 @@ bool iprint_cache_reload(void);
 
 bool sysv_cache_reload(void);
 
+/* The following definitions come from printing/print_standard.c  */
+
+bool std_pcap_cache_reload(const char *pcap_name);
+
 /* The following definitions come from printing/printfsp.c  */
 
 NTSTATUS print_fsp_open(struct smb_request *req, connection_struct *conn,
index a6bf52a0a4ced7559cfa66520f1543ceac82fd88..f1cf58d6bcd9b760df107669bff7f70f594395dc 100644 (file)
 */
 
 /*
- *  This module contains code to parse and cache printcap data, possibly
- *  in concert with the CUPS/SYSV/AIX-specific code found elsewhere.
- *
- *  The way this module looks at the printcap file is very simplistic.
- *  Only the local printcap file is inspected (no searching of NIS
- *  databases etc).
- *
- *  There are assumed to be one or more printer names per record, held
- *  as a set of sub-fields separated by vertical bar symbols ('|') in the
- *  first field of the record. The field separator is assumed to be a colon
- *  ':' and the record separator a newline.
- * 
- *  Lines ending with a backspace '\' are assumed to flag that the following
- *  line is a continuation line so that a set of lines can be read as one
- *  printcap entry.
- *
- *  A line stating with a hash '#' is assumed to be a comment and is ignored
- *  Comments are discarded before the record is strung together from the
- *  set of continuation lines.
- *
- *  Opening a pipe for "lpc status" and reading that would probably 
- *  be pretty effective. Code to do this already exists in the freely
- *  distributable PCNFS server code.
- *
  *  Modified to call SVID/XPG4 support if printcap name is set to "lpstat"
  *  in smb.conf under Solaris.
  *
@@ -130,8 +106,6 @@ void pcap_cache_reload(void)
        const char *pcap_name = lp_printcapname();
        bool pcap_reloaded = False;
        struct pcap_cache *tmp_cache = NULL;
-       XFILE *pcap_file;
-       char *pcap_line;
 
        DEBUG(3, ("reloading printcap cache\n"));
 
@@ -172,72 +146,7 @@ void pcap_cache_reload(void)
        }
 #endif
 
-       /* handle standard printcap - moved from pcap_printer_fn() */
-
-       if ((pcap_file = x_fopen(pcap_name, O_RDONLY, 0)) == NULL) {
-               DEBUG(0, ("Unable to open printcap file %s for read!\n", pcap_name));
-               goto done;
-       }
-
-       for (; (pcap_line = fgets_slash(NULL, 1024, pcap_file)) != NULL; free(pcap_line)) {
-               char name[MAXPRINTERLEN+1];
-               char comment[62];
-               char *p, *q;
-
-               if (*pcap_line == '#' || *pcap_line == 0)
-                       continue;
-
-               /* now we have a real printer line - cut at the first : */      
-               if ((p = strchr_m(pcap_line, ':')) != NULL)
-                       *p = 0;
-      
-               /*
-                * now find the most likely printer name and comment 
-                * this is pure guesswork, but it's better than nothing
-                */
-               for (*name = *comment = 0, p = pcap_line; p != NULL; p = q) {
-                       bool has_punctuation;
-
-                       if ((q = strchr_m(p, '|')) != NULL)
-                               *q++ = 0;
-
-                       has_punctuation = (strchr_m(p, ' ') ||
-                                          strchr_m(p, '\t') ||
-                                          strchr_m(p, '(') ||
-                                          strchr_m(p, ')'));
-
-                       if (strlen(p) > strlen(comment) && has_punctuation) {
-                               strlcpy(comment, p, sizeof(comment));
-                               continue;
-                       }
-
-                       if (strlen(p) <= MAXPRINTERLEN &&
-                           strlen(p) > strlen(name) && !has_punctuation) {
-                               if (!*comment) {
-                                       strlcpy(comment, name, sizeof(comment));
-                               }
-                               strlcpy(name, p, sizeof(name));
-                               continue;
-                       }
-
-                       if (!strchr_m(comment, ' ') &&
-                           strlen(p) > strlen(comment)) {
-                               strlcpy(comment, p, sizeof(comment));
-                               continue;
-                       }
-               }
-
-               comment[60] = 0;
-               name[MAXPRINTERLEN] = 0;
-
-               if (*name && !pcap_cache_add(name, comment)) {
-                       x_fclose(pcap_file);
-                       goto done;
-               }
-       }
-
-       x_fclose(pcap_file);
-       pcap_reloaded = True;
+       pcap_reloaded = std_pcap_cache_reload(pcap_name);
 
 done:
        DEBUG(3, ("reload status: %s\n", (pcap_reloaded) ? "ok" : "error"));
diff --git a/source3/printing/print_standard.c b/source3/printing/print_standard.c
new file mode 100644 (file)
index 0000000..91d1c33
--- /dev/null
@@ -0,0 +1,129 @@
+/*
+   Unix SMB/CIFS implementation.
+   printcap parsing
+   Copyright (C) Karl Auer 1993-1998
+
+   Re-working by Martin Kiff, 1994
+
+   Re-written again by Andrew Tridgell
+
+   Modified for SVID support by Norm Jacobs, 1997
+
+   Modified for CUPS support by Michael Sweet, 1999
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+/*
+ *  This module contains code to parse and cache printcap data, possibly
+ *  in concert with the CUPS/SYSV/AIX-specific code found elsewhere.
+ *
+ *  The way this module looks at the printcap file is very simplistic.
+ *  Only the local printcap file is inspected (no searching of NIS
+ *  databases etc).
+ *
+ *  There are assumed to be one or more printer names per record, held
+ *  as a set of sub-fields separated by vertical bar symbols ('|') in the
+ *  first field of the record. The field separator is assumed to be a colon
+ *  ':' and the record separator a newline.
+ *
+ *  Lines ending with a backspace '\' are assumed to flag that the following
+ *  line is a continuation line so that a set of lines can be read as one
+ *  printcap entry.
+ *
+ *  A line stating with a hash '#' is assumed to be a comment and is ignored
+ *  Comments are discarded before the record is strung together from the
+ *  set of continuation lines.
+ *
+ *  Opening a pipe for "lpc status" and reading that would probably
+ *  be pretty effective. Code to do this already exists in the freely
+ *  distributable PCNFS server code.
+ */
+
+/* printcap parsing specific code moved here from printing/pcap.c */
+
+
+#include "includes.h"
+
+/* handle standard printcap - moved from pcap_printer_fn() */
+bool std_pcap_cache_reload(const char *pcap_name)
+{
+       XFILE *pcap_file;
+       char *pcap_line;
+
+       if ((pcap_file = x_fopen(pcap_name, O_RDONLY, 0)) == NULL) {
+               DEBUG(0, ("Unable to open printcap file %s for read!\n", pcap_name));
+               return false;
+       }
+
+       for (; (pcap_line = fgets_slash(NULL, 1024, pcap_file)) != NULL; free(pcap_line)) {
+               char name[MAXPRINTERLEN+1];
+               char comment[62];
+               char *p, *q;
+
+               if (*pcap_line == '#' || *pcap_line == 0)
+                       continue;
+
+               /* now we have a real printer line - cut at the first : */
+               if ((p = strchr_m(pcap_line, ':')) != NULL)
+                       *p = 0;
+
+               /*
+                * now find the most likely printer name and comment
+                * this is pure guesswork, but it's better than nothing
+                */
+               for (*name = *comment = 0, p = pcap_line; p != NULL; p = q) {
+                       bool has_punctuation;
+
+                       if ((q = strchr_m(p, '|')) != NULL)
+                               *q++ = 0;
+
+                       has_punctuation = (strchr_m(p, ' ') ||
+                                          strchr_m(p, '\t') ||
+                                          strchr_m(p, '(') ||
+                                          strchr_m(p, ')'));
+
+                       if (strlen(p) > strlen(comment) && has_punctuation) {
+                               strlcpy(comment, p, sizeof(comment));
+                               continue;
+                       }
+
+                       if (strlen(p) <= MAXPRINTERLEN &&
+                           strlen(p) > strlen(name) && !has_punctuation) {
+                               if (!*comment) {
+                                       strlcpy(comment, name, sizeof(comment));
+                               }
+                               strlcpy(name, p, sizeof(name));
+                               continue;
+                       }
+
+                       if (!strchr_m(comment, ' ') &&
+                           strlen(p) > strlen(comment)) {
+                               strlcpy(comment, p, sizeof(comment));
+                               continue;
+                       }
+               }
+
+               comment[60] = 0;
+               name[MAXPRINTERLEN] = 0;
+
+               if (*name && !pcap_cache_add(name, comment)) {
+                       x_fclose(pcap_file);
+                       return false;
+               }
+       }
+
+       x_fclose(pcap_file);
+       return true;
+}