From: David Ranch <linux-hams@trinnet.net>
To: David Miller <davem@davemloft.net>, yepeilin.cs@gmail.com
Cc: syzkaller-bugs@googlegroups.com, linux-kernel@vger.kernel.org,
ralf@linux-mips.org, netdev@vger.kernel.org,
linux-hams@vger.kernel.org, kuba@kernel.org,
linux-kernel-mentees@lists.linuxfoundation.org, jreuter@yaina.de
Subject: Re: [Linux-kernel-mentees] [PATCH net] AX.25: Prevent out-of-bounds read in ax25_sendmsg()
Date: Fri, 24 Jul 2020 15:25:18 -0700 [thread overview]
Message-ID: <9ed88432-ef95-f01a-9655-7c60158e3b09@trinnet.net> (raw)
In-Reply-To: <20200722.180723.102622644879670834.davem@davemloft.net>
I need to ask the following question to the Linux kernel community as
the AX25 amateur radio community is already having to
work around a few broken commits that were committed 4.1.22+:
Is anyone actually _*testing*_ these proposed changes to make sure
the AX.25 and related ecosystem still work afterwards?
I've personally tried multiple times to recruit some help to get some of
these previous commits rolled back from some
recommended kernel people including even some the original authors like
Alan Cox, etc without any success. I fear that if
more commits come into the kernel without any testing, the whole AX.25
stack will become toxic and unusable.
I am not a kernel class developer but I am *personally* willing to help
out on the testing effort and even help setup regression
topologies (VMs, containers, whatever) if there is some place that they
can be ideally run in a continuous delivery.
If there is anyone also willing to help fix some of these previous
commits and get the AX.25 stack kernel back on track,
I do have a bunch of details of the commit details, details on why those
committers THOUGHT they were a good idea, etc.
--David
KI6ZHD
On 07/22/2020 06:07 PM, David Miller wrote:
> From: Peilin Ye <yepeilin.cs@gmail.com>
> Date: Wed, 22 Jul 2020 12:05:12 -0400
>
>> Checks on `addr_len` and `usax->sax25_ndigis` are insufficient.
>> ax25_sendmsg() can go out of bounds when `usax->sax25_ndigis` equals to 7
>> or 8. Fix it.
>>
>> It is safe to remove `usax->sax25_ndigis > AX25_MAX_DIGIS`, since
>> `addr_len` is guaranteed to be less than or equal to
>> `sizeof(struct full_sockaddr_ax25)`
>>
>> Signed-off-by: Peilin Ye <yepeilin.cs@gmail.com>
> Applied.
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
prev parent reply other threads:[~2020-07-24 23:29 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-22 16:05 [Linux-kernel-mentees] [PATCH net] AX.25: Prevent out-of-bounds read in ax25_sendmsg() Peilin Ye
2020-07-23 1:07 ` David Miller
2020-07-24 22:25 ` David Ranch [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9ed88432-ef95-f01a-9655-7c60158e3b09@trinnet.net \
--to=linux-hams@trinnet.net \
--cc=davem@davemloft.net \
--cc=jreuter@yaina.de \
--cc=kuba@kernel.org \
--cc=linux-hams@vger.kernel.org \
--cc=linux-kernel-mentees@lists.linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=ralf@linux-mips.org \
--cc=syzkaller-bugs@googlegroups.com \
--cc=yepeilin.cs@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).