[dictd: 2/3] hardened build enables -Wunused-var, add patch to ignore some return values

Karsten Hopp karsten at fedoraproject.org
Tue Feb 25 15:35:17 UTC 2014


commit 4be30f587060cfc35f85b5cfa9f46d65c8cede8d
Author: Karsten Hopp <karsten at redhat.com>
Date:   Tue Feb 25 16:34:23 2014 +0100

    hardened build enables -Wunused-var, add patch to ignore some return values

 dictd-1.12.1-unused-return.patch |   86 ++++++++++++++++++++++++++++++++++++++
 1 files changed, 86 insertions(+), 0 deletions(-)
---
diff --git a/dictd-1.12.1-unused-return.patch b/dictd-1.12.1-unused-return.patch
new file mode 100644
index 0000000..3edcb6a
--- /dev/null
+++ b/dictd-1.12.1-unused-return.patch
@@ -0,0 +1,86 @@
+diff -purN dictd-1.12.1/dictd.c dictd-1.12.1_new/dictd.c
+--- dictd-1.12.1/dictd.c	2011-01-09 17:53:27.000000000 +0100
++++ dictd-1.12.1_new/dictd.c	2014-02-25 16:12:49.752301674 +0100
+@@ -329,6 +329,7 @@ static void xsigprocmask (int how, const
+    }
+ }
+ 
++/*
+ static void block_signals (void)
+ {
+    sigset_t set;
+@@ -350,6 +351,7 @@ static void unblock_signals (void)
+ 
+    xsigprocmask (SIG_UNBLOCK, &set, NULL);
+ }
++*/
+ 
+ static void handler( int sig )
+ {
+@@ -1264,21 +1266,22 @@ static void release_root_privileges( voi
+  * -- Bob Hilliard
+  */
+ {
++   int                unused __attribute__((unused));
+    if (geteuid() == 0) {
+       struct passwd *pwd;
+ 
+       if ((pwd = getpwnam("dictd"))) {
+-         setgid(pwd->pw_gid);
++         unused = setgid(pwd->pw_gid);
+          initgroups("dictd",pwd->pw_gid);
+-         setuid(pwd->pw_uid);
++         unused = setuid(pwd->pw_uid);
+       } else if ((pwd = getpwnam("nobody"))) {
+-         setgid(pwd->pw_gid);
++         unused = setgid(pwd->pw_gid);
+          initgroups("nobody",pwd->pw_gid);
+-         setuid(pwd->pw_uid);
++         unused = setuid(pwd->pw_uid);
+       } else {
+-         setgid(GID_NOGROUP);
++         unused = setgid(GID_NOGROUP);
+          initgroups("nobody", GID_NOGROUP);
+-         setuid(UID_NOBODY);
++         unused = setuid(UID_NOBODY);
+       }
+    }
+ }
+@@ -1464,6 +1467,7 @@ static void pid_file_write ()
+ static void reopen_012 (void)
+ {
+    int fd = open ("/dev/null", O_RDWR);
++   int unused __attribute__((unused));
+    if (fd == -1)
+       err_fatal_errno (__func__, ":E: can't open /dev/null");
+ 
+@@ -1471,9 +1475,9 @@ static void reopen_012 (void)
+    close (1);
+    close (2);
+ 
+-   dup (fd);
+-   dup (fd);
+-   dup (fd);
++   unused = dup (fd);
++   unused = dup (fd);
++   unused = dup (fd);
+ }
+ 
+ int main (int argc, char **argv, char **envp)
+@@ -1489,6 +1493,7 @@ int main (int argc, char **argv, char **
+    int                i;
+ 
+    int                errno_accept = 0;
++   int                unused __attribute__((unused));
+ 
+    const char *       default_strategy_arg = "???";
+ 
+@@ -1696,7 +1701,7 @@ int main (int argc, char **argv, char **
+ 
+    if (detach){
+       /* become a daemon */
+-      daemon (0, 1);
++      unused = daemon (0, 1);
+       reopen_012 ();
+ 
+       /* after fork from daemon(3) */


More information about the scm-commits mailing list