Commit 7fe8097c authored by Erik Hugne's avatar Erik Hugne Committed by David S. Miller

tipc: fix nullpointer bug when subscribing to events

If a subscription request is sent to a topology server
connection, and any error occurs (malformed request, oom
or limit reached) while processing this request, TIPC should
terminate the subscriber connection. While doing so, it tries
to access fields in an already freed (or never allocated)
subscription element leading to a nullpointer exception.
We fix this by removing the subscr_terminate function and
terminate the connection immediately upon any subscription
failure.
Signed-off-by: default avatarErik Hugne <erik.hugne@ericsson.com>
Reviewed-by: default avatarJon Maloy <jon.maloy@ericsson.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3622c36f
...@@ -162,19 +162,6 @@ static void subscr_del(struct tipc_subscription *sub) ...@@ -162,19 +162,6 @@ static void subscr_del(struct tipc_subscription *sub)
atomic_dec(&tn->subscription_count); atomic_dec(&tn->subscription_count);
} }
/**
* subscr_terminate - terminate communication with a subscriber
*
* Note: Must call it in process context since it might sleep.
*/
static void subscr_terminate(struct tipc_subscription *sub)
{
struct tipc_subscriber *subscriber = sub->subscriber;
struct tipc_net *tn = net_generic(sub->net, tipc_net_id);
tipc_conn_terminate(tn->topsrv, subscriber->conid);
}
static void subscr_release(struct tipc_subscriber *subscriber) static void subscr_release(struct tipc_subscriber *subscriber)
{ {
struct tipc_subscription *sub; struct tipc_subscription *sub;
...@@ -312,16 +299,14 @@ static void subscr_conn_msg_event(struct net *net, int conid, ...@@ -312,16 +299,14 @@ static void subscr_conn_msg_event(struct net *net, int conid,
{ {
struct tipc_subscriber *subscriber = usr_data; struct tipc_subscriber *subscriber = usr_data;
struct tipc_subscription *sub = NULL; struct tipc_subscription *sub = NULL;
struct tipc_net *tn = net_generic(net, tipc_net_id);
spin_lock_bh(&subscriber->lock); spin_lock_bh(&subscriber->lock);
if (subscr_subscribe(net, (struct tipc_subscr *)buf, subscriber, subscr_subscribe(net, (struct tipc_subscr *)buf, subscriber, &sub);
&sub) < 0) {
spin_unlock_bh(&subscriber->lock);
subscr_terminate(sub);
return;
}
if (sub) if (sub)
tipc_nametbl_subscribe(sub); tipc_nametbl_subscribe(sub);
else
tipc_conn_terminate(tn->topsrv, subscriber->conid);
spin_unlock_bh(&subscriber->lock); spin_unlock_bh(&subscriber->lock);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment