Linux wireless drivers development
 help / color / mirror / Atom feed
From: Heyang Tan <thy15333007817@163.com>
To: chunkeey@googlemail.com
Cc: linux-wireless@vger.kernel.org
Subject: Re: [PATCH v2] p54: validate firmware record lengths
Date: Sat, 19 Sep 2026 21:33:32 +0800	[thread overview]
Message-ID: <20260919133333.317-1-thy15333007817@163.com> (raw)

Hi Christian,

Thanks for the PCI/USB testing and the review comments. I will simplify the
component-ID length check and the component-version handling in the next
revision.

Could you also help with one focused parser-only validation on Linux, if
convenient? The goal is to distinguish the malformed-record boundary from the
normal firmware compatibility testing:

1. Run the private truncated-descriptor fixture through the unpatched parser
   under a KASAN-enabled kernel (a small KUnit/test-module harness is fine;
   physical p54 hardware should not be required).
2. Record whether KASAN reports a read beyond the allocated firmware buffer
   (`fw->data + fw->size`), rather than merely crossing into a following
   record.
3. Run the same fixture with the revised patch and confirm it returns
   `-EINVAL` without a KASAN report.

Fixture SHA-256:
8dc5f14057d46ae0a08f2406d0119991c645365d2dcf8c71c6f5de6fe0f8ad53

This is a private derived parser-boundary fixture, not an official
linux-firmware image. I am not claiming a CVE or a completed dynamic
reproduction until this comparison is available. If the harness is too much
work, a brief statement that the test cannot be run is also useful.

Regards,
Heyang Tan


             reply	other threads:[~2026-09-19 13:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-19 13:33 Heyang Tan [this message]
  -- strict thread matches above, loose matches on Subject: below --
2026-10-10  4:16 [PATCH v2] p54: validate firmware record lengths Heyang Tan
2026-09-19 13:36 Heyang Tan
2026-09-19 13:26 Heyang Tan
2026-09-18  8:33 Heyang Tan
2026-09-19 11:01 ` Christian Lamparter

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=20260919133333.317-1-thy15333007817@163.com \
    --to=thy15333007817@163.com \
    --cc=chunkeey@googlemail.com \
    --cc=linux-wireless@vger.kernel.org \
    /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