linux-kernel - Re: 5b9f8ff7b3 ("sched/debug: Output SD flag names rather than .."): [ 320.831182] BUG: KASAN: double-free or invalid-free in sd_ctl

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 17 Nov 2020 17:21:26 +0000
From:   Valentin Schneider <valentin.schneider@....com>
To:     kernel test robot <rong.a.chen@...el.com>
Cc:     Ingo Molnar <mingo@...nel.org>, linux-kernel@...r.kernel.org,
        LKP <lkp@...ts.01.org>, lkp@...el.com
Subject: Re: 5b9f8ff7b3 ("sched/debug: Output SD flag names rather than .."): [  320.831182] BUG: KASAN: double-free or invalid-free in sd_ctl_doflags


Hi,

On 17/11/20 00:31, kernel test robot wrote:
> Greetings,
>
> 0day kernel testing robot got the below dmesg and the first bad commit is
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>
> commit 5b9f8ff7b320a34af3dbcf04edb40d9b04f22f4a
> Author:     Valentin Schneider <valentin.schneider@....com>
> AuthorDate: Mon Aug 17 12:29:52 2020 +0100
> Commit:     Ingo Molnar <mingo@...nel.org>
> CommitDate: Wed Aug 19 10:49:48 2020 +0200
>
>     sched/debug: Output SD flag names rather than their values
>     
>     Decoding the output of /proc/sys/kernel/sched_domain/cpu*/domain*/flags has
>     always been somewhat annoying, as one needs to go fetch the bit -> name
>     mapping from the source code itself. This encoding can be saved in a script
>     somewhere, but that isn't safe from flags being added, removed or even
>     shuffled around.
>     
>     What matters for debugging purposes is to get *which* flags are set in a
>     given domain, their associated value is pretty much meaningless.
>     
>     Make the sd flags debug file output flag names.
>     
>     Signed-off-by: Valentin Schneider <valentin.schneider@....com>
>     Signed-off-by: Ingo Molnar <mingo@...nel.org>
>     Acked-by: Peter Zijlstra <a.p.zijlstra@...llo.nl>
>     Link: https://lore.kernel.org/r/20200817113003.20802-7-valentin.schneider@arm.com
>
> 65c5e25316  sched/topology: Verify SD_* flags setup when sched_debug is on
> 5b9f8ff7b3  sched/debug: Output SD flag names rather than their values
> 3cea11cd5e  Linux 5.10-rc2
> +-------------------------------------------------+------------+------------+-----------+
> |                                                 | 65c5e25316 | 5b9f8ff7b3 | v5.10-rc2 |
> +-------------------------------------------------+------------+------------+-----------+
> | boot_successes                                  | 824        | 523        | 322       |
> | boot_failures                                   | 491        | 331        | 145       |
> | WARNING:at_mm/usercopy.c:#usercopy_warn         | 439        | 292        | 143       |
> | RIP:usercopy_warn                               | 439        | 292        | 143       |
> | INFO:rcu_sched_self-detected_stall_on_CPU       | 38         | 22         |           |
> | RIP:iov_iter_copy_from_user_atomic              | 26         | 15         |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c0:#] | 6          | 3          |           |
> | Kernel_panic-not_syncing                        | 39         | 23         |           |
> | RIP:ftrace_likely_update                        | 33         | 19         |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c6:#] | 5          | 3          |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c4:#] | 10         | 4          |           |
> | WARNING:kernel_stack                            | 3          | 1          |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c2:#] | 6          | 2          |           |
> | RIP:init_numa_balancing                         | 1          |            |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c5:#] | 5          | 2          |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c7:#] | 3          | 3          |           |
> | RIP:default_idle                                | 2          | 2          |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c3:#] | 4          | 4          |           |
> | BUG:kernel_hang_in_boot_stage                   | 8          | 1          | 1         |
> | WARNING:at_fs/read_write.c:#vfs_copy_file_range | 1          |            |           |
> | RIP:vfs_copy_file_range                         | 1          |            |           |
> | invoked_oom-killer:gfp_mask=0x                  | 0          | 1          |           |
> | Mem-Info                                        | 0          | 2          |           |
> | BUG:KASAN:double-free_or_invalid-free_in_s      | 0          | 10         | 1         |
> | RIP:_raw_spin_unlock_irq                        | 0          | 1          |           |
> | BUG:kernel_reboot-without-warning_in_test_stage | 0          | 1          |           |
> | BUG:soft_lockup-CPU##stuck_for#s![trinity-c1:#] | 0          | 1          |           |
> | canonical_address#:#[##]                        | 0          | 1          |           |
> | RIP:write_port                                  | 0          | 1          |           |
> +-------------------------------------------------+------------+------------+-----------+
>
> If you fix the issue, kindly add following tag
> Reported-by: kernel test robot <rong.a.chen@...el.com>
>

This has been fixed by Colin, and is in v5.10-rc4:

8d4d9c7b4333 ("sched/debug: Fix memory corruption caused by multiple small reads of flags")

Thanks.