merging from 3.0

diff --git a/source/client/client.c b/source/client/client.c
index d4ad5fa1fe2b9dd280a45377a48cd2a4e63b4c20..547ebe8029633e9a3ce9402f43f8bc4fe79547f7 100644 (file)
--- a/source/client/client.c
+++ b/source/client/client.c
@@ -2003,7 +2003,7 @@ static void browse_fn(const char *name, uint32 m,
           in any of these fields, they can corrupt the output.  We
           should remove them. */
        if (!grepable) {
-               d_printf("\t%-15.15s%-10.10s%s\n",
+               d_printf("\t%-15s %-10.10s%s\n",
                                name,typestr,comment);
        } else {
                d_printf ("%s|%s|%s\n",typestr,name,comment);
@@ -2018,8 +2018,8 @@ static BOOL browse_host(BOOL sort)
 {
        int ret;
        if (!grepable) {
-               d_printf("\n\tSharename      Type      Comment\n");
-               d_printf("\t---------      ----      -------\n");
+               d_printf("\n\tSharename       Type      Comment\n");
+               d_printf("\t---------       ----      -------\n");
        }
 
        if((ret = cli_RNetShareEnum(cli, browse_fn, NULL)) == -1)
@@ -2037,7 +2037,7 @@ static void server_fn(const char *name, uint32 m,
 {
        
        if (!grepable){
-               d_printf("\t%-16.16s     %s\n", name, comment);
+               d_printf("\t%-16s     %s\n", name, comment);
        } else {
                d_printf("%s|%s|%s\n",(char *)state, name, comment);
        }

diff --git a/source/param/loadparm.c b/source/param/loadparm.c
index 58f4919465e4956bb0441d8a0059496a5488b333..a72f7f878bd2fe67ba221716079eb168c3cc84c0 100644 (file)
--- a/source/param/loadparm.c
+++ b/source/param/loadparm.c
@@ -1176,61 +1176,66 @@ static struct parm_struct parm_table[] = {
  Initialise the sDefault parameter structure for the printer values.
 ***************************************************************************/
 
-static void init_printer_values(void)
+static void init_printer_values(service *pService)
 {
+       if ( pService == NULL ) {
+               DEBUG(0,("init_printer_values: NULL pointer\n"));
+               return;
+       }
+               
        /* choose defaults depending on the type of printing */
-       switch (sDefault.iPrinting) {
+       switch (pService->iPrinting) {
                case PRINT_BSD:
                case PRINT_AIX:
                case PRINT_LPRNT:
                case PRINT_LPROS2:
-                       string_set(&sDefault.szLpqcommand, "lpq -P'%p'");
-                       string_set(&sDefault.szLprmcommand, "lprm -P'%p' %j");
-                       string_set(&sDefault.szPrintcommand,
+                       string_set(&pService->szLpqcommand, "lpq -P'%p'");
+                       string_set(&pService->szLprmcommand, "lprm -P'%p' %j");
+                       string_set(&pService->szPrintcommand,
                                   "lpr -r -P'%p' %s");
                        break;
 
                case PRINT_LPRNG:
                case PRINT_PLP:
-                       string_set(&sDefault.szLpqcommand, "lpq -P'%p'");
-                       string_set(&sDefault.szLprmcommand, "lprm -P'%p' %j");
-                       string_set(&sDefault.szPrintcommand,
+                       string_set(&pService->szLpqcommand, "lpq -P'%p'");
+                       string_set(&pService->szLprmcommand, "lprm -P'%p' %j");
+                       string_set(&pService->szPrintcommand,
                                   "lpr -r -P'%p' %s");
-                       string_set(&sDefault.szQueuepausecommand,
+                       string_set(&pService->szQueuepausecommand,
                                   "lpc stop '%p'");
-                       string_set(&sDefault.szQueueresumecommand,
+                       string_set(&pService->szQueueresumecommand,
                                   "lpc start '%p'");
-                       string_set(&sDefault.szLppausecommand,
+                       string_set(&pService->szLppausecommand,
                                   "lpc hold '%p' %j");
-                       string_set(&sDefault.szLpresumecommand,
+                       string_set(&pService->szLpresumecommand,
                                   "lpc release '%p' %j");
                        break;
 
                case PRINT_CUPS:
 #ifdef HAVE_CUPS
-                       string_set(&sDefault.szLpqcommand, "");
-                       string_set(&sDefault.szLprmcommand, "");
-                       string_set(&sDefault.szPrintcommand, "");
-                       string_set(&sDefault.szLppausecommand, "");
-                       string_set(&sDefault.szLpresumecommand, "");
-                       string_set(&sDefault.szQueuepausecommand, "");
-                       string_set(&sDefault.szQueueresumecommand, "");
+                       string_set(&pService->szLpqcommand, "");
+                       string_set(&pService->szLprmcommand, "");
+                       string_set(&pService->szPrintcommand, "");
+                       string_set(&pService->szLppausecommand, "");
+                       string_set(&pService->szLpresumecommand, "");
+                       string_set(&pService->szQueuepausecommand, "");
+                       string_set(&pService->szQueueresumecommand, "");
 
                        string_set(&Globals.szPrintcapname, "cups");
 #else
-                       string_set(&sDefault.szLpqcommand,
+                       string_set(&pService->szLpqcommand,
                                   "/usr/bin/lpstat -o '%p'");
-                       string_set(&sDefault.szLprmcommand,
+                       string_set(&pService->szLprmcommand,
                                   "/usr/bin/cancel '%p-%j'");
-                       string_set(&sDefault.szPrintcommand,
+                       string_set(&pService->szPrintcommand,
                                   "/usr/bin/lp -d '%p' %s; rm %s");
-                       string_set(&sDefault.szLppausecommand,
+                       string_set(&pService->szLppausecommand,
                                   "lp -i '%p-%j' -H hold");
-                       string_set(&sDefault.szLpresumecommand,
+                       string_set(&pService->szLpresumecommand,
                                   "lp -i '%p-%j' -H resume");
-                       string_set(&sDefault.szQueuepausecommand,
+                       string_set(&pService->szQueuepausecommand,
                                   "/usr/bin/disable '%p'");
-                       string_set(&sDefault.szQueueresumecommand,
+                       string_set(&pService->szQueueresumecommand,
                                   "/usr/bin/enable '%p'");
                        string_set(&Globals.szPrintcapname, "lpstat");
 #endif /* HAVE_CUPS */
@@ -1238,38 +1243,38 @@ static void init_printer_values(void)
 
                case PRINT_SYSV:
                case PRINT_HPUX:
-                       string_set(&sDefault.szLpqcommand, "lpstat -o%p");
-                       string_set(&sDefault.szLprmcommand, "cancel %p-%j");
-                       string_set(&sDefault.szPrintcommand,
+                       string_set(&pService->szLpqcommand, "lpstat -o%p");
+                       string_set(&pService->szLprmcommand, "cancel %p-%j");
+                       string_set(&pService->szPrintcommand,
                                   "lp -c -d%p %s; rm %s");
-                       string_set(&sDefault.szQueuepausecommand,
+                       string_set(&pService->szQueuepausecommand,
                                   "disable %p");
-                       string_set(&sDefault.szQueueresumecommand,
+                       string_set(&pService->szQueueresumecommand,
                                   "enable %p");
 #ifndef HPUX
-                       string_set(&sDefault.szLppausecommand,
+                       string_set(&pService->szLppausecommand,
                                   "lp -i %p-%j -H hold");
-                       string_set(&sDefault.szLpresumecommand,
+                       string_set(&pService->szLpresumecommand,
                                   "lp -i %p-%j -H resume");
 #endif /* HPUX */
                        break;
 
                case PRINT_QNX:
-                       string_set(&sDefault.szLpqcommand, "lpq -P%p");
-                       string_set(&sDefault.szLprmcommand, "lprm -P%p %j");
-                       string_set(&sDefault.szPrintcommand, "lp -r -P%p %s");
+                       string_set(&pService->szLpqcommand, "lpq -P%p");
+                       string_set(&pService->szLprmcommand, "lprm -P%p %j");
+                       string_set(&pService->szPrintcommand, "lp -r -P%p %s");
                        break;
 
 #ifdef DEVELOPER
        case PRINT_TEST:
        case PRINT_VLP:
-               string_set(&sDefault.szPrintcommand, "vlp print %p %s");
-               string_set(&sDefault.szLpqcommand, "vlp lpq %p");
-               string_set(&sDefault.szLprmcommand, "vlp lprm %p %j");
-               string_set(&sDefault.szLppausecommand, "vlp lppause %p %j");
-               string_set(&sDefault.szLpresumecommand, "vlp lpresum %p %j");
-               string_set(&sDefault.szQueuepausecommand, "vlp queuepause %p");
-               string_set(&sDefault.szQueueresumecommand, "vlp queueresume %p");
+               string_set(&pService->szPrintcommand, "vlp print %p %s");
+               string_set(&pService->szLpqcommand, "vlp lpq %p");
+               string_set(&pService->szLprmcommand, "vlp lprm %p %j");
+               string_set(&pService->szLppausecommand, "vlp lppause %p %j");
+               string_set(&pService->szLpresumecommand, "vlp lpresum %p %j");
+               string_set(&pService->szQueuepausecommand, "vlp queuepause %p");
+               string_set(&pService->szQueueresumecommand, "vlp queueresume %p");
                break;
 #endif /* DEVELOPER */
 
@@ -1297,8 +1302,6 @@ static void init_globals(void)
 
                string_set(&sDefault.fstype, FSTYPE_STRING);
 
-               init_printer_values();
-
                done_init = True;
        }
 
@@ -2396,7 +2399,7 @@ BOOL lp_add_printer(const char *pszPrintername, int iDefaultService)
        ServicePtrs[i]->bOpLocks = False;
        /* Printer services must be printable. */
        ServicePtrs[i]->bPrint_ok = True;
-
+       
        DEBUG(3, ("adding printer service %s\n", pszPrintername));
 
        return (True);
@@ -3884,6 +3887,7 @@ BOOL lp_load(const char *pszFname, BOOL global_only, BOOL save_defaults,
        }
 
        init_iconv();
+       init_printer_values(&sDefault);
 
        return (bRetval);
 }