From: Alex J Lennon <ajlennon@dynamicdevices.co.uk>
Date: Wed, 13 May 2015 16:08:47 +0000 (+0100)
Subject: cinterion: Correct use of freed structure
X-Git-Tag: upstream/1.17~35
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f8d9485dc28f976ed5665cbe75ef4a55d5f84ee1;p=platform%2Fupstream%2Fofono.git

cinterion: Correct use of freed structure

On error struct cb_data *cbd was used after in cinterion_set_online
after already being freed.
---

diff --git a/plugins/cinterion.c b/plugins/cinterion.c
index 6d851892..a18b9fe8 100644
--- a/plugins/cinterion.c
+++ b/plugins/cinterion.c
@@ -166,9 +166,9 @@ static void cinterion_set_online(struct ofono_modem *modem, ofono_bool_t online,
 	if (g_at_chat_send(chat, command, NULL, set_online_cb, cbd, g_free))
 		return;
 
-	g_free(cbd);
-
 	CALLBACK_WITH_FAILURE(cb, cbd->data);
+
+	g_free(cbd);
 }
 
 static void cinterion_pre_sim(struct ofono_modem *modem)