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 <me...@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 <me...@samba.org> Autobuild-Date(master): Fri Aug 10 17:35:38 CEST 2012 on sn-devel-104 commit 0e76bbc520b0052f1fed6bbd17fe8737249e8e68 Author: Stefan Metzmacher <me...@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 <me...@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 <me...@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 <me...@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