From: Travis Cross <tc@traviscross.com>
Date: Tue, 28 May 2013 05:28:19 +0000 (+0000)
Subject: Avoid using memory after we free it
X-Git-Tag: v1.4.1~141
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3c8db7c715b320087494aa9c9ca54fde737a5c8d;p=thirdparty%2Ffreeswitch.git

Avoid using memory after we free it

Without setting errmsg to NULL here, we'll try to print this freed
value again below if another error doesn't intercede.
---

diff --git a/src/switch_core_sqldb.c b/src/switch_core_sqldb.c
index b02d4e047e..4a9f1af957 100644
--- a/src/switch_core_sqldb.c
+++ b/src/switch_core_sqldb.c
@@ -1676,7 +1676,7 @@ static uint32_t do_trans(switch_sql_queue_manager_t *qm)
 		switch_cache_db_execute_sql_real(qm->event_db, qm->pre_trans_execute, &errmsg);
 		if (errmsg) {
 			switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_CRIT, "SQL PRE TRANS EXEC %s [%s]\n", qm->pre_trans_execute, errmsg);
-			free(errmsg);
+			free(errmsg); errmsg = NULL;
 		}
 	}