Convert sizeof foo to sizeof(foo).

[tinc] / src / tincd.c

diff --git a/src/tincd.c b/src/tincd.c
index 84ce0d4..c731e1d 100644 (file)
--- a/src/tincd.c
+++ b/src/tincd.c
@@ -413,15 +413,18 @@ static bool keygen(int bits) {
 
        if(!result) {
                fprintf(stderr, "Error during key generation!\n");
+               RSA_free(rsa_key);
                return false;
        } else
                fprintf(stderr, "Done.\n");
 
-       snprintf(filename, sizeof filename, "%s/rsa_key.priv", confbase);
+       snprintf(filename, sizeof(filename), "%s/rsa_key.priv", confbase);
        f = ask_and_open(filename, "private RSA key");
 
-       if(!f)
+       if(!f) {
+               RSA_free(rsa_key);
                return false;
+       }
 
 #ifdef HAVE_FCHMOD
        /* Make it unreadable for others. */
@@ -435,21 +438,25 @@ static bool keygen(int bits) {
        char *name = get_name();
 
        if(name) {
-               snprintf(filename, sizeof filename, "%s/hosts/%s", confbase, name);
+               snprintf(filename, sizeof(filename), "%s/hosts/%s", confbase, name);
                free(name);
        } else {
-               snprintf(filename, sizeof filename, "%s/rsa_key.pub", confbase);
+               snprintf(filename, sizeof(filename), "%s/rsa_key.pub", confbase);
        }
 
        f = ask_and_open(filename, "public RSA key");
 
-       if(!f)
+       if(!f) {
+               RSA_free(rsa_key);
                return false;
+       }
 
        fputc('\n', f);
        PEM_write_RSAPublicKey(f, rsa_key);
        fclose(f);
 
+       RSA_free(rsa_key);
+
        return true;
 }
 
@@ -536,7 +543,7 @@ static bool drop_privs() {
                               "initgroups", strerror(errno));
                        return false;
                }
-#ifndef __ANDROID__
+#ifndef ANDROID
 // Not supported in android NDK
                endgrent();
                endpwent();
@@ -737,7 +744,7 @@ end:
        ERR_free_strings();
 
        exit_configuration(&config_tree);
-       list_free(cmdline_conf);
+       list_delete_list(cmdline_conf);
        free_names();
 
        return status;