linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [syzbot] linux-next test error: WARNING: held lock freed in kernfs_destroy_root
@ 2021-11-26  6:38 syzbot
  2021-11-28 19:19 ` Phi Nguyen
  0 siblings, 1 reply; 2+ messages in thread
From: syzbot @ 2021-11-26  6:38 UTC (permalink / raw)
  To: gregkh, linux-kernel, linux-next, sfr, syzkaller-bugs, tj

Hello,

syzbot found the following issue on:

HEAD commit:    f30a24ed97b4 Add linux-next specific files for 20211126
git tree:       linux-next
console output: https://syzkaller.appspot.com/x/log.txt?x=16802fa1b00000
kernel config:  https://syzkaller.appspot.com/x/.config?x=be9183de0824e4d7
dashboard link: https://syzkaller.appspot.com/bug?extid=9e5d9f74a10f9590610b
compiler:       gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2

IMPORTANT: if you fix the issue, please add the following tag to the commit:
Reported-by: syzbot+9e5d9f74a10f9590610b@syzkaller.appspotmail.com

cgroup: Unknown subsys name 'net'
=========================
WARNING: held lock freed!
5.16.0-rc2-next-20211126-syzkaller #0 Not tainted
-------------------------
syz-executor/6532 is freeing memory ffff88807f06e400-ffff88807f06e5ff, with a lock still held there!
ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_remove fs/kernfs/dir.c:1396 [inline]
ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_destroy_root+0x81/0xb0 fs/kernfs/dir.c:964
2 locks held by syz-executor/6532:
 #0: ffffffff8bbc5d48 (cgroup_mutex){+.+.}-{3:3}, at: cgroup_lock_and_drain_offline+0xa5/0x900 kernel/cgroup/cgroup.c:2998
 #1: ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_remove fs/kernfs/dir.c:1396 [inline]
 #1: ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_destroy_root+0x81/0xb0 fs/kernfs/dir.c:964

stack backtrace:
CPU: 0 PID: 6532 Comm: syz-executor Not tainted 5.16.0-rc2-next-20211126-syzkaller #0
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
Call Trace:
 <TASK>
 __dump_stack lib/dump_stack.c:88 [inline]
 dump_stack_lvl+0xcd/0x134 lib/dump_stack.c:106
 print_freed_lock_bug kernel/locking/lockdep.c:6388 [inline]
 debug_check_no_locks_freed.cold+0x9d/0xa9 kernel/locking/lockdep.c:6421
 slab_free_hook mm/slub.c:1695 [inline]
 slab_free_freelist_hook+0x73/0x1c0 mm/slub.c:1749
 slab_free mm/slub.c:3513 [inline]
 kfree+0xe0/0x430 mm/slub.c:4561
 kernfs_put.part.0+0x331/0x540 fs/kernfs/dir.c:548
 kernfs_put+0x42/0x50 fs/kernfs/dir.c:513
 __kernfs_remove+0x7a3/0xb20 fs/kernfs/dir.c:1382
 kernfs_remove fs/kernfs/dir.c:1397 [inline]
 kernfs_destroy_root+0x89/0xb0 fs/kernfs/dir.c:964
 cgroup_setup_root+0x3a6/0xad0 kernel/cgroup/cgroup.c:2077
 cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
 cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
 vfs_get_tree+0x89/0x2f0 fs/super.c:1500
 do_new_mount fs/namespace.c:2988 [inline]
 path_mount+0x1320/0x1fa0 fs/namespace.c:3318
 do_mount fs/namespace.c:3331 [inline]
 __do_sys_mount fs/namespace.c:3539 [inline]
 __se_sys_mount fs/namespace.c:3516 [inline]
 __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
 do_syscall_x64 arch/x86/entry/common.c:50 [inline]
 do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
 entry_SYSCALL_64_after_hwframe+0x44/0xae
RIP: 0033:0x7f517c6f401a
Code: 48 c7 c2 bc ff ff ff f7 d8 64 89 02 b8 ff ff ff ff eb d2 e8 b8 04 00 00 0f 1f 84 00 00 00 00 00 49 89 ca b8 a5 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 c7 c1 bc ff ff ff f7 d8 64 89 01 48
RSP: 002b:00007ffe8ba71698 EFLAGS: 00000246 ORIG_RAX: 00000000000000a5
RAX: ffffffffffffffda RBX: 00007ffe8ba71828 RCX: 00007f517c6f401a
RDX: 00007f517c756fd6 RSI: 00007f517c74d29a RDI: 00007f517c74bd71
RBP: 00007f517c74d29a R08: 00007f517c74d3f7 R09: 0000000000000026
R10: 0000000000000000 R11: 0000000000000246 R12: 00007ffe8ba716a0
R13: 00007ffe8ba71848 R14: 00007ffe8ba71770 R15: 00007f517c74d3f1
 </TASK>
==================================================================
BUG: KASAN: use-after-free in __up_write kernel/locking/rwsem.c:1318 [inline]
BUG: KASAN: use-after-free in up_write+0x3ac/0x470 kernel/locking/rwsem.c:1576
Read of size 8 at addr ffff88807f06e540 by task syz-executor/6532

CPU: 1 PID: 6532 Comm: syz-executor Not tainted 5.16.0-rc2-next-20211126-syzkaller #0
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
Call Trace:
 <TASK>
 __dump_stack lib/dump_stack.c:88 [inline]
 dump_stack_lvl+0xcd/0x134 lib/dump_stack.c:106
 print_address_description.constprop.0.cold+0xa5/0x3ed mm/kasan/report.c:247
 __kasan_report mm/kasan/report.c:433 [inline]
 kasan_report.cold+0x83/0xdf mm/kasan/report.c:450
 __up_write kernel/locking/rwsem.c:1318 [inline]
 up_write+0x3ac/0x470 kernel/locking/rwsem.c:1576
 cgroup_setup_root+0x3a6/0xad0 kernel/cgroup/cgroup.c:2077
 cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
 cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
 vfs_get_tree+0x89/0x2f0 fs/super.c:1500
 do_new_mount fs/namespace.c:2988 [inline]
 path_mount+0x1320/0x1fa0 fs/namespace.c:3318
 do_mount fs/namespace.c:3331 [inline]
 __do_sys_mount fs/namespace.c:3539 [inline]
 __se_sys_mount fs/namespace.c:3516 [inline]
 __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
 do_syscall_x64 arch/x86/entry/common.c:50 [inline]
 do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
 entry_SYSCALL_64_after_hwframe+0x44/0xae
RIP: 0033:0x7f517c6f401a
Code: 48 c7 c2 bc ff ff ff f7 d8 64 89 02 b8 ff ff ff ff eb d2 e8 b8 04 00 00 0f 1f 84 00 00 00 00 00 49 89 ca b8 a5 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 c7 c1 bc ff ff ff f7 d8 64 89 01 48
RSP: 002b:00007ffe8ba71698 EFLAGS: 00000246 ORIG_RAX: 00000000000000a5
RAX: ffffffffffffffda RBX: 00007ffe8ba71828 RCX: 00007f517c6f401a
RDX: 00007f517c756fd6 RSI: 00007f517c74d29a RDI: 00007f517c74bd71
RBP: 00007f517c74d29a R08: 00007f517c74d3f7 R09: 0000000000000026
R10: 0000000000000000 R11: 0000000000000246 R12: 00007ffe8ba716a0
R13: 00007ffe8ba71848 R14: 00007ffe8ba71770 R15: 00007f517c74d3f1
 </TASK>

Allocated by task 6532:
 kasan_save_stack+0x1e/0x50 mm/kasan/common.c:38
 kasan_set_track mm/kasan/common.c:46 [inline]
 set_alloc_info mm/kasan/common.c:434 [inline]
 ____kasan_kmalloc mm/kasan/common.c:513 [inline]
 ____kasan_kmalloc mm/kasan/common.c:472 [inline]
 __kasan_kmalloc+0xa9/0xd0 mm/kasan/common.c:522
 kmalloc include/linux/slab.h:590 [inline]
 kzalloc include/linux/slab.h:724 [inline]
 kernfs_create_root+0x4c/0x410 fs/kernfs/dir.c:913
 cgroup_setup_root+0x243/0xad0 kernel/cgroup/cgroup.c:2018
 cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
 cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
 vfs_get_tree+0x89/0x2f0 fs/super.c:1500
 do_new_mount fs/namespace.c:2988 [inline]
 path_mount+0x1320/0x1fa0 fs/namespace.c:3318
 do_mount fs/namespace.c:3331 [inline]
 __do_sys_mount fs/namespace.c:3539 [inline]
 __se_sys_mount fs/namespace.c:3516 [inline]
 __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
 do_syscall_x64 arch/x86/entry/common.c:50 [inline]
 do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
 entry_SYSCALL_64_after_hwframe+0x44/0xae

Freed by task 6532:
 kasan_save_stack+0x1e/0x50 mm/kasan/common.c:38
 kasan_set_track+0x21/0x30 mm/kasan/common.c:46
 kasan_set_free_info+0x20/0x30 mm/kasan/generic.c:370
 ____kasan_slab_free mm/kasan/common.c:366 [inline]
 ____kasan_slab_free mm/kasan/common.c:328 [inline]
 __kasan_slab_free+0x103/0x170 mm/kasan/common.c:374
 kasan_slab_free include/linux/kasan.h:235 [inline]
 slab_free_hook mm/slub.c:1723 [inline]
 slab_free_freelist_hook+0x8b/0x1c0 mm/slub.c:1749
 slab_free mm/slub.c:3513 [inline]
 kfree+0xe0/0x430 mm/slub.c:4561
 kernfs_put.part.0+0x331/0x540 fs/kernfs/dir.c:548
 kernfs_put+0x42/0x50 fs/kernfs/dir.c:513
 __kernfs_remove+0x7a3/0xb20 fs/kernfs/dir.c:1382
 kernfs_remove fs/kernfs/dir.c:1397 [inline]
 kernfs_destroy_root+0x89/0xb0 fs/kernfs/dir.c:964
 cgroup_setup_root+0x3a6/0xad0 kernel/cgroup/cgroup.c:2077
 cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
 cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
 vfs_get_tree+0x89/0x2f0 fs/super.c:1500
 do_new_mount fs/namespace.c:2988 [inline]
 path_mount+0x1320/0x1fa0 fs/namespace.c:3318
 do_mount fs/namespace.c:3331 [inline]
 __do_sys_mount fs/namespace.c:3539 [inline]
 __se_sys_mount fs/namespace.c:3516 [inline]
 __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
 do_syscall_x64 arch/x86/entry/common.c:50 [inline]
 do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
 entry_SYSCALL_64_after_hwframe+0x44/0xae

The buggy address belongs to the object at ffff88807f06e400
 which belongs to the cache kmalloc-512 of size 512
The buggy address is located 320 bytes inside of
 512-byte region [ffff88807f06e400, ffff88807f06e600)
The buggy address belongs to the page:
page:ffffea0001fc1b00 refcount:1 mapcount:0 mapping:0000000000000000 index:0x0 pfn:0x7f06c
head:ffffea0001fc1b00 order:2 compound_mapcount:0 compound_pincount:0
flags: 0xfff00000010200(slab|head|node=0|zone=1|lastcpupid=0x7ff)
raw: 00fff00000010200 ffffea00008d1b00 dead000000000002 ffff888010c41c80
raw: 0000000000000000 0000000080100010 00000001ffffffff 0000000000000000
page dumped because: kasan: bad access detected
page_owner tracks the page as allocated
page last allocated via order 2, migratetype Unmovable, gfp_mask 0xd20c0(__GFP_IO|__GFP_FS|__GFP_NOWARN|__GFP_NORETRY|__GFP_COMP|__GFP_NOMEMALLOC), pid 5789, ts 47991282663, free_ts 38208673784
 prep_new_page mm/page_alloc.c:2433 [inline]
 get_page_from_freelist+0xa72/0x2f40 mm/page_alloc.c:4164
 __alloc_pages+0x1b2/0x500 mm/page_alloc.c:5376
 alloc_pages+0x1a7/0x300 mm/mempolicy.c:2192
 alloc_slab_page mm/slub.c:1793 [inline]
 allocate_slab mm/slub.c:1930 [inline]
 new_slab+0x261/0x460 mm/slub.c:1993
 ___slab_alloc+0x798/0xf30 mm/slub.c:3022
 __slab_alloc.constprop.0+0x4d/0xa0 mm/slub.c:3109
 slab_alloc_node mm/slub.c:3200 [inline]
 slab_alloc mm/slub.c:3242 [inline]
 __kmalloc+0x2fb/0x340 mm/slub.c:4419
 kmalloc include/linux/slab.h:595 [inline]
 load_elf_phdrs+0x103/0x210 fs/binfmt_elf.c:479
 load_elf_binary+0x18f4/0x4da0 fs/binfmt_elf.c:959
 search_binary_handler fs/exec.c:1724 [inline]
 exec_binprm fs/exec.c:1765 [inline]
 bprm_execve fs/exec.c:1834 [inline]
 bprm_execve+0x7ef/0x19b0 fs/exec.c:1796
 do_execveat_common+0x5e3/0x780 fs/exec.c:1923
 do_execve fs/exec.c:1991 [inline]
 __do_sys_execve fs/exec.c:2067 [inline]
 __se_sys_execve fs/exec.c:2062 [inline]
 __x64_sys_execve+0x8f/0xc0 fs/exec.c:2062
 do_syscall_x64 arch/x86/entry/common.c:50 [inline]
 do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
 entry_SYSCALL_64_after_hwframe+0x44/0xae
page last free stack trace:
 reset_page_owner include/linux/page_owner.h:24 [inline]
 free_pages_prepare mm/page_alloc.c:1351 [inline]
 free_pcp_prepare+0x414/0xb60 mm/page_alloc.c:1403
 free_unref_page_prepare mm/page_alloc.c:3324 [inline]
 free_unref_page+0x19/0x690 mm/page_alloc.c:3403
 qlink_free mm/kasan/quarantine.c:146 [inline]
 qlist_free_all+0x5a/0xf0 mm/kasan/quarantine.c:165
 kasan_quarantine_reduce+0x180/0x200 mm/kasan/quarantine.c:272
 __kasan_slab_alloc+0xa2/0xc0 mm/kasan/common.c:444
 kasan_slab_alloc include/linux/kasan.h:259 [inline]
 slab_post_alloc_hook mm/slab.h:519 [inline]
 slab_alloc_node mm/slub.c:3234 [inline]
 slab_alloc mm/slub.c:3242 [inline]
 __kmalloc+0x1e7/0x340 mm/slub.c:4419
 kmalloc include/linux/slab.h:595 [inline]
 tomoyo_realpath_from_path+0xc3/0x620 security/tomoyo/realpath.c:254
 tomoyo_get_realpath security/tomoyo/file.c:151 [inline]
 tomoyo_path_perm+0x21b/0x400 security/tomoyo/file.c:822
 security_inode_getattr+0xcf/0x140 security/security.c:1334
 vfs_getattr fs/stat.c:157 [inline]
 vfs_statx+0x164/0x390 fs/stat.c:225
 vfs_fstatat fs/stat.c:243 [inline]
 vfs_lstat include/linux/fs.h:3358 [inline]
 __do_sys_newlstat+0x91/0x110 fs/stat.c:398
 do_syscall_x64 arch/x86/entry/common.c:50 [inline]
 do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
 entry_SYSCALL_64_after_hwframe+0x44/0xae

Memory state around the buggy address:
 ffff88807f06e400: fa fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
 ffff88807f06e480: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>ffff88807f06e500: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
                                           ^
 ffff88807f06e580: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
 ffff88807f06e600: fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc
==================================================================


---
This report is generated by a bot. It may contain errors.
See https://goo.gl/tpsmEJ for more information about syzbot.
syzbot engineers can be reached at syzkaller@googlegroups.com.

syzbot will keep track of this issue. See:
https://goo.gl/tpsmEJ#status for how to communicate with syzbot.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [syzbot] linux-next test error: WARNING: held lock freed in kernfs_destroy_root
  2021-11-26  6:38 [syzbot] linux-next test error: WARNING: held lock freed in kernfs_destroy_root syzbot
@ 2021-11-28 19:19 ` Phi Nguyen
  0 siblings, 0 replies; 2+ messages in thread
