Fix a potential memory leak.

Found by the Clang static analyzer.

diff --git a/src/meta.c b/src/meta.c
index ccc3b2e670d5ae9a0210929d804703323f97983f..60814c78d59fc2ab59611d63f7859ed58494f59a 100644 (file)
--- a/src/meta.c
+++ b/src/meta.c
@@ -38,11 +38,6 @@ bool send_meta(connection_t *c, const char *buffer, int length) {
        int outlen;
        int result;
 
-       if(!c) {
-               logger(LOG_ERR, "send_meta() called with NULL pointer!");
-               abort();
-       }
-
        ifdebug(META) logger(LOG_DEBUG, "Sending %d bytes of metadata to %s (%s)", length,
                           c->name, c->hostname);
 

diff --git a/src/multicast_device.c b/src/multicast_device.c
index ea43e2c04a6efbaf4abb555ea97b098ce5fd9fe6..ceaa3efc78d2881dc0bbd11d682571839b28fb14 100644 (file)
--- a/src/multicast_device.c
+++ b/src/multicast_device.c
@@ -70,8 +70,10 @@ static bool setup_device(void) {
        }
 
        ai = str2addrinfo(host, port, SOCK_DGRAM);
-       if(!ai)
+       if(!ai) {
+               free(host);
                return false;
+       }
 
        device_fd = socket(ai->ai_family, SOCK_DGRAM, IPPROTO_UDP);
        if(device_fd < 0) {