kdbus: Remove the unnecessary code that unlink the reply twice.
authorINSUN PYO <insun.pyo@samsung.com>
Thu, 24 Nov 2016 07:03:42 +0000 (16:03 +0900)
committerSeung-Woo Kim <sw0312.kim@samsung.com>
Wed, 14 Dec 2016 04:53:41 +0000 (13:53 +0900)
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

ipc/kdbus/connection.c

index 2dd907e..02deba3 100644 (file)
@@ -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;