diff options
author | wangweidong <wangweidong1@huawei.com> | 2013-12-11 20:36:39 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-12-16 12:48:35 -0500 |
commit | 0cee6bbe06f67ff7dd83a4bc794f23c5cd5e7929 (patch) | |
tree | 59864290d3b49dbf64e42c53030e883461125998 /net/tipc/port.c | |
parent | 810c23a3553cbb4602edf9534b548d2616ba5520 (diff) |
tipc: remove unnecessary variables and conditions
We remove a number of unnecessary variables and branches
in TIPC. This patch is cosmetic and does not change the
operation of TIPC in any way.
Reviewed-by: Jon Maloy <jon.maloy@ericsson.com>
Reviewed-by: Erik Hugne <erik.hugne@ericsson.com>
Signed-off-by: Wang Weidong <wangweidong1@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/port.c')
-rw-r--r-- | net/tipc/port.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/net/tipc/port.c b/net/tipc/port.c index c081a7632302..5fd4c8cec08e 100644 --- a/net/tipc/port.c +++ b/net/tipc/port.c | |||
@@ -832,17 +832,14 @@ exit: | |||
832 | */ | 832 | */ |
833 | int __tipc_disconnect(struct tipc_port *tp_ptr) | 833 | int __tipc_disconnect(struct tipc_port *tp_ptr) |
834 | { | 834 | { |
835 | int res; | ||
836 | |||
837 | if (tp_ptr->connected) { | 835 | if (tp_ptr->connected) { |
838 | tp_ptr->connected = 0; | 836 | tp_ptr->connected = 0; |
839 | /* let timer expire on it's own to avoid deadlock! */ | 837 | /* let timer expire on it's own to avoid deadlock! */ |
840 | tipc_nodesub_unsubscribe(&tp_ptr->subscription); | 838 | tipc_nodesub_unsubscribe(&tp_ptr->subscription); |
841 | res = 0; | 839 | return 0; |
842 | } else { | ||
843 | res = -ENOTCONN; | ||
844 | } | 840 | } |
845 | return res; | 841 | |
842 | return -ENOTCONN; | ||
846 | } | 843 | } |
847 | 844 | ||
848 | /* | 845 | /* |