[SCM] Samba Shared Repository - branch master updated

Stefan Metzmacher metze at samba.org
Fri Aug 10 09:36:03 MDT 2012


The branch, master has been updated
       via  8defcb8 Revert "s3:smbd: include smbXsrv.h before smbd/proto.h to have the smbXsrv_ structs available"
       via  0e76bbc Revert "s3:smbd: Include smbXsrv.h before vfs.h (in smbd.h) so that the smbXsrv structures are available"
       via  2cbfdd4 Revert "s3:smb: include "smbXsrv.h" before "vfs.h""
       via  205185e s3:smbXsrv.idl: remove smbXsrv_*0 defines
       via  2b41f37 s3:param: fix compiler warnings with FN_GLOBAL_CONST_STRING()
      from  13f8674 build: rename security → samba-security

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 8defcb8bd1292376c2c00f1d432fe751c207f872
Author: Stefan Metzmacher <metze at samba.org>
Date:   Fri Aug 10 11:58:39 2012 +0200

    Revert "s3:smbd: include smbXsrv.h before smbd/proto.h to have the smbXsrv_ structs available"
    
    This reverts commit 98ccca8dca70b87d04a93c8ef5232a071ab7c2af.
    
    Autobuild-User(master): Stefan Metzmacher <metze at samba.org>
    Autobuild-Date(master): Fri Aug 10 17:35:38 CEST 2012 on sn-devel-104

commit 0e76bbc520b0052f1fed6bbd17fe8737249e8e68
Author: Stefan Metzmacher <metze at samba.org>
Date:   Fri Aug 10 11:56:21 2012 +0200

    Revert "s3:smbd: Include smbXsrv.h before vfs.h (in smbd.h) so that the smbXsrv structures are available"
    
    This reverts commit e332bfaff51e54638bd37cd1fe08e57608e16e86.

commit 2cbfdd433e208a53bc8d8b959fbe23303fc60492
Author: Stefan Metzmacher <metze at samba.org>
Date:   Fri Aug 10 11:58:28 2012 +0200

    Revert "s3:smb: include "smbXsrv.h" before "vfs.h""
    
    This reverts commit db0c233624e633b3cc1a6e0e44dccc09aaa121f2.

commit 205185e88c8724e672675f893b386a57f2b8547d
Author: Stefan Metzmacher <metze at samba.org>
Date:   Fri Aug 10 11:55:13 2012 +0200

    s3:smbXsrv.idl: remove smbXsrv_*0 defines
    
    This makes ctags more usable.
    
    metze

commit 2b41f3702fd7f46696bf6eaf96ad1a58b797ec07
Author: Stefan Metzmacher <metze at samba.org>
Date:   Fri Aug 10 12:05:15 2012 +0200

    s3:param: fix compiler warnings with FN_GLOBAL_CONST_STRING()
    
    metze

-----------------------------------------------------------------------

Summary of changes:
 source3/include/smb.h          |    6 ---
 source3/librpc/idl/smbXsrv.idl |   84 ++++++++++++++++++++--------------------
 source3/param/loadparm.c       |    2 +-
 source3/smbd/smbd.h            |    1 -
 4 files changed, 43 insertions(+), 50 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/include/smb.h b/source3/include/smb.h
index c6e6fb3..2aa2ab3 100644
--- a/source3/include/smb.h
+++ b/source3/include/smb.h
@@ -154,12 +154,6 @@ struct sys_notify_context {
 /* Include VFS stuff */
 
 #include "smb_acls.h"
-/*
- * smbXsrv.h: currently needed for vfs.h, as long as
- * the smbXsrv structures are still referenced as a
- * backling from files_struct and connection_struct.
- */
-#include "librpc/gen_ndr/smbXsrv.h"
 #include "vfs.h"
 
 struct current_user {
diff --git a/source3/librpc/idl/smbXsrv.idl b/source3/librpc/idl/smbXsrv.idl
index 9111b3d..b3f2250 100644
--- a/source3/librpc/idl/smbXsrv.idl
+++ b/source3/librpc/idl/smbXsrv.idl
@@ -4,42 +4,6 @@ import "server_id.idl";
 import "security.idl";
 import "auth.idl";
 
-/*
- * The main server code should just work with
- * 'struct smbXsrv_session' and never use
- * smbXsrv_session0, smbXsrv_sessionU
- * and smbXsrv_sessionB directly.
- *
- * If we need to change the smbXsrv_session,
- * we can just point it to smbXsrv_session1
- * and could implement transparent mapping.
- */
-cpp_quote("#define smbXsrv_session smbXsrv_session0")
-
-/*
- * The main server code should just work with
- * 'struct smbXsrv_tcon' and never use
- * smbXsrv_tcon0, smbXsrv_tconU
- * and smbXsrv_tconB directly.
- *
- * If we need to change the smbXsrv_tcon,
- * we can just point it to smbXsrv_tcon1
- * and could implement transparent mapping.
- */
-cpp_quote("#define smbXsrv_tcon smbXsrv_tcon0")
-
-/*
- * The main server code should just work with
- * 'struct smbXsrv_open' and never use
- * smbXsrv_open0, smbXsrv_openU
- * and smbXsrv_openB directly.
- *
- * If we need to change the smbXsrv_open,
- * we can just point it to smbXsrv_open1
- * and could implement transparent mapping.
- */
-cpp_quote("#define smbXsrv_open smbXsrv_open0")
-
 [
 	uuid("07408340-ae31-11e1-97dc-539f7fddc06f"),
 	version(0.0),
@@ -161,6 +125,18 @@ interface smbXsrv
 		[in] smbXsrv_session_globalB blob
 		);
 
+	/*
+	 * The main server code should just work with
+	 * 'struct smbXsrv_session' and never use
+	 * smbXsrv_session0, smbXsrv_sessionU
+	 * and smbXsrv_sessionB directly.
+	 *
+	 * If we need to change the smbXsrv_session,
+	 * we can just rename smbXsrv_session
+	 * to smbXsrv_session0 and add a new
+	 * smbXsrv_session for version 1
+	 * and could implement transparent mapping.
+	 */
 	typedef struct {
 		[ignore] smbXsrv_session_table		*table;
 		[ignore] db_record 			*db_rec;
@@ -172,10 +148,10 @@ interface smbXsrv
 		[ignore] gensec_security		*gensec;
 		[ignore] user_struct			*compat;
 		[ignore] smbXsrv_tcon_table		*tcon_table;
-	} smbXsrv_session0;
+	} smbXsrv_session;
 
 	typedef union {
-		[case(0)] smbXsrv_session0		*info0;
+		[case(0)] smbXsrv_session		*info0;
 		[default] hyper				*dummy;
 	} smbXsrv_sessionU;
 
@@ -242,6 +218,18 @@ interface smbXsrv
 		[in] smbXsrv_tcon_globalB blob
 		);
 
