Fix: Possible passing of NULL pointer to memcpy()
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Sat, 5 Sep 2015 16:02:10 +0000 (12:02 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Sat, 5 Sep 2015 16:02:10 +0000 (12:02 -0400)
_cmd_enable_event() will not jump to the error label in case of
memory allocation of the filter bytecode copy. This causes the NULL
return of zmalloc to be used by memcpy() directly.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
src/bin/lttng-sessiond/cmd.c

index 860622d0065b9c3c80f22c32ee0b2d44b11f511d..6237d673efca6b1abd5c778949c8863be6fc72ef 100644 (file)
@@ -1826,6 +1826,7 @@ static int _cmd_enable_event(struct ltt_session *session,
                                filter_copy = zmalloc(filter_size);
                                if (!filter_copy) {
                                        ret = LTTNG_ERR_NOMEM;
+                                       goto error;
                                }
                                memcpy(filter_copy, filter, filter_size);
 
This page took 0.037211 seconds and 4 git commands to generate.