In switch() had do return, and never use the 'return NULL'. The 'break' after return or goto has no effect. Remove it. Signed-off-by: Wang Weidong <wangweidong1@xxxxxxxxxx> --- net/sctp/input.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/net/sctp/input.c b/net/sctp/input.c index 2a192a7..c956dc9 100644 --- a/net/sctp/input.c +++ b/net/sctp/input.c @@ -1120,15 +1120,10 @@ static struct sctp_association *__sctp_rcv_lookup_harder(struct net *net, case SCTP_CID_INIT: case SCTP_CID_INIT_ACK: return __sctp_rcv_init_lookup(net, skb, laddr, transportp); - break; default: return __sctp_rcv_walk_lookup(net, skb, laddr, transportp); - break; } - - - return NULL; } /* Lookup an association for an inbound skb. */ -- 1.7.12 -- To unsubscribe from this list: send the line "unsubscribe linux-sctp" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html