Fix false positives from valgrind: prctl(PR_GET_DUMPABLE)

Pass in zero to the unusued arguments of prctl(PR_GET_DUMPABLE) to
avoid false positives from valgrind.

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>

diff --git a/lib/blkid/ChangeLog b/lib/blkid/ChangeLog
index 767f11e1cdd42242cba7445c2be597de93e12374..1bb8cd31173bab85172159e415fdb8beda09fd55 100644 (file)
--- a/lib/blkid/ChangeLog
+++ b/lib/blkid/ChangeLog
@@ -1,3 +1,9 @@
+2005-07-25  Theodore Ts'o  <tytso@mit.edu>
+
+       * cache.c (safe_getenv): Pass in zero to the unusued arguments of
+               prctl(PR_GET_DUMPABLE) to avoid false positives from
+               valgrind.
+
 2005-07-09  Andreas Dilger <adilger@clusterfs.com>
 
        * getsize.c (blkid_get_dev_size): Use fstat/fstat64 to get size of

diff --git a/lib/blkid/cache.c b/lib/blkid/cache.c
index b43be606cf369af9c021b24e8e162edad21ad8a6..b47b4880bb0d672e6b2db32446f9c67a32e523d9 100644 (file)
--- a/lib/blkid/cache.c
+++ b/lib/blkid/cache.c
@@ -36,11 +36,11 @@ static char *safe_getenv(const char *arg)
        if ((getuid() != geteuid()) || (getgid() != getegid()))
                return NULL;
 #if HAVE_PRCTL
-       if (prctl(PR_GET_DUMPABLE) == 0)
+       if (prctl(PR_GET_DUMPABLE, 0, 0, 0, 0) == 0)
                return NULL;
 #else
 #if (defined(linux) && defined(SYS_prctl))
-       if (syscall(SYS_prctl, PR_GET_DUMPABLE) == 0)
+       if (syscall(SYS_prctl, PR_GET_DUMPABLE, 0, 0, 0, 0) == 0)
                return NULL;
 #endif
 #endif

diff --git a/lib/ext2fs/ChangeLog b/lib/ext2fs/ChangeLog
index b586a6b267873585ff2b142320b10a889cfa326e..8202608bf7198552a5a52a08c5cf4a85ca467b6c 100644 (file)
--- a/lib/ext2fs/ChangeLog
+++ b/lib/ext2fs/ChangeLog
@@ -1,11 +1,18 @@
+2005-07-25  Theodore Ts'o  <tytso@mit.edu>
+
+       * test_io.c (safe_getenv): Pass in zero to the unusued arguments
+               of prctl(PR_GET_DUMPABLE) to avoid false positives from
+               valgrind.
+
 2005-07-19  Theodore Ts'o  <tytso@mit.edu>
 
        * mkjournal.c (ext2fs_add_journal_inode): Check for the case where
                the filesystem is apparently not mounted, but the device
-               is still busy.  This can happy when the luser doesn't
-               bother to mount /proc and has a bogus /etc/mtab.  Add a
-               safety check to save him from his own stupidity, at least
-               on 2.6 kernels.  (Addresses Debian Bug #319002)
+               is still busy.  This can happen when the luser doesn't
+               bother to mount /proc and has a bogus /etc/mtab, but still
+               wants to mount the filesystem before using tune2fs(?!?).
+               Add a safety check to save him from his own stupidity, at
+               least on 2.6 kernels.  (Addresses Debian Bug #319002)
 
 2005-07-09  Andreas Dilger <adilger@clusterfs.com>
 

diff --git a/lib/ext2fs/test_io.c b/lib/ext2fs/test_io.c
index 2d426173c2224eab8b9872a2ea326504610be39b..03d63ce13a9f8f9a4d737b2835532c94cdc7dbef 100644 (file)
--- a/lib/ext2fs/test_io.c
+++ b/lib/ext2fs/test_io.c
@@ -145,11 +145,11 @@ static char *safe_getenv(const char *arg)
        if ((getuid() != geteuid()) || (getgid() != getegid()))
                return NULL;
 #if HAVE_PRCTL
-       if (prctl(PR_GET_DUMPABLE) == 0)
+       if (prctl(PR_GET_DUMPABLE, 0, 0, 0, 0) == 0)
                return NULL;
 #else
 #if (defined(linux) && defined(SYS_prctl))
-       if (syscall(SYS_prctl, PR_GET_DUMPABLE) == 0)
+       if (syscall(SYS_prctl, PR_GET_DUMPABLE, 0, 0, 0, 0) == 0)
                return NULL;
 #endif
 #endif

diff --git a/lib/ss/ChangeLog b/lib/ss/ChangeLog
index 389b35f89caa6825ce72c199072555663a7a4972..13447b05b5cd579ffa4c56bd490543cdf876f0f7 100644 (file)
--- a/lib/ss/ChangeLog
+++ b/lib/ss/ChangeLog
@@ -1,3 +1,9 @@
+2005-07-25  Theodore Ts'o  <tytso@mit.edu>
+
+       * pager.c (ss_safe_getenv): Pass in zero to the unusued arguments
+               of prctl(PR_GET_DUMPABLE) to avoid false positives from
+               valgrind.
+
 2006-06-30  Theodore Ts'o  <tytso@mit.edu>
 
        * Release of E2fsprogs 1.38

diff --git a/lib/ss/pager.c b/lib/ss/pager.c
index e5327782d896b07dbb724d55d653434eb6494f55..a7a2a09279dad197f962997bcf2882897aba65c5 100644 (file)
--- a/lib/ss/pager.c
+++ b/lib/ss/pager.c
@@ -46,11 +46,11 @@ char *ss_safe_getenv(const char *arg)
        if ((getuid() != geteuid()) || (getgid() != getegid()))
                return NULL;
 #if HAVE_PRCTL
-       if (prctl(PR_GET_DUMPABLE) == 0)
+       if (prctl(PR_GET_DUMPABLE, 0, 0, 0, 0) == 0)
                return NULL;
 #else
 #if (defined(linux) && defined(SYS_prctl))
-       if (syscall(SYS_prctl, PR_GET_DUMPABLE) == 0)
+       if (syscall(SYS_prctl, PR_GET_DUMPABLE, 0, 0, 0, 0) == 0)
                return NULL;
 #endif
 #endif