diff options
| author | INSUN PYO <insun.pyo@samsung.com> | 2016-11-24 16:03:42 +0900 |
|---|---|---|
| committer | Seung-Woo Kim <sw0312.kim@samsung.com> | 2016-12-14 13:53:41 +0900 |
| commit | 323cc4064a8ef9ce86a97b27fc184813b4d351ec (patch) | |
| tree | 52161d877a56e6ada7a656df421888c96c43f4dc /ipc | |
| parent | cb56c8470863ed34b1f9303f0e00159701a445b6 (diff) | |
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
Diffstat (limited to 'ipc')
| -rw-r--r-- | ipc/kdbus/connection.c | 1 |
1 files changed, 0 insertions, 1 deletions
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; |