diff options
| author | Claudio Jeker <claudio@cvs.openbsd.org> | 2010-10-28 13:49:55 +0000 |
|---|---|---|
| committer | Claudio Jeker <claudio@cvs.openbsd.org> | 2010-10-28 13:49:55 +0000 |
| commit | 24c64c90c25711433236441e0509ebae8231f426 (patch) | |
| tree | 13294cc6904c6ef8dbd7449ed5f236e9172a0235 /sys/net/bridgestp.c | |
| parent | aa110b956fdc2a8f219a61129ab1da128cad8900 (diff) | |
A function that only returns NULL should return void.
bstp_input() always consumes the packet so remove the mbuf handling
dance around it.
Diffstat (limited to 'sys/net/bridgestp.c')
| -rw-r--r-- | sys/net/bridgestp.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sys/net/bridgestp.c b/sys/net/bridgestp.c index 5cba8f9201d..666bb45acb0 100644 --- a/sys/net/bridgestp.c +++ b/sys/net/bridgestp.c @@ -1,4 +1,4 @@ -/* $OpenBSD: bridgestp.c,v 1.36 2008/09/10 14:01:23 blambert Exp $ */ +/* $OpenBSD: bridgestp.c,v 1.37 2010/10/28 13:49:54 claudio Exp $ */ /* * Copyright (c) 2000 Jason L. Wright (jason@thought.net) @@ -596,7 +596,7 @@ bstp_pdu_flags(struct bstp_port *bp) return (flags); } -struct mbuf * +void bstp_input(struct bstp_state *bs, struct bstp_port *bp, struct ether_header *eh, struct mbuf *m) { @@ -656,7 +656,6 @@ bstp_input(struct bstp_state *bs, struct bstp_port *bp, out: if (m) m_freem(m); - return (NULL); } void |