changeset 17818:734596c36b58

vasnprintf: fix potential use after free * lib/vasnprintf.c (VASNPRINTF): Fix free-memory read, flagged by clang-analyzer 3.4.2.
author Pádraig Brady <P@draigBrady.com>
date Sat, 06 Dec 2014 01:14:02 +0000
parents eb502f8dffb3
children 2c202161ad79
files ChangeLog lib/vasnprintf.c
diffstat 2 files changed, 7 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2014-12-06  Pádraig Brady  <P@draigBrady.com>
+
+	vasnprintf: fix potential use after free
+	* lib/vasnprintf.c (VASNPRINTF): Fix free-memory read,
+	flagged by clang-analyzer 3.4.2.
+
 2014-12-05  Pádraig Brady  <P@draigBrady.com>
 
 	filevercmp, posixtm: avoid compiler warnings with -O3
--- a/lib/vasnprintf.c
+++ b/lib/vasnprintf.c
@@ -5184,13 +5184,13 @@
                           free (result);
                         if (buf_malloced != NULL)
                           free (buf_malloced);
-                        CLEANUP ();
                         errno =
                           (saved_errno != 0
                            ? saved_errno
                            : (dp->conversion == 'c' || dp->conversion == 's'
                               ? EILSEQ
                               : EINVAL));
+                        CLEANUP ();
                         return NULL;
                       }