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.2.13~294
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=04b8de06193b7258a2b0c86bf7dc374e6dfef35c;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 9f4609f63c..30edc92e62 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;
 		}
 	}