ttauth: Coverity (memory corruption)
authorJon Trulson <jon@radscan.com>
Fri, 26 Dec 2014 21:12:10 +0000 (14:12 -0700)
committerJon Trulson <jon@radscan.com>
Fri, 26 Dec 2014 21:12:10 +0000 (14:12 -0700)
cde/lib/tt/bin/ttauth/process.c

index bbaddd4f3c787300c62d55dc6f6e6859b85dc183..bafd259c39b18e54a7cb91efccb67857bf35607f 100644 (file)
@@ -1252,7 +1252,7 @@ static int do_add (inputfilename, lineno, argc, argv)
     char *netid;
     char *authname;
     char *authdata_hex;
-    char *authdata;
+    char *authdata = NULL;
     int protodata_len, authdata_len;
     _tt_AuthFileEntry *entry;
     _tt_AuthFileEntryList *list;
@@ -1345,6 +1345,9 @@ static int do_add (inputfilename, lineno, argc, argv)
     entry->auth_data_length = authdata_len;
     entry->auth_data = authdata;
 
+    /* Avoid a double free later on in the event of an error */
+    authdata = NULL;
+
     if (!entry->protocol_name ||
        (!entry->protocol_data && entry->protocol_data_length > 0) ||
         !entry->network_id || !entry->auth_name ||