chiark / gitweb /
bus: zero cmd_free before passing to ioctl
authorDavid Herrmann <dh.herrmann@gmail.com>
Thu, 11 Dec 2014 09:45:58 +0000 (10:45 +0100)
committerDavid Herrmann <dh.herrmann@gmail.com>
Thu, 11 Dec 2014 09:45:58 +0000 (10:45 +0100)
Make sure the whole cmd_free object is zeroed before passing it into the
kernel. This makes valgrind happy and makes us future proof.

src/libsystemd/sd-bus/bus-message.c

index 0803999e070fc81e24dcd072b6ec53e7a674e4f9..6bf39cef6b3174812fbe921993ccde0bbc8b0a1d 100644 (file)
@@ -130,7 +130,7 @@ static void message_free(sd_bus_message *m) {
         message_reset_parts(m);
 
         if (m->release_kdbus) {
         message_reset_parts(m);
 
         if (m->release_kdbus) {
-                struct kdbus_cmd_free cmd_free;
+                struct kdbus_cmd_free cmd_free = { };
 
                 cmd_free.flags = 0;
                 cmd_free.offset = (uint8_t *)m->kdbus - (uint8_t *)m->bus->kdbus_buffer;
 
                 cmd_free.flags = 0;
                 cmd_free.offset = (uint8_t *)m->kdbus - (uint8_t *)m->bus->kdbus_buffer;