From 9a846daf441c3d82e4fc750382fd12fcb8fb9a6d Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Sat, 15 Sep 2001 02:10:22 +0000 Subject: Restore the profiling data shmem parinoia. This whole area needs to be fixed - an mmaped file or the like would be a good idea. (This used to be commit bc1385fc5e55eeed626615fad92877296064a27e) --- source3/lib/util_sec.c | 10 ++++++++++ source3/profile/profile.c | 6 ++---- 2 files changed, 12 insertions(+), 4 deletions(-) (limited to 'source3') diff --git a/source3/lib/util_sec.c b/source3/lib/util_sec.c index 985b07f421..231f0b178d 100644 --- a/source3/lib/util_sec.c +++ b/source3/lib/util_sec.c @@ -48,6 +48,7 @@ extern int DEBUGLEVEL; /* are we running as non-root? This is used by the regresison test code, and potentially also for sites that want non-root smbd */ static uid_t initial_uid; +static gid_t initial_gid; /**************************************************************************** remember what uid we got started as - this allows us to run correctly @@ -56,6 +57,7 @@ as non-root while catching trapdoor systems void sec_init(void) { initial_uid = geteuid(); + initial_gid = getegid(); } /**************************************************************************** @@ -66,6 +68,14 @@ uid_t sec_initial_uid(void) return initial_uid; } +/**************************************************************************** +some code (eg. winbindd, profiling shm) needs to know what gid we started as +****************************************************************************/ +gid_t sec_initial_gid(void) +{ + return initial_gid; +} + /**************************************************************************** are we running in non-root mode? ****************************************************************************/ diff --git a/source3/profile/profile.c b/source3/profile/profile.c index 430732c6f8..5ddc72c490 100644 --- a/source3/profile/profile.c +++ b/source3/profile/profile.c @@ -130,12 +130,10 @@ BOOL profile_setup(BOOL rdonly) return False; } -#if 0 - if (shm_ds.shm_perm.cuid != 0 || shm_ds.shm_perm.cgid != 0) { - DEBUG(0,("ERROR: root did not create the shmem\n")); + if (shm_ds.shm_perm.cuid != sec_initial_uid() || shm_ds.shm_perm.cgid != sec_initial_gid()) { + DEBUG(0,("ERROR: we did not create the shmem (owned by another user)\n")); return False; } -#endif if (shm_ds.shm_segsz != sizeof(*profile_h)) { DEBUG(0,("WARNING: profile size is %d (expected %d). Deleting\n", -- cgit