diff options
| author | Allan Stephens <allan.stephens@windriver.com> | 2011-05-23 13:14:18 -0400 |
|---|---|---|
| committer | Paul Gortmaker <paul.gortmaker@windriver.com> | 2011-06-24 16:18:15 -0400 |
| commit | 5e726900380cfff50436ca6c5e08b35b3357d82a (patch) | |
| tree | cac273bd08cc73d1923c6458b98f04fd2939bff9 /net/tipc/bcast.c | |
| parent | 21e842579a5fd1ce746be4e34ac53ef67d6f0a6b (diff) | |
| download | op-kernel-dev-5e726900380cfff50436ca6c5e08b35b3357d82a.zip op-kernel-dev-5e726900380cfff50436ca6c5e08b35b3357d82a.tar.gz | |
tipc: Convert fatal broadcast sanity check to non-fatal check
Modifies the existing broadcast link sanity check that detects an
attempt to send a message off-node when there are no available
destinations so that it no longer causes a kernel panic; instead,
the check now issues a warning and stack trace and then returns
without sending the message anywhere.
Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Diffstat (limited to 'net/tipc/bcast.c')
| -rw-r--r-- | net/tipc/bcast.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/net/tipc/bcast.c b/net/tipc/bcast.c index fa68d1e..759b318 100644 --- a/net/tipc/bcast.c +++ b/net/tipc/bcast.c @@ -552,12 +552,16 @@ static int tipc_bcbearer_send(struct sk_buff *buf, if (likely(!msg_non_seq(buf_msg(buf)))) { struct tipc_msg *msg; - assert(tipc_bcast_nmap.count != 0); bcbuf_set_acks(buf, tipc_bcast_nmap.count); msg = buf_msg(buf); msg_set_non_seq(msg, 1); msg_set_mc_netid(msg, tipc_net_id); bcl->stats.sent_info++; + + if (WARN_ON(!tipc_bcast_nmap.count)) { + dump_stack(); + return 0; + } } /* Send buffer over bearers until all targets reached */ |
