diff options
| author | Justus Winter <[email protected]> | 2016-11-29 15:48:35 +0000 |
|---|---|---|
| committer | Justus Winter <[email protected]> | 2016-11-29 15:48:35 +0000 |
| commit | 52385a2ba1bf7e53f500ffde5fd34f28e69cf76b (patch) | |
| tree | ec762bb4b56d3111168a0d4718135a90bcfdfc32 | |
| parent | tools,build: Build WKS tools against libintl. (diff) | |
| download | gnupg-52385a2ba1bf7e53f500ffde5fd34f28e69cf76b.tar.gz gnupg-52385a2ba1bf7e53f500ffde5fd34f28e69cf76b.zip | |
g10: Fix memory leak.
* g10/decrypt.c (decrypt_messages): Properly decrease the reference
count of the armor filters after pushing them.
Signed-off-by: Justus Winter <[email protected]>
| -rw-r--r-- | g10/decrypt.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/g10/decrypt.c b/g10/decrypt.c index 751b7be55..c99f0640e 100644 --- a/g10/decrypt.c +++ b/g10/decrypt.c @@ -180,7 +180,6 @@ void decrypt_messages (ctrl_t ctrl, int nfiles, char *files[]) { IOBUF fp; - armor_filter_context_t *afx = NULL; progress_filter_context_t *pfx; char *p, *output = NULL; int rc=0,use_stdin=0; @@ -254,8 +253,11 @@ decrypt_messages (ctrl_t ctrl, int nfiles, char *files[]) { if (use_armor_filter(fp)) { - afx = new_armor_context (); - push_armor_filter ( afx, fp ); + armor_filter_context_t *afx = new_armor_context (); + rc = push_armor_filter (afx, fp); + if (rc) + log_error("failed to push armor filter"); + release_armor_context (afx); } } rc = proc_packets (ctrl,NULL, fp); @@ -275,6 +277,5 @@ decrypt_messages (ctrl_t ctrl, int nfiles, char *files[]) } set_next_passphrase(NULL); - release_armor_context (afx); release_progress_context (pfx); } |