Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 40bf2c0c authored by Marc Kleine-Budde's avatar Marc Kleine-Budde Committed by Greg Kroah-Hartman
Browse files

can: af_can: can_rcv(): replace WARN_ONCE by pr_warn_once 



commit 8cb68751c115d176ec851ca56ecfbb411568c9e8 upstream.

If an invalid CAN frame is received, from a driver or from a tun
interface, a Kernel warning is generated.

This patch replaces the WARN_ONCE by a simple pr_warn_once, so that a
kernel, bootet with panic_on_warn, does not panic. A printk seems to be
more appropriate here.

Reported-by: default avatar <syzbot+4386709c0c1284dca827@syzkaller.appspotmail.com>
Suggested-by: default avatarDmitry Vyukov <dvyukov@google.com>
Acked-by: default avatarOliver Hartkopp <socketcan@hartkopp.net>
Signed-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: default avatarOliver Hartkopp <socketcan@hartkopp.net>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 69e78e72

net/can/af_can.c

+5 −6
Original line number Diff line number Diff line
@@ -722,13 +722,12 @@ static int can_rcv(struct sk_buff *skb, struct net_device *dev, 
	if (unlikely(!net_eq(dev_net(dev), &init_net)))

		goto drop;



	if (WARN_ONCE(dev->type != ARPHRD_CAN ||

		      skb->len != CAN_MTU ||

		      cfd->len > CAN_MAX_DLEN,

		      "PF_CAN: dropped non conform CAN skbuf: "

		      "dev type %d, len %d, datalen %d\n",

		      dev->type, skb->len, cfd->len))

	if (unlikely(dev->type != ARPHRD_CAN || skb->len != CAN_MTU ||

		     cfd->len > CAN_MAX_DLEN)) {

		pr_warn_once("PF_CAN: dropped non conform CAN skbuf: dev type %d, len %d, datalen %d\n",

			     dev->type, skb->len, cfd->len);

		goto drop;

	}



	can_receive(skb, dev);

	return NET_RX_SUCCESS;