Author: metze Date: 2007-08-22 12:06:27 +0000 (Wed, 22 Aug 2007) New Revision: 24621
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=24621 Log: - deferr calling build_options();exit(0); - use poptPrintUsage() to give the user more info metze Modified: branches/SAMBA_3_2/source/nmbd/nmbd.c branches/SAMBA_3_2/source/nsswitch/winbindd.c branches/SAMBA_3_2/source/smbd/server.c Changeset: Modified: branches/SAMBA_3_2/source/nmbd/nmbd.c =================================================================== --- branches/SAMBA_3_2/source/nmbd/nmbd.c 2007-08-22 12:04:42 UTC (rev 24620) +++ branches/SAMBA_3_2/source/nmbd/nmbd.c 2007-08-22 12:06:27 UTC (rev 24621) @@ -678,8 +678,9 @@ while ((opt = poptGetNextOpt(pc)) != -1) { switch (opt) { default: - d_fprintf(stderr, "\nInvalid option %s: %s\n", + d_fprintf(stderr, "\nInvalid option %s: %s\n\n", poptBadOption(pc, 0), poptStrerror(opt)); + poptPrintUsage(pc, stderr, 0); exit(1); } }; Modified: branches/SAMBA_3_2/source/nsswitch/winbindd.c =================================================================== --- branches/SAMBA_3_2/source/nsswitch/winbindd.c 2007-08-22 12:04:42 UTC (rev 24620) +++ branches/SAMBA_3_2/source/nsswitch/winbindd.c 2007-08-22 12:06:27 UTC (rev 24621) @@ -1017,8 +1017,9 @@ while ((opt = poptGetNextOpt(pc)) != -1) { switch (opt) { default: - d_fprintf(stderr, "\nInvalid option %s: %s\n", + d_fprintf(stderr, "\nInvalid option %s: %s\n\n", poptBadOption(pc, 0), poptStrerror(opt)); + poptPrintUsage(pc, stderr, 0); exit(1); } } @@ -1036,6 +1037,8 @@ exit(1); } + poptFreeContext(pc); + if (!override_logfile) { pstr_sprintf(logfile, "%s/log.winbindd", dyn_LOGFILEBASE); lp_set_logfile(logfile); @@ -1167,8 +1170,6 @@ messaging_register(winbind_messaging_context(), NULL, MSG_DUMP_EVENT_LIST, winbind_msg_dump_event_list); - poptFreeContext(pc); - netsamlogon_cache_init(); /* Non-critical */ /* clear the cached list of trusted domains */ Modified: branches/SAMBA_3_2/source/smbd/server.c =================================================================== --- branches/SAMBA_3_2/source/smbd/server.c 2007-08-22 12:04:42 UTC (rev 24620) +++ branches/SAMBA_3_2/source/smbd/server.c 2007-08-22 12:06:27 UTC (rev 24621) @@ -818,6 +818,7 @@ const char *profile_level = NULL; int opt; poptContext pc; + BOOL print_build_options = False; enum smb_server_mode server_mode = SERVER_MODE_DAEMON; @@ -849,21 +850,25 @@ #endif pc = poptGetContext("smbd", argc, argv, long_options, 0); - while((opt = poptGetNextOpt(pc)) != -1) { switch (opt) { case 'b': - build_options(True); /* Display output to screen as well as debug */ - exit(0); + print_build_options = True; + break; default: - d_fprintf(stderr, "\nInvalid option %s: %s\n", + d_fprintf(stderr, "\nInvalid option %s: %s\n\n", poptBadOption(pc, 0), poptStrerror(opt)); + poptPrintUsage(pc, stderr, 0); exit(1); } } - poptFreeContext(pc); + if (print_build_options) { + build_options(True); /* Display output to screen as well as debug */ + exit(0); + } + #ifdef HAVE_SETLUID /* needed for SecureWare on SCO */ setluid(0);