From: Phi Nguyen @ 2021-11-28 19:19 UTC (permalink / raw)
  To: syzbot, gregkh, linux-kernel, linux-next, sfr, syzkaller-bugs, tj

On 11/26/2021 2:38 PM, syzbot wrote:
> Hello,
> 
> syzbot found the following issue on:
> 
> HEAD commit:    f30a24ed97b4 Add linux-next specific files for 20211126
> git tree:       linux-next
> console output: https://syzkaller.appspot.com/x/log.txt?x=16802fa1b00000
> kernel config:  https://syzkaller.appspot.com/x/.config?x=be9183de0824e4d7
> dashboard link: https://syzkaller.appspot.com/bug?extid=9e5d9f74a10f9590610b
> compiler:       gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2
> 
> IMPORTANT: if you fix the issue, please add the following tag to the commit:
> Reported-by: syzbot+9e5d9f74a10f9590610b@syzkaller.appspotmail.com
> 
> cgroup: Unknown subsys name 'net'
> =========================
> WARNING: held lock freed!
> 5.16.0-rc2-next-20211126-syzkaller #0 Not tainted
> -------------------------
> syz-executor/6532 is freeing memory ffff88807f06e400-ffff88807f06e5ff, with a lock still held there!
> ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_remove fs/kernfs/dir.c:1396 [inline]
> ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_destroy_root+0x81/0xb0 fs/kernfs/dir.c:964
> 2 locks held by syz-executor/6532:
>   #0: ffffffff8bbc5d48 (cgroup_mutex){+.+.}-{3:3}, at: cgroup_lock_and_drain_offline+0xa5/0x900 kernel/cgroup/cgroup.c:2998
>   #1: ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_remove fs/kernfs/dir.c:1396 [inline]
>   #1: ffff88807f06e548 (&root->kernfs_rwsem){++++}-{3:3}, at: kernfs_destroy_root+0x81/0xb0 fs/kernfs/dir.c:964
> 
> stack backtrace:
> CPU: 0 PID: 6532 Comm: syz-executor Not tainted 5.16.0-rc2-next-20211126-syzkaller #0
> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
> Call Trace:
>   <TASK>
>   __dump_stack lib/dump_stack.c:88 [inline]
>   dump_stack_lvl+0xcd/0x134 lib/dump_stack.c:106
>   print_freed_lock_bug kernel/locking/lockdep.c:6388 [inline]
>   debug_check_no_locks_freed.cold+0x9d/0xa9 kernel/locking/lockdep.c:6421
>   slab_free_hook mm/slub.c:1695 [inline]
>   slab_free_freelist_hook+0x73/0x1c0 mm/slub.c:1749
>   slab_free mm/slub.c:3513 [inline]
>   kfree+0xe0/0x430 mm/slub.c:4561
>   kernfs_put.part.0+0x331/0x540 fs/kernfs/dir.c:548
>   kernfs_put+0x42/0x50 fs/kernfs/dir.c:513
>   __kernfs_remove+0x7a3/0xb20 fs/kernfs/dir.c:1382
>   kernfs_remove fs/kernfs/dir.c:1397 [inline]
>   kernfs_destroy_root+0x89/0xb0 fs/kernfs/dir.c:964
>   cgroup_setup_root+0x3a6/0xad0 kernel/cgroup/cgroup.c:2077
>   cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
>   cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
>   vfs_get_tree+0x89/0x2f0 fs/super.c:1500
>   do_new_mount fs/namespace.c:2988 [inline]
>   path_mount+0x1320/0x1fa0 fs/namespace.c:3318
>   do_mount fs/namespace.c:3331 [inline]
>   __do_sys_mount fs/namespace.c:3539 [inline]
>   __se_sys_mount fs/namespace.c:3516 [inline]
>   __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
>   do_syscall_x64 arch/x86/entry/common.c:50 [inline]
>   do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
>   entry_SYSCALL_64_after_hwframe+0x44/0xae
> RIP: 0033:0x7f517c6f401a
> Code: 48 c7 c2 bc ff ff ff f7 d8 64 89 02 b8 ff ff ff ff eb d2 e8 b8 04 00 00 0f 1f 84 00 00 00 00 00 49 89 ca b8 a5 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 c7 c1 bc ff ff ff f7 d8 64 89 01 48
> RSP: 002b:00007ffe8ba71698 EFLAGS: 00000246 ORIG_RAX: 00000000000000a5
> RAX: ffffffffffffffda RBX: 00007ffe8ba71828 RCX: 00007f517c6f401a
> RDX: 00007f517c756fd6 RSI: 00007f517c74d29a RDI: 00007f517c74bd71
> RBP: 00007f517c74d29a R08: 00007f517c74d3f7 R09: 0000000000000026
> R10: 0000000000000000 R11: 0000000000000246 R12: 00007ffe8ba716a0
> R13: 00007ffe8ba71848 R14: 00007ffe8ba71770 R15: 00007f517c74d3f1
>   </TASK>
> ==================================================================
> BUG: KASAN: use-after-free in __up_write kernel/locking/rwsem.c:1318 [inline]
> BUG: KASAN: use-after-free in up_write+0x3ac/0x470 kernel/locking/rwsem.c:1576
> Read of size 8 at addr ffff88807f06e540 by task syz-executor/6532
> 
> CPU: 1 PID: 6532 Comm: syz-executor Not tainted 5.16.0-rc2-next-20211126-syzkaller #0
> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
> Call Trace:
>   <TASK>
>   __dump_stack lib/dump_stack.c:88 [inline]
>   dump_stack_lvl+0xcd/0x134 lib/dump_stack.c:106
>   print_address_description.constprop.0.cold+0xa5/0x3ed mm/kasan/report.c:247
>   __kasan_report mm/kasan/report.c:433 [inline]
>   kasan_report.cold+0x83/0xdf mm/kasan/report.c:450
>   __up_write kernel/locking/rwsem.c:1318 [inline]
>   up_write+0x3ac/0x470 kernel/locking/rwsem.c:1576
>   cgroup_setup_root+0x3a6/0xad0 kernel/cgroup/cgroup.c:2077
>   cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
>   cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
>   vfs_get_tree+0x89/0x2f0 fs/super.c:1500
>   do_new_mount fs/namespace.c:2988 [inline]
>   path_mount+0x1320/0x1fa0 fs/namespace.c:3318
>   do_mount fs/namespace.c:3331 [inline]
>   __do_sys_mount fs/namespace.c:3539 [inline]
>   __se_sys_mount fs/namespace.c:3516 [inline]
>   __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
>   do_syscall_x64 arch/x86/entry/common.c:50 [inline]
>   do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
>   entry_SYSCALL_64_after_hwframe+0x44/0xae
> RIP: 0033:0x7f517c6f401a
> Code: 48 c7 c2 bc ff ff ff f7 d8 64 89 02 b8 ff ff ff ff eb d2 e8 b8 04 00 00 0f 1f 84 00 00 00 00 00 49 89 ca b8 a5 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 c7 c1 bc ff ff ff f7 d8 64 89 01 48
> RSP: 002b:00007ffe8ba71698 EFLAGS: 00000246 ORIG_RAX: 00000000000000a5
> RAX: ffffffffffffffda RBX: 00007ffe8ba71828 RCX: 00007f517c6f401a
> RDX: 00007f517c756fd6 RSI: 00007f517c74d29a RDI: 00007f517c74bd71
> RBP: 00007f517c74d29a R08: 00007f517c74d3f7 R09: 0000000000000026
> R10: 0000000000000000 R11: 0000000000000246 R12: 00007ffe8ba716a0
> R13: 00007ffe8ba71848 R14: 00007ffe8ba71770 R15: 00007f517c74d3f1
>   </TASK>
> 
> Allocated by task 6532:
>   kasan_save_stack+0x1e/0x50 mm/kasan/common.c:38
>   kasan_set_track mm/kasan/common.c:46 [inline]
>   set_alloc_info mm/kasan/common.c:434 [inline]
>   ____kasan_kmalloc mm/kasan/common.c:513 [inline]
>   ____kasan_kmalloc mm/kasan/common.c:472 [inline]
>   __kasan_kmalloc+0xa9/0xd0 mm/kasan/common.c:522
>   kmalloc include/linux/slab.h:590 [inline]
>   kzalloc include/linux/slab.h:724 [inline]
>   kernfs_create_root+0x4c/0x410 fs/kernfs/dir.c:913
>   cgroup_setup_root+0x243/0xad0 kernel/cgroup/cgroup.c:2018
>   cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
>   cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
>   vfs_get_tree+0x89/0x2f0 fs/super.c:1500
>   do_new_mount fs/namespace.c:2988 [inline]
>   path_mount+0x1320/0x1fa0 fs/namespace.c:3318
>   do_mount fs/namespace.c:3331 [inline]
>   __do_sys_mount fs/namespace.c:3539 [inline]
>   __se_sys_mount fs/namespace.c:3516 [inline]
>   __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
>   do_syscall_x64 arch/x86/entry/common.c:50 [inline]
>   do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
>   entry_SYSCALL_64_after_hwframe+0x44/0xae
> 
> Freed by task 6532:
>   kasan_save_stack+0x1e/0x50 mm/kasan/common.c:38
>   kasan_set_track+0x21/0x30 mm/kasan/common.c:46
>   kasan_set_free_info+0x20/0x30 mm/kasan/generic.c:370
>   ____kasan_slab_free mm/kasan/common.c:366 [inline]
>   ____kasan_slab_free mm/kasan/common.c:328 [inline]
>   __kasan_slab_free+0x103/0x170 mm/kasan/common.c:374
>   kasan_slab_free include/linux/kasan.h:235 [inline]
>   slab_free_hook mm/slub.c:1723 [inline]
>   slab_free_freelist_hook+0x8b/0x1c0 mm/slub.c:1749
>   slab_free mm/slub.c:3513 [inline]
>   kfree+0xe0/0x430 mm/slub.c:4561
>   kernfs_put.part.0+0x331/0x540 fs/kernfs/dir.c:548
>   kernfs_put+0x42/0x50 fs/kernfs/dir.c:513
>   __kernfs_remove+0x7a3/0xb20 fs/kernfs/dir.c:1382
>   kernfs_remove fs/kernfs/dir.c:1397 [inline]
>   kernfs_destroy_root+0x89/0xb0 fs/kernfs/dir.c:964
>   cgroup_setup_root+0x3a6/0xad0 kernel/cgroup/cgroup.c:2077
>   cgroup1_root_to_use kernel/cgroup/cgroup-v1.c:1194 [inline]
>   cgroup1_get_tree+0xd33/0x1390 kernel/cgroup/cgroup-v1.c:1211
>   vfs_get_tree+0x89/0x2f0 fs/super.c:1500
>   do_new_mount fs/namespace.c:2988 [inline]
>   path_mount+0x1320/0x1fa0 fs/namespace.c:3318
>   do_mount fs/namespace.c:3331 [inline]
>   __do_sys_mount fs/namespace.c:3539 [inline]
>   __se_sys_mount fs/namespace.c:3516 [inline]
>   __x64_sys_mount+0x27f/0x300 fs/namespace.c:3516
>   do_syscall_x64 arch/x86/entry/common.c:50 [inline]
>   do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
>   entry_SYSCALL_64_after_hwframe+0x44/0xae
> 
> The buggy address belongs to the object at ffff88807f06e400
>   which belongs to the cache kmalloc-512 of size 512
> The buggy address is located 320 bytes inside of
>   512-byte region [ffff88807f06e400, ffff88807f06e600)
> The buggy address belongs to the page:
> page:ffffea0001fc1b00 refcount:1 mapcount:0 mapping:0000000000000000 index:0x0 pfn:0x7f06c
> head:ffffea0001fc1b00 order:2 compound_mapcount:0 compound_pincount:0
> flags: 0xfff00000010200(slab|head|node=0|zone=1|lastcpupid=0x7ff)
> raw: 00fff00000010200 ffffea00008d1b00 dead000000000002 ffff888010c41c80
> raw: 0000000000000000 0000000080100010 00000001ffffffff 0000000000000000
> page dumped because: kasan: bad access detected
> page_owner tracks the page as allocated
> page last allocated via order 2, migratetype Unmovable, gfp_mask 0xd20c0(__GFP_IO|__GFP_FS|__GFP_NOWARN|__GFP_NORETRY|__GFP_COMP|__GFP_NOMEMALLOC), pid 5789, ts 47991282663, free_ts 38208673784
>   prep_new_page mm/page_alloc.c:2433 [inline]
>   get_page_from_freelist+0xa72/0x2f40 mm/page_alloc.c:4164
>   __alloc_pages+0x1b2/0x500 mm/page_alloc.c:5376
>   alloc_pages+0x1a7/0x300 mm/mempolicy.c:2192
>   alloc_slab_page mm/slub.c:1793 [inline]
>   allocate_slab mm/slub.c:1930 [inline]
>   new_slab+0x261/0x460 mm/slub.c:1993
>   ___slab_alloc+0x798/0xf30 mm/slub.c:3022
>   __slab_alloc.constprop.0+0x4d/0xa0 mm/slub.c:3109
>   slab_alloc_node mm/slub.c:3200 [inline]
>   slab_alloc mm/slub.c:3242 [inline]
>   __kmalloc+0x2fb/0x340 mm/slub.c:4419
>   kmalloc include/linux/slab.h:595 [inline]
>   load_elf_phdrs+0x103/0x210 fs/binfmt_elf.c:479
>   load_elf_binary+0x18f4/0x4da0 fs/binfmt_elf.c:959
>   search_binary_handler fs/exec.c:1724 [inline]
>   exec_binprm fs/exec.c:1765 [inline]
>   bprm_execve fs/exec.c:1834 [inline]
>   bprm_execve+0x7ef/0x19b0 fs/exec.c:1796
>   do_execveat_common+0x5e3/0x780 fs/exec.c:1923
>   do_execve fs/exec.c:1991 [inline]
>   __do_sys_execve fs/exec.c:2067 [inline]
>   __se_sys_execve fs/exec.c:2062 [inline]
>   __x64_sys_execve+0x8f/0xc0 fs/exec.c:2062
>   do_syscall_x64 arch/x86/entry/common.c:50 [inline]
>   do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
>   entry_SYSCALL_64_after_hwframe+0x44/0xae
> page last free stack trace:
>   reset_page_owner include/linux/page_owner.h:24 [inline]
>   free_pages_prepare mm/page_alloc.c:1351 [inline]
>   free_pcp_prepare+0x414/0xb60 mm/page_alloc.c:1403
>   free_unref_page_prepare mm/page_alloc.c:3324 [inline]
>   free_unref_page+0x19/0x690 mm/page_alloc.c:3403
>   qlink_free mm/kasan/quarantine.c:146 [inline]
>   qlist_free_all+0x5a/0xf0 mm/kasan/quarantine.c:165
>   kasan_quarantine_reduce+0x180/0x200 mm/kasan/quarantine.c:272
>   __kasan_slab_alloc+0xa2/0xc0 mm/kasan/common.c:444
>   kasan_slab_alloc include/linux/kasan.h:259 [inline]
>   slab_post_alloc_hook mm/slab.h:519 [inline]
>   slab_alloc_node mm/slub.c:3234 [inline]
>   slab_alloc mm/slub.c:3242 [inline]
>   __kmalloc+0x1e7/0x340 mm/slub.c:4419
>   kmalloc include/linux/slab.h:595 [inline]
>   tomoyo_realpath_from_path+0xc3/0x620 security/tomoyo/realpath.c:254
>   tomoyo_get_realpath security/tomoyo/file.c:151 [inline]
>   tomoyo_path_perm+0x21b/0x400 security/tomoyo/file.c:822
>   security_inode_getattr+0xcf/0x140 security/security.c:1334
>   vfs_getattr fs/stat.c:157 [inline]
>   vfs_statx+0x164/0x390 fs/stat.c:225
>   vfs_fstatat fs/stat.c:243 [inline]
>   vfs_lstat include/linux/fs.h:3358 [inline]
>   __do_sys_newlstat+0x91/0x110 fs/stat.c:398
>   do_syscall_x64 arch/x86/entry/common.c:50 [inline]
>   do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
>   entry_SYSCALL_64_after_hwframe+0x44/0xae
> 
> Memory state around the buggy address:
>   ffff88807f06e400: fa fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>   ffff88807f06e480: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>> ffff88807f06e500: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>                                             ^
>   ffff88807f06e580: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>   ffff88807f06e600: fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc
> ==================================================================
> 
> 
> ---
> This report is generated by a bot. It may contain errors.
> See https://goo.gl/tpsmEJ for more information about syzbot.
> syzbot engineers can be reached at syzkaller@googlegroups.com.
> 
> syzbot will keep track of this issue. See:
> https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
> 

This bug was caused by Commit 393c3714081a("kernfs: switch global 
kernfs_rwsem lock to per-fs lock").

In kernfs_remove() function, it locks the @kernfs_rwsem then call 
_kernfs_remove() on @kn, but _kernfs_remove(kn) will free @root object, 
which @kernfs_rwsem belong to.

It also causes use-after-free bug with up_write() called on 
@kernfs_rwsem at the end of kernfs_remove() function.

void kernfs_remove(struct kernfs_node *kn)
{
	struct kernfs_root *root = kernfs_root(kn);
	down_write(&root->kernfs_rwsem);
  	__kernfs_remove(kn);
	up_write(&root->kernfs_rwsem);
}

Best regards,
Phi.



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-11-28 19:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-26  6:38 [syzbot] linux-next test error: WARNING: held lock freed in kernfs_destroy_root syzbot
2021-11-28 19:19 ` Phi Nguyen

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).