Merge a bunch of trivial changes from HEAD. The difference remaining

should actual functional differences between HEAD and 3.0.

 - Mostly reformatting
 - Removal of unecessary #include "smb.h"
 - Merge of dyn_DRIVERFILE removal
 - Silly bug fix for python code
(This used to be commit d3998307adc50ba50defe610cb656c73799ae3b9)

source3/auth/auth_builtin.c

@@ -41,8 +41,9 @@ static NTSTATUS check_guest_security(const struct auth_context *auth_context,
 	NTSTATUS nt_status = NT_STATUS_LOGON_FAILURE;
 
 	if (!(user_info->internal_username.str 
-	      && *user_info->internal_username.str)) 
+	      && *user_info->internal_username.str)) {
 		nt_status = make_server_info_guest(server_info);
+	}
 
 	return nt_status;
 }

source3/dynconfig.c

@@ -66,9 +66,5 @@ pstring dyn_LIBDIR = LIBDIR;
 const pstring dyn_LOCKDIR = LOCKDIR;
 const pstring dyn_PIDDIR  = PIDDIR;
 
-const pstring dyn_DRIVERFILE = DRIVERFILE;
-
 const pstring dyn_SMB_PASSWD_FILE = SMB_PASSWD_FILE;
 const pstring dyn_PRIVATE_DIR = PRIVATE_DIR;
-
-

source3/include/dynconfig.h

@@ -32,6 +32,5 @@ extern pstring dyn_LOGFILEBASE, dyn_LMHOSTSFILE;
 extern pstring dyn_LIBDIR;
 extern const pstring dyn_LOCKDIR; 
 extern const pstring dyn_PIDDIR;
-extern const pstring dyn_DRIVERFILE; 
 extern const pstring dyn_SMB_PASSWD_FILE;
 extern const pstring dyn_PRIVATE_DIR;

source3/include/smb.h

@@ -953,7 +953,7 @@ struct bitmap {
 #define SMBntcancel      0xA4   /* NT cancel */
 
 /* These are the trans subcommands */
-#define TRANSACT_SETNAMEDPIPEHANDLESTATE  0x01
+#define TRANSACT_SETNAMEDPIPEHANDLESTATE  0x01 
 #define TRANSACT_DCERPCCMD                0x26
 #define TRANSACT_WAITNAMEDPIPEHANDLESTATE 0x53
 

source3/lib/util_uuid.c

@@ -71,7 +71,7 @@ void uuid_generate_random(GUID *out)
 	uuid_pack(&uu, out);
 }
 
-char *uuid_to_string(const GUID in)
+char *guid_to_string(const GUID in)
 {
 	struct uuid uu;
 	char *out;

source3/libsmb/errormap.c

@@ -1484,7 +1484,6 @@ WERROR ntstatus_to_werror(NTSTATUS error)
 	return W_ERROR(NT_STATUS_V(error) & 0xffff);
 }
 
