summaryrefslogtreecommitdiffstats
path: root/net/phonet
diff options
context:
space:
mode:
authorOliver Hartkopp <oliver@hartkopp.net>2009-08-13 22:54:25 +0000
committerDavid S. Miller <davem@davemloft.net>2009-08-14 16:36:57 -0700
commit1758c0947605211ef953cc91d6bbdf847a21b822 (patch)
tree695e60450ec85f7e48b00949c29324c89eca7f20 /net/phonet
parent49d4b8ba97034469b941d00d5ca1e3b897394e35 (diff)
downloadop-kernel-dev-1758c0947605211ef953cc91d6bbdf847a21b822.zip
op-kernel-dev-1758c0947605211ef953cc91d6bbdf847a21b822.tar.gz
can: Use WARN_ONCE() instead of BUG_ON() for sanity check in receive path
To ensure a proper handling of CAN frames transported in skbuffs some checks need to be performed at receive time. As stated by Michael Olbrich and Luotao Fu BUG_ON() might be to restrictive. This is right as we can just drop the non conform skbuff and the Kernel can continue working. This patch replaces the BUG_ON() with a WARN_ONCE() so that the system remains healthy but we made the problem visible (once). Signed-off-by: Oliver Hartkopp <oliver@hartkopp.net> Signed-off-by: Urs Thuermann <urs@isnogud.escape.de> CC: Michael Olbrich <m.olbrich@pengutronix.de> CC: Luotao Fu <l.fu@pengutronix.de> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/phonet')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud