svn commit: samba r12730 - in branches/SAMBA_4_0/source: param utils

[abartlet at samba.org]

Author: abartlet
Date: 2006-01-06 02:28:36 +0000 (Fri, 06 Jan 2006)
New Revision: 12730

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=12730

Log:
Reimplement --parameter-name, and bring in common samba options.

This changes -s from meaning 'suppress prompt' to 'services file'.

Andrew Bartlett

Modified:
   branches/SAMBA_4_0/source/param/loadparm.c
   branches/SAMBA_4_0/source/utils/testparm.c


Changeset:
Modified: branches/SAMBA_4_0/source/param/loadparm.c
===================================================================
--- branches/SAMBA_4_0/source/param/loadparm.c	2006-01-06 02:13:01 UTC (rev 12729)
+++ branches/SAMBA_4_0/source/param/loadparm.c	2006-01-06 02:28:36 UTC (rev 12730)
@@ -2228,7 +2228,45 @@
         }
 }
 
+BOOL lp_dump_a_parameter(int snum, char *parm_name, FILE * f, BOOL isGlobal)
+{
+	service * pService = ServicePtrs[snum];
+	int i, result = False;
+	parm_class p_class;
+	unsigned flag = 0;
 
+	if (isGlobal) {
+		p_class = P_GLOBAL;
+		flag = FLAG_GLOBAL;
+	} else
+		p_class = P_LOCAL;
+	
+	for (i = 0; parm_table[i].label; i++) {
+		if (strwicmp(parm_table[i].label, parm_name) == 0 &&
+		    (parm_table[i].class == p_class || parm_table[i].flags & flag) &&
+		    parm_table[i].ptr &&
+		    (*parm_table[i].label != '-') &&
+		    (i == 0 || (parm_table[i].ptr != parm_table[i - 1].ptr))) 
+		{
+			void *ptr;
+
+			if (isGlobal)
+				ptr = parm_table[i].ptr;
+			else
+				ptr = ((char *)pService) +
+					PTR_DIFF(parm_table[i].ptr, &sDefault);
+
+			print_parameter(&parm_table[i],
+					ptr, f);
+			fprintf(f, "\n");
+			result = True;
+			break;
+		}
+	}
+
+	return result;
+}
+
 /***************************************************************************
  Return info about the next service  in a service. snum==-1 gives the globals.
  Return NULL when out of parameters.

Modified: branches/SAMBA_4_0/source/utils/testparm.c
===================================================================
--- branches/SAMBA_4_0/source/utils/testparm.c	2006-01-06 02:13:01 UTC (rev 12729)
+++ branches/SAMBA_4_0/source/utils/testparm.c	2006-01-06 02:28:36 UTC (rev 12730)
@@ -81,17 +81,17 @@
 	static const char *term_code = "";
 /*
 	static BOOL show_all_parameters = False;
-	static char *parameter_name = NULL;
 	static char *new_local_machine = NULL;
 */
 	static const char *section_name = NULL;
+	static char *parameter_name = NULL;
 	static const char *cname;
 	static const char *caddr;
 	static int show_defaults;
 
 	struct poptOption long_options[] = {
 		POPT_AUTOHELP
-		{"suppress-prompt", 's', POPT_ARG_VAL, &silent_mode, 1, "Suppress prompt for enter"},
+		{"suppress-prompt", '\0', POPT_ARG_VAL, &silent_mode, 1, "Suppress prompt for enter"},
 		{"verbose", 'v', POPT_ARG_NONE, &show_defaults, 1, "Show default options too"},
 /*
   We need support for smb.conf macros before this will work again 
@@ -100,11 +100,12 @@
 /*
   These are harder to do with the new code structure
 		{"show-all-parameters", '\0', POPT_ARG_VAL, &show_all_parameters, True, "Show the parameters, type, possible values" },
-		{"parameter-name", '\0', POPT_ARG_STRING, &parameter_name, 0, "Limit testparm to a named parameter" },
 */
 		{"section-name", '\0', POPT_ARG_STRING, &section_name, 0, "Limit testparm to a named section" },
+		{"parameter-name", '\0', POPT_ARG_STRING, &parameter_name, 0, "Limit testparm to a named parameter" },
 		{"client-name", '\0', POPT_ARG_STRING, &cname, 0, "Client DNS name for 'hosts allow' checking (should match reverse lookup)"},
 		{"client-ip", '\0', POPT_ARG_STRING, &caddr, 0, "Client IP address for 'hosts allow' checking"},
+		POPT_COMMON_SAMBA
 		POPT_COMMON_VERSION
 		POPT_TABLEEND
 	};
@@ -196,7 +197,7 @@
 			fflush(stdout);
 			getc(stdin);
 		}
-		if (section_name) {
+		if (section_name || parameter_name) {
 			BOOL isGlobal = False;
 			if (!section_name) {
 				section_name = GLOBAL_NAME;
@@ -207,10 +208,14 @@
 						section_name);
 					return(1);
 			}
-			if (isGlobal == True) {
-				lp_dump(stdout, show_defaults, 0);
+			if (!parameter_name) {
+				if (isGlobal == True) {
+					lp_dump(stdout, show_defaults, 0);
+				} else {
+					lp_dump_one(stdout, show_defaults, s);
+				}
 			} else {
-				lp_dump_one(stdout, show_defaults, s);
+				ret = lp_dump_a_parameter(s, parameter_name, stdout, isGlobal);
 			}
 		} else {
 			lp_dump(stdout, show_defaults, lp_numservices());