Eliminate global variable in_client and a plethora of extern declarations.

Derrell

source/client/client.c

@@ -33,7 +33,6 @@ extern int do_smb_browse(void); /* mDNS browsing */
 extern bool AllowDebugChange;
 extern bool override_logfile;
 extern char tar_type;
-extern bool in_client;
 
 static int port = 0;
 static char *service;
@@ -4701,7 +4700,7 @@ static int do_message_op(void)
 	pc = poptGetContext("smbclient", argc, (const char **) argv, long_options, 0);
 	poptSetOtherOptionHelp(pc, "service <password>");
 
-	in_client = true;   /* Make sure that we tell lp_load we are */
+        lp_set_in_client(true); /* Make sure that we tell lp_load we are */
 
 	while ((opt = poptGetNextOpt(pc)) != -1) {
 

source/client/smbspool.c

@@ -35,7 +35,6 @@
  * Globals...
  */
 
-extern bool		in_client;	/* Boolean for client library */
 
 
 /*
@@ -228,7 +227,7 @@ static bool smb_encrypt;
 
   setup_logging("smbspool", True);
 
-  in_client = True;   /* Make sure that we tell lp_load we are */
+  lp_set_in_client(True);       /* Make sure that we tell lp_load we are */
 
   load_case_tables();
 

source/libsmb/libsmb_context.c

@@ -27,8 +27,6 @@
 #include "libsmb_internal.h"
 
 
-extern bool in_client;
-
 /*
  * Is the logging working / configfile read ? 
  */
@@ -450,7 +448,7 @@ smbc_init_context(SMBCCTX *context)
                 
                 /* Here we would open the smb.conf file if needed ... */
                 
-                in_client = True; /* FIXME, make a param */
+                lp_set_in_client(True);
                 
                 home = getenv("HOME");
                 if (home) {

source/pam_smbpass/pam_smb_acct.c

@@ -55,12 +55,11 @@ int pam_sm_acct_mgmt( pam_handle_t *pamh, int flags,
 	const char *name;
 	struct samu *sampass = NULL;
 	void (*oldsig_handler)(int);
-	extern bool in_client;
 
 	/* Samba initialization. */
 	load_case_tables();
 	setup_logging( "pam_smbpass", False );
-	in_client = True;
+        lp_set_in_client(True);
 
 	ctrl = set_ctrl( flags, argc, argv );
 

source/pam_smbpass/pam_smb_auth.c

@@ -72,7 +72,6 @@ int pam_sm_authenticate(pam_handle_t *pamh, int flags,
 	unsigned int ctrl;
 	int retval, *ret_data = NULL;
 	struct samu *sampass = NULL;
-	extern bool in_client;
 	const char *name;
 	void (*oldsig_handler)(int) = NULL;
 	bool found;
@@ -83,7 +82,7 @@ int pam_sm_authenticate(pam_handle_t *pamh, int flags,
 	/* Samba initialization. */
 	load_case_tables();
 	setup_logging("pam_smbpass",False);
-	in_client = True;
+        lp_set_in_client(True);
 
 	ctrl = set_ctrl(flags, argc, argv);
 

source/pam_smbpass/pam_smb_passwd.c

@@ -98,8 +98,6 @@ int pam_sm_chauthtok(pam_handle_t *pamh, int flags,
     unsigned int ctrl;
     int retval;
 
-    extern bool in_client;
-
     struct samu *sampass = NULL;
     void (*oldsig_handler)(int);
     const char *user;
@@ -109,7 +107,7 @@ int pam_sm_chauthtok(pam_handle_t *pamh, int flags,
     /* Samba initialization. */
     load_case_tables();
     setup_logging( "pam_smbpass", False );
-    in_client = True;
+    lp_set_in_client(True);
 
     ctrl = set_ctrl(flags, argc, argv);
 

source/param/loadparm.c

@@ -54,7 +54,6 @@
 #include "includes.h"
 #include "libnet/libnet.h"
 
-bool in_client = False;		/* Not in the client by default */
 bool bLoaded = False;
 
 extern enum protocol_types Protocol;
@@ -72,6 +71,7 @@ extern userdom_struct current_user_info;
 #define HOMES_NAME "homes"
 #endif
 
+static bool in_client = False;		/* Not in the client by default */
 static uint64_t conf_last_seqnum = 0;
 static struct libnet_conf_ctx *conf_ctx = NULL;
 
@@ -5502,6 +5502,27 @@ void gfree_loadparm(void)
 	}
 }
 
+
+/***************************************************************************
+ Allow client apps to specify that they are a client
+***************************************************************************/
+void lp_set_in_client(bool b)
+{
+    in_client = b;
+}
+
+
+/***************************************************************************
+ Determine if we're running in a client app
+***************************************************************************/
+bool lp_is_in_client(void)
+{
+    return in_client;
+}
+
+
+
+
 /***************************************************************************
  Load the services array from the services file. Return True on success, 
  False on failure.
@@ -5608,7 +5629,7 @@ bool lp_load(const char *pszFname,
 
 	/* Now we check bWINSsupport and set szWINSserver to 127.0.0.1 */
 	/* if bWINSsupport is true and we are in the client            */
-	if (in_client && Globals.bWINSsupport) {
+	if (lp_is_in_client() && Globals.bWINSsupport) {
 		lp_do_parameter(GLOBAL_SECTION_SNUM, "wins server", "127.0.0.1");
 	}
 

source/param/params.c

@@ -80,8 +80,6 @@
 
 #include "includes.h"
 
-extern bool in_client;
-
 /* -------------------------------------------------------------------------- **
  * Constants...
  */
@@ -519,7 +517,7 @@ static bool Parse( DATA_BLOB *buf, myFILE *InFile,
 static myFILE *OpenConfFile( const char *FileName )
 {
 	const char *func = "params.c:OpenConfFile() -";
-	int lvl = in_client?1:0;
+	int lvl = lp_is_in_client() ? 1 : 0;
 	myFILE *ret;
 
 	ret = SMB_MALLOC_P(myFILE);