Re: [PATCH net] netrom: do some basic forms of validation on incoming frames
Simon Horman <[email protected]> Thu, 9 Apr 2026 20:03:28 +0100
| Newsgroups | gmane.linux.hams,gmane.linux.network,gmane.linux.kernel,gmane.linux.kernel.stable |
|---|---|
| Message-ID | <[email protected]> |
On Tue, Apr 07, 2026 at 10:45:31AM +0200, Greg Kroah-Hartman wrote: > There is a lack of much validation of frame size coming from a > netrom-based device. While these devices are "trusted" doing some > sanity checks is good to at least keep the fuzzing tools happy when they > stumble across this ancient protocol and light up with a range of bug > reports. > > Cc: "David S. Miller" <[email protected]> > Cc: Eric Dumazet <[email protected]> > Cc: Jakub Kicinski <[email protected]> > Cc: Paolo Abeni <[email protected]> > Cc: Simon Horman <[email protected]> > Cc: [email protected] > Assisted-by: gregkh_clanker_2000 > Reviewed-by: Yizhe Zhuang <[email protected]> > Cc: stable <[email protected]> > Signed-off-by: Greg Kroah-Hartman <[email protected]> Hi Greg 2000! I expect that checking skb->len isn't sufficient here and pskb_may_pull needs to be used to ensure that the data is also available in the linear section of the skb. Also, although I'm all for incremental enhancements, I do suspect that similar problems exist in the call chain of these functions. ...