diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2018-11-30 11:59:04 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2018-11-30 16:19:26 +0100 |
| commit | cd92e7a9b6b77accd6e3e288d0bdc246c250cf19 (patch) | |
| tree | fa32e63b9fb287b9eec9486325fed01f93eabd90 | |
| parent | 8e2549b86c4022cdad272c265a66c88f06fc3560 (diff) | |
| download | curl-bagder/writeout-memleak.tar.gz | |
curl: fix memory leaks when reading from filesbagder/writeout-memleak
The file2memory and file2string functions are used for several commanad
line options when reading from files using the "@file" syntax. If
another string had been set first, these functions would leak the
previously allocated memory.
Reported-by: Brian Carpenter
Fixes #3322
| -rw-r--r-- | src/tool_getparam.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/tool_getparam.c b/src/tool_getparam.c index cd21f774c..c0d3a84f2 100644 --- a/src/tool_getparam.c +++ b/src/tool_getparam.c @@ -2066,6 +2066,7 @@ ParameterError getparameter(const char *flag, /* f or -long-flag */ fname = nextarg; file = fopen(nextarg, FOPEN_READTEXT); } + Curl_safefree(config->writeout); err = file2string(&config->writeout, file); if(file && (file != stdin)) fclose(file); |