diff options
| author | Allan Stephens <Allan.Stephens@windriver.com> | 2010-12-31 13:59:30 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-01-01 16:57:55 -0500 |
| commit | e83504f72456809cdbdbc91700d3ba6370c9da1c (patch) | |
| tree | 1391c0d5a31ea1f907d82ce43efb9a73423167a9 /net/tipc | |
| parent | 886ef52a8ce6930a9d0c58267d5b5038ac3e8d30 (diff) | |
tipc: remove pointless check for NULL prior to kfree
It is acceptable to call kfree() with NULL, so these checks are not
serving any useful purpose.
Signed-off-by: Allan Stephens <Allan.Stephens@windriver.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc')
| -rw-r--r-- | net/tipc/name_distr.c | 4 | ||||
| -rw-r--r-- | net/tipc/port.c | 4 |
2 files changed, 2 insertions, 6 deletions
diff --git a/net/tipc/name_distr.c b/net/tipc/name_distr.c index 376a30b9fd7..a6c989fd498 100644 --- a/net/tipc/name_distr.c +++ b/net/tipc/name_distr.c | |||
| @@ -240,9 +240,7 @@ static void node_is_down(struct publication *publ) | |||
| 240 | publ->type, publ->lower, publ->node, publ->ref, publ->key); | 240 | publ->type, publ->lower, publ->node, publ->ref, publ->key); |
| 241 | } | 241 | } |
| 242 | 242 | ||
| 243 | if (p) { | 243 | kfree(p); |
| 244 | kfree(p); | ||
| 245 | } | ||
| 246 | } | 244 | } |
| 247 | 245 | ||
| 248 | /** | 246 | /** |
diff --git a/net/tipc/port.c b/net/tipc/port.c index 8bacd572a9f..db14b7e0afd 100644 --- a/net/tipc/port.c +++ b/net/tipc/port.c | |||
| @@ -269,9 +269,7 @@ int tipc_deleteport(u32 ref) | |||
| 269 | buf = port_build_peer_abort_msg(p_ptr, TIPC_ERR_NO_PORT); | 269 | buf = port_build_peer_abort_msg(p_ptr, TIPC_ERR_NO_PORT); |
| 270 | tipc_nodesub_unsubscribe(&p_ptr->subscription); | 270 | tipc_nodesub_unsubscribe(&p_ptr->subscription); |
| 271 | } | 271 | } |
| 272 | if (p_ptr->user_port) { | 272 | kfree(p_ptr->user_port); |
| 273 | kfree(p_ptr->user_port); | ||
| 274 | } | ||
| 275 | 273 | ||
| 276 | spin_lock_bh(&tipc_port_list_lock); | 274 | spin_lock_bh(&tipc_port_list_lock); |
| 277 | list_del(&p_ptr->port_list); | 275 | list_del(&p_ptr->port_list); |
