From b5fd5969f4593917ccd07b34c06f0968931b695f Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Fri, 15 May 2015 11:02:33 +0100 Subject: [PATCH] Unix Pageant: fix further double-frees. No need to sfree(err) before going to the cleanup code, because the whole point of shared cleanup code is that that will do it for us. --- unix/uxpgnt.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/unix/uxpgnt.c b/unix/uxpgnt.c index 27de94d4..bf737772 100644 --- a/unix/uxpgnt.c +++ b/unix/uxpgnt.c @@ -355,7 +355,6 @@ static int unix_add_keyfile(const char *filename_str) goto cleanup; } else if (status == PAGEANT_ACTION_FAILURE) { fprintf(stderr, "pageant: %s: %s\n", filename_str, err); - sfree(err); ret = FALSE; goto cleanup; } @@ -380,7 +379,6 @@ static int unix_add_keyfile(const char *filename_str) goto cleanup; } else if (status == PAGEANT_ACTION_FAILURE) { fprintf(stderr, "pageant: %s: %s\n", filename_str, err); - sfree(err); ret = FALSE; goto cleanup; }