]> granicus.if.org Git - neomutt/commitdiff
pgpewrap: free opts on exit 809/head
authorJelle van der Waa <jelle@vdwaa.nl>
Thu, 21 Sep 2017 19:41:11 +0000 (21:41 +0200)
committerRichard Russon <rich@flatcap.org>
Mon, 2 Oct 2017 11:32:51 +0000 (12:32 +0100)
opts is allocated with malloc, but never free'd in both exit scenario's
leading to a memory leak of opts.

pgpewrap.c

index 2c158cb45b3d4df724ccf456e1645c593d38d52c..da7d30cce30b4bd5bc6c4ef2c3060c71b553a186 100644 (file)
@@ -47,6 +47,7 @@ int main(int argc, char **argv)
       i += 2;
       if (i > argc)
       {
+        free(opts);
         print_usage(argv[0]);
       }
       pfx = argv[i - 1];
@@ -59,5 +60,6 @@ int main(int argc, char **argv)
 
   execvp(opts[0], opts);
   perror(argv[0]);
+  free(opts);
   return 2;
 }