[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Mon Sep 20 14:18:57 MDT 2010


The branch, master has been updated
       via  dc19278 Really enable core dumps in Linux
      from  2c9cb11 s3:registry: move prototypes of unused hilvl functions to ret_api_util.h and comment them out

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


- Log -----------------------------------------------------------------
commit dc192787955542510724541cf23b4d8ed4cc99d4
Author: Rusty Russell <rusty at rustcorp.com.au>
Date:   Mon Sep 20 13:49:19 2010 +0930

    Really enable core dumps in Linux
    
    commit e1f1ce68e1f685400a8c68bcec14018e3d1fc29d
    Author: Rusty Russell <rusty at rustcorp.com.au>
    Date:   Mon Sep 20 13:33:30 2010 +0930
    
        source3: dump core on Linux, even after seteuid/etc.
    
        The "dumpable" flag is reset on every call to set*uid, so we need to
        reset it to 1 in the signal handler itself.
    
        This code dates back to commit ac01fda2b97b 6 years ago :(
    
        Signed-off-by: Rusty Russell <rusty at rustcorp.com.au>
    
    Signed-off-by: Jeremy Allison <jra at samba.org>

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

Summary of changes:
 source3/lib/fault.c |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/lib/fault.c b/source3/lib/fault.c
index 02e054b..dd87ae1 100644
--- a/source3/lib/fault.c
+++ b/source3/lib/fault.c
@@ -319,14 +319,6 @@ void dump_core_setup(const char *progname)
 #endif
 #endif
 
-#if defined(HAVE_PRCTL) && defined(PR_SET_DUMPABLE)
-	/* On Linux we lose the ability to dump core when we change our user
-	 * ID. We know how to dump core safely, so let's make sure we have our
-	 * dumpable flag set.
-	 */
-	prctl(PR_SET_DUMPABLE, 1);
-#endif
-
 	/* FIXME: if we have a core-plus-pid facility, configurably set
 	 * this up here.
 	 */
@@ -382,6 +374,14 @@ void dump_core_setup(const char *progname)
 	umask(~(0700));
 	dbgflush();
 
+#if defined(HAVE_PRCTL) && defined(PR_SET_DUMPABLE)
+	/* On Linux we lose the ability to dump core when we change our user
+	 * ID. We know how to dump core safely, so let's make sure we have our
+	 * dumpable flag set.
+	 */
+	prctl(PR_SET_DUMPABLE, 1);
+#endif
+
 	/* Ensure we don't have a signal handler for abort. */
 #ifdef SIGABRT
 	CatchSignal(SIGABRT, SIG_DFL);


-- 
Samba Shared Repository


More information about the samba-cvs mailing list