On 11/19/24 08:36, Shuah Khan wrote:
On 11/18/24 18:55, Kuan-Wei Chiu wrote:
Hi Shuah,
On Mon, Nov 18, 2024 at 12:19:50PM -0700, Shuah Khan wrote:
Hi Linus,
Please pull the following kunit update for Linux 6.13-rc1.
kunit update for Linux 6.13-rc1
-- fixes user-after-free (UAF) bug in kunit_init_suite()
-- adds option to kunit tool to print just the summary of test results
-- adds option to kunit tool to print just the failed test results
-- fixes kunit_zalloc_skb() to use user passed in gfp value instead of hardcoding GFP_KERNEL
-- fixes kunit_zalloc_skb() kernel doc to include allocation flags variable
diff is attached.
Tests passed on my kunit repo:
- Build make allmodconfig
./tools/testing/kunit/kunit.py run ./tools/testing/kunit/kunit.py run --alltests
./tools/testing/kunit/kunit.py run --arch x86_64 ./tools/testing/kunit/kunit.py run --alltests --arch x86_64
thanks, -- Shuah
The following changes since commit 2d5404caa8c7bb5c4e0435f94b28834ae5456623:
Linux 6.12-rc7 (2024-11-10 14:19:35 -0800)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest tags/linux_kselftest-kunit-6.13-rc1
for you to fetch changes up to 67b6d342fb6d5abfbeb71e0f23141b9b96cf7bb1:
kunit: tool: print failed tests only (2024-11-14 09:38:19 -0700)
linux_kselftest-kunit-6.13-rc1
kunit update for Linux 6.13-rc1
-- fixes user-after-free (UAF) bug in kunit_init_suite()
-- adds option to kunit tool to print just the summary of test results
-- adds option to kunit tool to print just the failed test results
-- fixes kunit_zalloc_skb() to use user passed in gfp value instead of hardcoding GFP_KERNEL
-- fixes kunit_zalloc_skb() kernel doc to include allocation flags variable
Dan Carpenter (2): kunit: skb: use "gfp" variable instead of hardcoding GFP_KERNEL kunit: skb: add gfp to kernel doc for kunit_zalloc_skb()
David Gow (1): kunit: tool: Only print the summary
Jinjie Ruan (1): kunit: string-stream: Fix a UAF bug in kunit_init_suite()
The patch [1] intended to address the UAF issue in kunit_init_suite() is not correct and does not actually fix the problem. A v2 patch [2] with the proper fix has been sent.
Kuan-Wei, Thank you for spotting this.
Hmm. I picked this up because David reviewed it and added his Reviewed-by. I would like David's feedback on which patch we should be taking.
David, Can you take you let me know. I don't mind redoing the PR for Linus. I just have to know which patch to take.
David confirmed that v2 is the right one to take on a separate thread. I will will redo the pull request dropping the UAF v1 patch.
Linus, I will send you pull request later this week. Please ignore this one.
thanks, -- Shuah