From f8d9485dc28f976ed5665cbe75ef4a55d5f84ee1 Mon Sep 17 00:00:00 2001
From: Alex J Lennon <ajlennon@dynamicdevices.co.uk>
Date: Wed, 13 May 2015 17:08:47 +0100
Subject: cinterion: Correct use of freed structure

On error struct cb_data *cbd was used after in cinterion_set_online
after already being freed.
---
 plugins/cinterion.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'plugins')

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)
-- 
cgit v1.2.3