[systemd-commits] 2 commits - src/libsystemd-bus src/modules-load

Lennart Poettering lennart at kemper.freedesktop.org
Mon Mar 25 20:12:11 PDT 2013


 src/libsystemd-bus/sd-bus.c     |    4 +++-
 src/modules-load/modules-load.c |    2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

New commits:
commit 5a0f6033bed4c499f0b720e85f87ef9e46c5017e
Author: Lennart Poettering <lennart at poettering.net>
Date:   Tue Mar 26 03:42:03 2013 +0100

    bus: fix missing variable initialization

diff --git a/src/libsystemd-bus/sd-bus.c b/src/libsystemd-bus/sd-bus.c
index c82b738..5f2ede0 100644
--- a/src/libsystemd-bus/sd-bus.c
+++ b/src/libsystemd-bus/sd-bus.c
@@ -539,8 +539,10 @@ static int parse_exec_address(sd_bus *b, const char **p, char **guid) {
                 skip_address_key(p);
         }
 
-        if (!path)
+        if (!path) {
+                r = -EINVAL;
                 goto fail;
+        }
 
         /* Make sure there are no holes in the array, with the
          * exception of argv[0] */

commit b56c267ffbd3a5598dfa1dba5e6a51b38fcaed57
Author: Lennart Poettering <lennart at poettering.net>
Date:   Tue Mar 26 03:13:41 2013 +0100

    modules-load: there's really no point in mentioning that a certain modules is already loaded
    
    After all, this runs in parallel to udev, so there's quite a chance it
    already is....

diff --git a/src/modules-load/modules-load.c b/src/modules-load/modules-load.c
index 28b53ec..9ee2603 100644
--- a/src/modules-load/modules-load.c
+++ b/src/modules-load/modules-load.c
@@ -148,7 +148,7 @@ static int load_module(struct kmod_ctx *ctx, const char *m) {
                         break;
 
                 case KMOD_MODULE_LIVE:
-                        log_info("Module '%s' is already loaded", kmod_module_get_name(mod));
+                        log_debug("Module '%s' is already loaded", kmod_module_get_name(mod));
                         break;
 
                 default:



More information about the systemd-commits mailing list