-
 /* Mapping between Unix, DOS and NT error numbers */
 
 const struct unix_error_map unix_dos_nt_errmap[] = {

source3/printing/pcap.c

@@ -62,8 +62,6 @@
 
 #include "includes.h"
 
-#include "smb.h"
-
 #ifdef AIX
 /*  ******************************************
      Extend for AIX system and qconfig file

source3/printing/print_cups.c

@@ -19,7 +19,6 @@
  */
 
 #include "printing.h"
-#include "smb.h"
 
 #ifdef HAVE_CUPS
 #include <cups/cups.h>

source3/printing/print_svid.c

@@ -33,7 +33,6 @@
 
 
 #include "includes.h"
-#include "smb.h"
 
 #ifdef SYSV
 

source3/python/py_conv.c

@@ -45,7 +45,7 @@ PyObject *from_struct(void *s, struct pyconv *conv)
 			fstring str = "";
 
 			if (u->buffer)
-				fstr_pull(s, u);
+				fstr_pull(str, u);
 
 			item = PyString_FromString(str);
 			PyDict_SetItemString(obj, conv[i].name, item);

source3/rpc_server/srv_reg_nt.c

@@ -131,7 +131,6 @@ static NTSTATUS open_registry_key(pipes_struct *p, POLICY_HND *hnd, REGISTRY_KEY
 	if ( fetch_reg_keys( regkey, &subkeys ) == -1 )  {
 	
 		/* don't really know what to return here */
-		
 		result = NT_STATUS_NO_SUCH_FILE;
 	}
 	else {

source3/rpc_server/srv_samr.c

@@ -155,7 +155,6 @@ static BOOL api_samr_set_sec_obj(pipes_struct *p)
 		return False;
 	}
 	
-	
 	return True;
 }
 

source3/script/uninstallbin.sh

@@ -34,8 +34,8 @@ cat << EOF
 ======================================================================
 The binaries have been uninstalled. You may restore the binaries using
 the command "make installbin" or "make install" to install binaries, 
-man pages and shell scripts. You can restore a previous version of the
-binaries (if there were any) using "make revert".
+man pages, modules and shell scripts. You can restore a previous
+version of the binaries (if there were any) using "make revert".
 ======================================================================
 EOF
 

source3/smbd/build_options.c

@@ -188,7 +188,6 @@ void build_options(BOOL screen)
        output(screen,"   SBINDIR: %s\n", dyn_SBINDIR);
        output(screen,"   BINDIR: %s\n", dyn_BINDIR);
        output(screen,"   LOCKDIR: %s\n",dyn_LOCKDIR);
-       output(screen,"   DRIVERFILE: %s\n", dyn_DRIVERFILE);
        output(screen,"   LOGFILEBASE: %s\n", dyn_LOGFILEBASE);
 
        /*Output various other options (most map to defines in the configure script*/

source3/smbd/change_trust_pw.c

@@ -120,7 +120,6 @@ account password for domain %s.\n", domain));
         if(NT_STATUS_IS_OK(res = modify_trust_password( domain, dc_name,
                                          old_trust_passwd_hash)))
           break;
-
     } else {
 	    res = modify_trust_password( domain, remote_machine,
 					 old_trust_passwd_hash);

source3/utils/net_rpc.c

@@ -1659,6 +1659,10 @@ static int rpc_trustdom_establish(int argc, const char **argv)
 
 	domain_name = smb_xstrdup(argv[0]);
 	strupper(domain_name);
+
+	/* account name used at first is our domain's name with '$' */
+	asprintf(&acct_name, "%s$", lp_workgroup());
+	strupper(acct_name);
 	
 	/*
 	 * opt_workgroup will be used by connection functions further,
@@ -1669,9 +1673,6 @@ static int rpc_trustdom_establish(int argc, const char **argv)
 		opt_workgroup = smb_xstrdup(domain_name);
 	};
 	
-	asprintf(&acct_name, "%s$", lp_workgroup());
-	strupper(acct_name);
-	
 	opt_user_name = acct_name;
 
 	/* find the domain controller */

source3/utils/smbcontrol.c

@@ -470,7 +470,7 @@ static BOOL do_command(char *dest, char *msg_name, int iparams, char **params)
 				JOB_STATUS_DELETED,
 				SPOOLSS_NOTIFY_MSG_UNIX_JOBID);
 		}
-
+		
 		/* printer change notify */
 		
 		if (strequal(cmd, "printer")) {

source3/utils/smbfilter.c

@@ -19,7 +19,6 @@
 */
 
 #include "includes.h"
-#include "smb.h"
 
 #define SECURITY_MASK 0
 #define SECURITY_SET  0

source3/utils/testparm.c

@@ -33,7 +33,6 @@
  */
 
 #include "includes.h"
-#include "smb.h"
 
 extern BOOL AllowDebugChange;
 

source3/utils/testprns.c

@@ -31,7 +31,6 @@
  */
 
 #include "includes.h"
-#include "smb.h"
 
 int main(int argc, char *argv[])
 {