can: rx-offload: can_rx_offload_offload_one(): use ERR_PTR() to propagate error value...
authorMarc Kleine-Budde <mkl@pengutronix.de>
Wed, 9 Oct 2019 19:00:32 +0000 (21:00 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 5 Dec 2019 14:37:01 +0000 (15:37 +0100)
commitf0aad062e96ed750e3daaec68ca5685dcf70f90f
tree3cf1b1dcc1326b7fd7716430d96d101e6660e896
parent8c7dd979e1f70beece01e1fbd6006e1a4a81e880
can: rx-offload: can_rx_offload_offload_one(): use ERR_PTR() to propagate error value in case of errors

[ Upstream commit d763ab3044f0bf50bd0e6179f6b2cf1c125d1d94 ]

Before this patch can_rx_offload_offload_one() returns a pointer to a
skb containing the read CAN frame or a NULL pointer.

However the meaning of the NULL pointer is ambiguous, it can either mean
the requested mailbox is empty or there was an error.

This patch fixes this situation by returning:
- pointer to skb on success
- NULL pointer if mailbox is empty
- ERR_PTR() in case of an error

All users of can_rx_offload_offload_one() have been adopted, no
functional change intended.

Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/can/rx-offload.c