summaryrefslogtreecommitdiffstats
path: root/build-aux
diff options
context:
space:
mode:
authorAndre Heinecke <aheinecke@intevation.de>2017-06-07 15:30:14 +0200
committerAndre Heinecke <aheinecke@intevation.de>2017-06-07 15:34:30 +0200
commit13dc75a4e7cc2959003c08940fc53c6ece7b77e4 (patch)
tree6545d0a0086d71b268e227ce7931271a4d8f4d18 /build-aux
parentspeedo: Fix source tar call ambiguity (diff)
downloadgnupg2-13dc75a4e7cc2959003c08940fc53c6ece7b77e4.tar.xz
gnupg2-13dc75a4e7cc2959003c08940fc53c6ece7b77e4.zip
speedo: Fix a minor memleak in the installer
* build-aux/speedo/w32/g4wihelp.c (path_remove): Free path_new on early return. -- It's a weird condition in a once run function in a throwaway process but -- yeah. It's a memleak and static analysis can see it. GnuPG-Bug-Id: T3197 Signed-off-by: Andre Heinecke <aheinecke@intevation.de>
Diffstat (limited to 'build-aux')
-rw-r--r--build-aux/speedo/w32/g4wihelp.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/build-aux/speedo/w32/g4wihelp.c b/build-aux/speedo/w32/g4wihelp.c
index d62d03647..626f3f12c 100644
--- a/build-aux/speedo/w32/g4wihelp.c
+++ b/build-aux/speedo/w32/g4wihelp.c
@@ -1159,7 +1159,10 @@ path_remove (HWND hwndParent, int string_size, char *variables,
free (path);
if (! changed)
- return;
+ {
+ free (path_new);
+ return;
+ }
/* Set a key for our CLSID. */
RegCreateKey (root_key, env_reg, &key_handle);