diff options
author | David Teigland <teigland@redhat.com> | 2015-05-29 09:54:47 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2015-06-15 16:50:54 -0500 |
commit | f615218f798cb6e8933172dc10967f917ade3f8f (patch) | |
tree | 0989ee773a3c1061c2adf4a511f80793e69fe956 | |
parent | e29c4811025dbcf89534f97f28d00001c74d8f08 (diff) | |
download | lvm2-f615218f798cb6e8933172dc10967f917ade3f8f.tar.gz |
lib lvmlockd: fix segfault in logging result flags
-rw-r--r-- | lib/locking/lvmlockd.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/lib/locking/lvmlockd.c b/lib/locking/lvmlockd.c index 600280d3c..0865aa230 100644 --- a/lib/locking/lvmlockd.c +++ b/lib/locking/lvmlockd.c @@ -138,8 +138,8 @@ static void _flags_str_to_lockd_flags(const char *flags_str, uint32_t *lockd_fla static int _lockd_result(daemon_reply reply, int *result, uint32_t *lockd_flags) { int reply_result; - const char *flags_str; - const char *lock_type; + const char *flags_str = NULL; + const char *lock_type = NULL; if (reply.error) { log_error("lockd_result reply error %d", reply.error); @@ -162,15 +162,13 @@ static int _lockd_result(daemon_reply reply, int *result, uint32_t *lockd_flags) *result = reply_result; - if (!lockd_flags) - goto out; - - flags_str = daemon_reply_str(reply, "result_flags", NULL); - if (flags_str) - _flags_str_to_lockd_flags(flags_str, lockd_flags); + if (lockd_flags) { + if ((flags_str = daemon_reply_str(reply, "result_flags", NULL))) + _flags_str_to_lockd_flags(flags_str, lockd_flags); + } - out: - log_debug("lockd_result %d %s lm %s", reply_result, flags_str, lock_type); + log_debug("lockd_result %d flags %s lm %s", reply_result, + flags_str ? flags_str : "none", lock_type); return 1; } |