Skip to content

AX.25 and NET/ROM fixes and improvments.

Ralf Baechle kernel-janitors linux-hams netdev 1 version Applied

Status from this message: Applied.

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 8 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/4NET: AX.25: Check ioctl arguments to avoid overflows further down the road.net/ax25/af_ax25.c+11 -61 reply
2/4NET: NETROM: When adding a route verify length of mnemonic string.net/netrom/nr_route.c+5 -02 replies
3/4NET: NETROM: Cleanup argument SIOCADDRT ioctl argument checking.net/netrom/nr_route.c+2 -44 replies
4/4NET: NETROM: Fix formatting.net/netrom/af_netrom.c+2 -11 reply

Reviews trailers per patch

1 of 4 patches has a Reviewed-by or Acked-by.

PatchDan Carpenter
1/4
2/4A-b
3/4
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/4] AX.25 and NET/ROM fixes and improvments.

AX.25 ioctl didn't do sufficient argument checking. The result of these overflows is harmless as it will be dealt with further further down in the stack but an application should get an error code when trying to set such a bogus value. To not restrict the more extreme use cases of AX.25 there is no attempt to clamp values to a "sensible" range.

The NET/ROM stack's routing ioctl didn't check the lengths of the mnemonic string that is being passed as part of the nr_route_struct structure to the kernel. In theory this could result in an oops but no memory corruption but again is fairly harmless because it requires CAP_NET_ADMIN priviledges which in practice only root has and ax25-tools don't send malformed ioctls.

Two further patches simplify the checks at the beginning of nr_rt_ioctl and do minor reformatting to nr_ioctl.

Patches 1 and 2 are meant for v3.2; 3 and 4 are only cosmetic and thus are v3.3 material.

Ralf Baechle (4):
NET: AX.25: Check ioctl arguments to avoid overflows further down the
road.
NET: NETROM: When adding a route verify length of mnemonic string.
NET: NETROM: Cleanup argument SIOCADDRT ioctl argument checking.
NET: NETROM: Fix formatting.

 net/ax25/af_ax25.c     |   17 +++++++++++------
 net/netrom/af_netrom.c |    3 ++-
 net/netrom/nr_route.c  |   11 +++++++----
 3 files changed, 20 insertions(+), 11 deletions(-)
Signature · 1 line
1.7.4.4