[systemd-commits] autogen.sh src/core src/shared src/tmpfiles

[Kay Sievers]

 autogen.sh                    |    2 +-
 src/core/machine-id-setup.c   |    2 +-
 src/core/manager.c            |    2 +-
 src/shared/ask-password-api.c |    3 ++-
 src/tmpfiles/tmpfiles.c       |    7 +++++--
 5 files changed, 10 insertions(+), 6 deletions(-)

New commits:
commit df28bc0836ad6a79cb537ad0b463819738088f3e
Author: Kay Sievers <kay at vrfy.org>
Date:   Mon Oct 21 15:24:18 2013 +0200

    build-sys: use -Og instead of -O0 to catch warnings
    
      $ touch src/core/dbus.c; make CFLAGS=-O0
      make --no-print-directory all-recursive
      Making all in .
        CC       src/core/libsystemd_core_la-dbus.lo
        CCLD     libsystemd-core.la
    
      $ touch src/core/dbus.c; make CFLAGS=-Og
      make --no-print-directory all-recursive
      Making all in .
        CC       src/core/libsystemd_core_la-dbus.lo
      src/core/dbus.c: In function 'init_registered_system_bus':
      src/core/dbus.c:798:18: warning: 'id' may be used uninitialized in this function [-Wmaybe-uninitialized]
               dbus_free(id);
                        ^
        CCLD     libsystemd-core.la
    
    -Og Optimize debugging experience. -Og enables optimizations that do
    not interfere with debugging. It should be the optimization level of
    choice for the standard edit-compile-debug cycle, offering a
    reasonable level of optimization while maintaining fast compilation
    and a good debugging experience.

diff --git a/autogen.sh b/autogen.sh
index eeb0c1b..d087342 100755
--- a/autogen.sh
+++ b/autogen.sh
@@ -54,7 +54,7 @@ args="$args \
 fi
 
 if [ "x$1" = "xc" ]; then
-        ./configure CFLAGS='-g -O0' $args
+        ./configure CFLAGS='-g -Og' $args
         make clean
 else
         echo
diff --git a/src/core/machine-id-setup.c b/src/core/machine-id-setup.c
index 18e015f..1b55da7 100644
--- a/src/core/machine-id-setup.c
+++ b/src/core/machine-id-setup.c
@@ -153,7 +153,7 @@ static int generate(char id[34]) {
 int machine_id_setup(void) {
         _cleanup_close_ int fd = -1;
         int r;
-        bool writable;
+        bool writable = false;
         struct stat st;
         char id[34]; /* 32 + \n + \0 */
 
diff --git a/src/core/manager.c b/src/core/manager.c
index 01db2b0..60c22e3 100644
--- a/src/core/manager.c
+++ b/src/core/manager.c
@@ -2062,7 +2062,7 @@ void manager_dispatch_bus_query_pid_done(
 
 int manager_open_serialization(Manager *m, FILE **_f) {
         char *path = NULL;
-        int fd;
+        int fd = -1;
         FILE *f;
 
         assert(_f);
diff --git a/src/shared/ask-password-api.c b/src/shared/ask-password-api.c
index 4557155..755abf0 100644
--- a/src/shared/ask-password-api.c
+++ b/src/shared/ask-password-api.c
@@ -250,7 +250,8 @@ static int create_socket(char **name) {
         } sa = {
                 .un.sun_family = AF_UNIX,
         };
-        int one = 1, r;
+        int one = 1;
+        int r = 0;
         char *c;
 
         assert(name);
diff --git a/src/tmpfiles/tmpfiles.c b/src/tmpfiles/tmpfiles.c
index e23847b..8051cb3 100644
--- a/src/tmpfiles/tmpfiles.c
+++ b/src/tmpfiles/tmpfiles.c
@@ -465,8 +465,10 @@ static int item_set_perms(Item *i, const char *path) {
 }
 
 static int write_one_file(Item *i, const char *path) {
-        int r, e, fd, flags;
+        int e, flags;
+        int fd = -1;
         struct stat st;
+        int r = 0;
 
         flags = i->type == CREATE_FILE ? O_CREAT|O_APPEND :
                 i->type == TRUNCATE_FILE ? O_CREAT|O_TRUNC : 0;
@@ -638,8 +640,9 @@ static int glob_item(Item *i, int (*action)(Item *, const char *)) {
 }
 
 static int create_item(Item *i) {
-        int r, e;
+        int e;
         struct stat st;
+        int r = 0;
 
         assert(i);