diff options
author | Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de> | 2010-01-30 14:25:51 +0100 |
---|---|---|
committer | Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de> | 2010-02-05 19:47:26 +0100 |
commit | 16aa0744c6820c5400b73fd4889608c38fc55b39 (patch) | |
tree | b4d8617279cf73be2d71b9fe765c6e5009ff2ac4 | |
parent | a51d750652671a41a2828b80feaa9e4d81219002 (diff) | |
download | samba-16aa0744c6820c5400b73fd4889608c38fc55b39.tar.gz samba-16aa0744c6820c5400b73fd4889608c38fc55b39.tar.bz2 samba-16aa0744c6820c5400b73fd4889608c38fc55b39.zip |
s4:UID wrapper - Fix includes
The includes of the UID wrapper headers werent't really efficient according
to metze's post on the technical mailing list (http://lists.samba.org/archive/samba-technical/2010-February/069165.html).
To achieve this move the "uid_wrapper.h" includes into "lib/util/unix_privs.c",
"lib/util/util.c", "ntvfs/posix/pvfs_acl.c" and "ntvfs/unixuid/vfs_unixuid.c".
-rw-r--r-- | lib/util/unix_privs.c | 11 | ||||
-rw-r--r-- | lib/util/util.c | 9 | ||||
-rw-r--r-- | source4/heimdal_build/config.h | 9 | ||||
-rw-r--r-- | source4/include/includes.h | 9 | ||||
-rw-r--r-- | source4/ntvfs/posix/pvfs_acl.c | 8 | ||||
-rw-r--r-- | source4/ntvfs/unixuid/vfs_unixuid.c | 10 |
6 files changed, 37 insertions, 19 deletions
diff --git a/lib/util/unix_privs.c b/lib/util/unix_privs.c index f55e739a9b..b30b2f5163 100644 --- a/lib/util/unix_privs.c +++ b/lib/util/unix_privs.c @@ -20,9 +20,18 @@ */ #include "includes.h" -#include "system/filesys.h" +#include "system/passwd.h" #include "../lib/util/unix_privs.h" +#if defined(UID_WRAPPER) +#if !defined(UID_WRAPPER_REPLACE) && !defined(UID_WRAPPER_NOT_REPLACE) +#define UID_WRAPPER_REPLACE +#include "../uid_wrapper/uid_wrapper.h" +#endif +#else +#define uwrap_enabled() 0 +#endif + /** * @file * @brief Gaining/losing root privileges diff --git a/lib/util/util.c b/lib/util/util.c index 67b166b421..25e010830e 100644 --- a/lib/util/util.c +++ b/lib/util/util.c @@ -31,6 +31,15 @@ #undef strdup #undef realloc +#if defined(UID_WRAPPER) +#if !defined(UID_WRAPPER_REPLACE) && !defined(UID_WRAPPER_NOT_REPLACE) +#define UID_WRAPPER_REPLACE +#include "../uid_wrapper/uid_wrapper.h" +#endif +#else +#define uwrap_enabled() 0 +#endif + /** * @file * @brief Misc utility functions diff --git a/source4/heimdal_build/config.h b/source4/heimdal_build/config.h index 8e9501b23b..be544cb83b 100644 --- a/source4/heimdal_build/config.h +++ b/source4/heimdal_build/config.h @@ -27,15 +27,6 @@ #undef HAVE_KRB5_ENCRYPT_BLOCK -#if defined(UID_WRAPPER) -#if !defined(UID_WRAPPER_REPLACE) && !defined(UID_WRAPPER_NOT_REPLACE) -#define UID_WRAPPER_REPLACE -#include "../uid_wrapper/uid_wrapper.h" -#endif -#else -#define uwrap_enabled() 0 -#endif - /*Workaround for heimdal define vs samba define*/ #ifdef HAVE_LIBINTL_H #define LIBINTL diff --git a/source4/include/includes.h b/source4/include/includes.h index 8d28808194..796e313572 100644 --- a/source4/include/includes.h +++ b/source4/include/includes.h @@ -69,13 +69,4 @@ #include "../lib/util/smb_threads.h" #include "../lib/util/smb_threads_internal.h" -#if defined(UID_WRAPPER) -#if !defined(UID_WRAPPER_REPLACE) && !defined(UID_WRAPPER_NOT_REPLACE) -#define UID_WRAPPER_REPLACE -#include "../uid_wrapper/uid_wrapper.h" -#endif -#else -#define uwrap_enabled() 0 -#endif - #endif /* _INCLUDES_H */ diff --git a/source4/ntvfs/posix/pvfs_acl.c b/source4/ntvfs/posix/pvfs_acl.c index c20358be2a..c100abe5e7 100644 --- a/source4/ntvfs/posix/pvfs_acl.c +++ b/source4/ntvfs/posix/pvfs_acl.c @@ -26,6 +26,14 @@ #include "libcli/security/security.h" #include "param/param.h" +#if defined(UID_WRAPPER) +#if !defined(UID_WRAPPER_REPLACE) && !defined(UID_WRAPPER_NOT_REPLACE) +#define UID_WRAPPER_REPLACE +#include "../uid_wrapper/uid_wrapper.h" +#endif +#else +#define uwrap_enabled() 0 +#endif /* the list of currently registered ACL backends */ static struct pvfs_acl_backend { diff --git a/source4/ntvfs/unixuid/vfs_unixuid.c b/source4/ntvfs/unixuid/vfs_unixuid.c index 8102c99536..85c70d7dd1 100644 --- a/source4/ntvfs/unixuid/vfs_unixuid.c +++ b/source4/ntvfs/unixuid/vfs_unixuid.c @@ -29,6 +29,16 @@ #define TEVENT_DEPRECATED #include <tevent.h> +#if defined(UID_WRAPPER) +#if !defined(UID_WRAPPER_REPLACE) && !defined(UID_WRAPPER_NOT_REPLACE) +#define UID_WRAPPER_REPLACE +#include "../uid_wrapper/uid_wrapper.h" +#endif +#else +#define uwrap_enabled() 0 +#endif + + struct unixuid_private { struct wbc_context *wbc_ctx; struct unix_sec_ctx *last_sec_ctx; |