OSDN Git Service

can: isotp: tx-path: zero initialize outgoing CAN frames
authorOliver Hartkopp <socketcan@hartkopp.net>
Fri, 19 Mar 2021 10:06:19 +0000 (11:06 +0100)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Sat, 20 Mar 2021 19:21:35 +0000 (20:21 +0100)
commitb5f020f82a8e41201c6ede20fa00389d6980b223
treec114ed8cc2b3c8760a6f5900d14c243c6956370b
parent5aa3c334a449bab24519c4967f5ac2b3304c8dcf
can: isotp: tx-path: zero initialize outgoing CAN frames

Commit d4eb538e1f48 ("can: isotp: TX-path: ensure that CAN frame flags are
initialized") ensured the TX flags to be properly set for outgoing CAN
frames.

In fact the root cause of the issue results from a missing initialization
of outgoing CAN frames created by isotp. This is no problem on the CAN bus
as the CAN driver only picks the correctly defined content from the struct
can(fd)_frame. But when the outgoing frames are monitored (e.g. with
candump) we potentially leak some bytes in the unused content of
struct can(fd)_frame.

Fixes: e057dd3fc20f ("can: add ISO 15765-2:2016 transport protocol")
Cc: Marc Kleine-Budde <mkl@pengutronix.de>
Link: https://lore.kernel.org/r/20210319100619.10858-1-socketcan@hartkopp.net
Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
net/can/isotp.c