]> git.meshlink.io Git - meshlink/commitdiff
Fix use of unitialised values in hash tables.
authorGuus Sliepen <guus@tinc-vpn.org>
Thu, 29 Nov 2012 13:35:08 +0000 (14:35 +0100)
committerGuus Sliepen <guus@tinc-vpn.org>
Thu, 29 Nov 2012 13:35:08 +0000 (14:35 +0100)
Not only was Valgrind unhappy about it, it could also result in cache misses.

src/hash.c
src/netutl.c

index cf5ba90a08e13e5dec7ef4e6bf23d7e2347b749b..1d203c5cb31194982e6920cf5c063cee2ee43de0 100644 (file)
@@ -55,7 +55,7 @@ hash_t *hash_alloc(size_t n, size_t size) {
        hash_t *hash = xmalloc_and_zero(sizeof *hash);
        hash->n = n;
        hash->size = size;
-       hash->keys = xmalloc(hash->n * hash->size);
+       hash->keys = xmalloc_and_zero(hash->n * hash->size);
        hash->values = xmalloc_and_zero(hash->n * sizeof *hash->values);
        return hash;
 }
@@ -100,6 +100,8 @@ void hash_clear(hash_t *hash) {
 void hash_resize(hash_t *hash, size_t n) {
        hash->keys = xrealloc(hash->keys, n * hash->size);
        hash->values = xrealloc(hash->values, n * sizeof *hash->values);
-       if(n > hash->n)
+       if(n > hash->n) {
+               memset(hash->keys + hash->n * hash->size, 0, (n - hash->n) * hash->size);
                memset(hash->values + hash->n, 0, (n - hash->n) * sizeof *hash->values);
+       }
 }
index a71b370f9f9306c0bcbe3b07a73fbbcfb10fbb35..a55eaea21255b71a647bd61151194133ceb7eacb 100644 (file)
@@ -52,7 +52,7 @@ struct addrinfo *str2addrinfo(const char *address, const char *service, int sock
 
 sockaddr_t str2sockaddr(const char *address, const char *port) {
        struct addrinfo *ai, hint = {0};
-       sockaddr_t result;
+       sockaddr_t result = {{0}};
        int err;
 
        hint.ai_family = AF_UNSPEC;