From: INSUN PYO <insun.pyo@samsung.com>
Date: Thu, 24 Nov 2016 07:03:42 +0000 (+0900)
Subject: kdbus: Remove the unnecessary code that unlink the reply twice.
X-Git-Tag: submit/tizen_3.0/20161205.010713~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=673c6533e3364856b77ece7002e791d02d083c61;p=platform%2Fkernel%2Flinux-exynos.git

kdbus: Remove the unnecessary code that unlink the reply twice.

In sync call situation, kdbus_conn_reply() calls kdbus_reply_unlink(reply).
Then, kdbus_conn_entry_sync_attach() also calls kdbus_reply_unlink(reply_wake),
so it is called twice. Even it has no problems because kdbus_reply_unlink()
does't free twice, but it is unnecessary.

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: Ia2b3552ee2e6a49ff97bb9d8f6e62964fe6d2cbf
(cherry picked from commit 81c82865a0f660cf3d552ca5879f674222e7060a)
---

diff --git a/ipc/kdbus/connection.c b/ipc/kdbus/connection.c
index 2dd907e4288b..02deba366839 100644
--- a/ipc/kdbus/connection.c
+++ b/ipc/kdbus/connection.c
@@ -871,7 +871,6 @@ static int kdbus_conn_entry_sync_attach(struct kdbus_conn *conn_dst,
 		remote_ret = -EREMOTEIO;
 
 	kdbus_sync_reply_wakeup(reply_wake, remote_ret);
-	kdbus_reply_unlink(reply_wake);
 	mutex_unlock(&reply_wake->reply_dst->lock);
 
 	return ret;