Merge pull request #7837 from yuwata/coverity-20170110

fix defects reported by Coverity Scan

src/basic/exec-util.c

@@ -67,7 +67,7 @@ static int do_spawn(const char *path, char *argv[], int stdout_fd, pid_t *pid) {
                             dup2(stdout_fd, STDOUT_FILENO) < 0)
                                 return -errno;
 
-                        fd_cloexec(STDOUT_FILENO, false);
+                        (void) fd_cloexec(STDOUT_FILENO, false);
                 }
 
                 if (!argv) {

src/basic/process-util.c

@@ -1162,7 +1162,7 @@ int safe_fork_full(
 
         pid_t original_pid, pid;
         sigset_t saved_ss, ss;
-        bool block_signals;
+        bool block_signals = false;
         int prio, r;
 
         /* A wrapper around fork(), that does a couple of important initializations in addition to mere forking. Always

src/test/test-socket-util.c

@@ -487,7 +487,7 @@ static void test_getpeercred_getpeergroups(void) {
         if (r == 0) {
                 static const gid_t gids[] = { 3, 4, 5, 6, 7 };
                 gid_t *test_gids;
-                gid_t *peer_groups = NULL;
+                _cleanup_free_ gid_t *peer_groups = NULL;
                 size_t n_test_gids;
                 uid_t test_uid;
                 gid_t test_gid;

src/timesync/timesyncd-gperf.gperf

@@ -10,7 +10,7 @@ struct ConfigPerfItem;
 %null_strings
 %language=ANSI-C
 %define slot-name section_and_lvalue
-%define hash-function-name timesyncdd_gperf_hash
+%define hash-function-name timesyncd_gperf_hash
 %define lookup-function-name timesyncd_gperf_lookup
 %readonly-tables
 %omit-struct-type