+	/*
+	 * The main server code should just work with
+	 * 'struct smbXsrv_tcon' and never use
+	 * smbXsrv_tcon0, smbXsrv_tconU
+	 * and smbXsrv_tconB directly.
+	 *
+	 * If we need to change the smbXsrv_tcon,
+	 * we can just rename smbXsrv_tcon
+	 * to smbXsrv_tcon0 and add a new
+	 * smbXsrv_tcon for version 1
+	 * and could implement transparent mapping.
+	 */
 	typedef struct {
 		[ignore] smbXsrv_tcon_table		*table;
 		[ignore] db_record 			*db_rec;
@@ -250,10 +238,10 @@ interface smbXsrv
 		NTSTATUS				status;
 		NTTIME					idle_time;
 		[ignore] connection_struct		*compat;
-	} smbXsrv_tcon0;
+	} smbXsrv_tcon;
 
 	typedef union {
-		[case(0)] smbXsrv_tcon0			*info0;
+		[case(0)] smbXsrv_tcon			*info0;
 		[default] hyper				*dummy;
 	} smbXsrv_tconU;
 
@@ -301,6 +289,18 @@ interface smbXsrv
 		[in] smbXsrv_open_globalB blob
 		);
 
+	/*
+	 * The main server code should just work with
+	 * 'struct smbXsrv_open' and never use
+	 * smbXsrv_open0, smbXsrv_openU
+	 * and smbXsrv_openB directly.
+	 *
+	 * If we need to change the smbXsrv_open,
+	 * we can just rename smbXsrv_open
+	 * to smbXsrv_open0 and add a new
+	 * smbXsrv_open for version 1
+	 * and could implement transparent mapping.
+	 */
 	typedef struct {
 		[ignore] smbXsrv_open_table		*table;
 		[ignore] db_record 			*db_rec;
@@ -309,10 +309,10 @@ interface smbXsrv
 		NTSTATUS				status;
 		NTTIME					idle_time;
 		[ignore] files_struct			*compat;
-	} smbXsrv_open0;
+	} smbXsrv_open;
 
 	typedef union {
-		[case(0)] smbXsrv_open0			*info0;
+		[case(0)] smbXsrv_open			*info0;
 		[default] hyper				*dummy;
 	} smbXsrv_openU;
 
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index 5dcb926..d7141ae 100644
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -1101,7 +1101,7 @@ static char *lp_string(TALLOC_CTX *ctx, const char *s)
 #define FN_GLOBAL_STRING(fn_name,ptr) \
 char *lp_ ## fn_name(TALLOC_CTX *ctx) {return(lp_string((ctx), *(char **)(&Globals.ptr) ? *(char **)(&Globals.ptr) : ""));}
 #define FN_GLOBAL_CONST_STRING(fn_name,ptr) \
- const char *lp_ ## fn_name(void) {return(*(const char **)(&Globals.ptr) ? *(const char **)(&Globals.ptr) : "");}
+ const char *lp_ ## fn_name(void) {return(*(const char * const *)(&Globals.ptr) ? *(const char * const *)(&Globals.ptr) : "");}
 #define FN_GLOBAL_LIST(fn_name,ptr) \
  const char **lp_ ## fn_name(void) {return(*(const char ***)(&Globals.ptr));}
 #define FN_GLOBAL_BOOL(fn_name,ptr) \
diff --git a/source3/smbd/smbd.h b/source3/smbd/smbd.h
index 009b183..a5b211a 100644
--- a/source3/smbd/smbd.h
+++ b/source3/smbd/smbd.h
@@ -22,7 +22,6 @@
 struct dptr_struct;
 
 #include "smb_acls.h"
-#include "librpc/gen_ndr/smbXsrv.h"
 #include "vfs.h"
 #include "smbd/proto.h"
 #include "locking/proto.h"


-- 
Samba Shared Repository


More information about the samba-cvs mailing list