Re: [syzbot] WARNING in io_link_timeout_fn

[Pavel Begunkov]

On 5/14/21 1:08 AM, syzbot wrote:
> Hello,
> 
> syzbot has tested the proposed patch but the reproducer is still triggering an issue:
> KASAN: use-after-free Read in hrtimer_active

#syz test: https://github.com/isilence/linux.git syz_test8

> ==================================================================
> BUG: KASAN: use-after-free in hrtimer_active+0x1d6/0x1f0 kernel/time/hrtimer.c:1462
> Read of size 8 at addr ffff8880129a64b8 by task syz-executor.0/9928
> 
> CPU: 0 PID: 9928 Comm: syz-executor.0 Not tainted 5.12.0-syzkaller #0
> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
> Call Trace:
>  __dump_stack lib/dump_stack.c:79 [inline]
>  dump_stack+0x141/0x1d7 lib/dump_stack.c:120
>  print_address_description.constprop.0.cold+0x5b/0x2f8 mm/kasan/report.c:233
>  __kasan_report mm/kasan/report.c:419 [inline]
>  kasan_report.cold+0x7c/0xd8 mm/kasan/report.c:436
>  hrtimer_active+0x1d6/0x1f0 kernel/time/hrtimer.c:1462
>  hrtimer_try_to_cancel+0x21/0x1e0 kernel/time/hrtimer.c:1180
>  io_kill_linked_timeout fs/io_uring.c:1794 [inline]
>  io_disarm_next+0x196/0xad0 fs/io_uring.c:1827
>  __io_req_find_next+0xca/0x160 fs/io_uring.c:1852
>  io_req_find_next fs/io_uring.c:1868 [inline]
>  io_queue_next fs/io_uring.c:2070 [inline]
>  io_free_req fs/io_uring.c:2078 [inline]
>  io_put_req_deferred_cb+0x253/0x4a0 fs/io_uring.c:2180
>  __tctx_task_work fs/io_uring.c:1909 [inline]
>  tctx_task_work+0x24e/0x550 fs/io_uring.c:1923
>  task_work_run+0xdd/0x1a0 kernel/task_work.c:161
>  tracehook_notify_signal include/linux/tracehook.h:212 [inline]
>  handle_signal_work kernel/entry/common.c:145 [inline]
>  exit_to_user_mode_loop kernel/entry/common.c:171 [inline]
>  exit_to_user_mode_prepare+0x24a/0x280 kernel/entry/common.c:208
>  __syscall_exit_to_user_mode_work kernel/entry/common.c:290 [inline]
>  syscall_exit_to_user_mode+0x19/0x60 kernel/entry/common.c:301
>  do_syscall_64+0x47/0xb0 arch/x86/entry/common.c:57
>  entry_SYSCALL_64_after_hwframe+0x44/0xae
> RIP: 0033:0x4665f9
> Code: ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 40 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 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:00007f9092a97188 EFLAGS: 00000246 ORIG_RAX: 00000000000001aa
> RAX: 0000000000000100 RBX: 000000000056bf60 RCX: 00000000004665f9
> RDX: 0000000000000000 RSI: 000000000000450c RDI: 0000000000000003
> RBP: 00000000004bfce1 R08: 0000000000000000 R09: 0000000000000000
> R10: 0000000000000000 R11: 0000000000000246 R12: 000000000056bf60
> R13: 0000000000a9fb1f R14: 00007f9092a97300 R15: 0000000000022000
> 
> Allocated by task 9928:
>  kasan_save_stack+0x1b/0x40 mm/kasan/common.c:38
>  kasan_set_track mm/kasan/common.c:46 [inline]
>  set_alloc_info mm/kasan/common.c:427 [inline]
>  ____kasan_kmalloc mm/kasan/common.c:506 [inline]
>  ____kasan_kmalloc mm/kasan/common.c:465 [inline]
>  __kasan_kmalloc+0x99/0xc0 mm/kasan/common.c:515
>  kmalloc include/linux/slab.h:561 [inline]
>  io_alloc_async_data fs/io_uring.c:3116 [inline]
>  io_timeout_prep+0x3d9/0x500 fs/io_uring.c:5637
>  io_req_prep fs/io_uring.c:5908 [inline]
>  io_submit_sqe fs/io_uring.c:6576 [inline]
>  io_submit_sqes+0x4e4c/0x6c50 fs/io_uring.c:6734
>  __do_sys_io_uring_enter+0xeaf/0x1d50 fs/io_uring.c:9319
>  do_syscall_64+0x3a/0xb0 arch/x86/entry/common.c:47
>  entry_SYSCALL_64_after_hwframe+0x44/0xae
> 
> Freed by task 4826:
>  kasan_save_stack+0x1b/0x40 mm/kasan/common.c:38
>  kasan_set_track+0x1c/0x30 mm/kasan/common.c:46
>  kasan_set_free_info+0x20/0x30 mm/kasan/generic.c:357
>  ____kasan_slab_free mm/kasan/common.c:360 [inline]
>  ____kasan_slab_free mm/kasan/common.c:325 [inline]
>  __kasan_slab_free+0xf5/0x130 mm/kasan/common.c:367
>  kasan_slab_free include/linux/kasan.h:199 [inline]
>  slab_free_hook mm/slub.c:1563 [inline]
>  slab_free_freelist_hook+0x92/0x210 mm/slub.c:1601
>  slab_free mm/slub.c:3162 [inline]
>  kfree+0xe5/0x7f0 mm/slub.c:4216
>  io_dismantle_req+0x116/0x250 fs/io_uring.c:1743
>  io_req_complete_post+0x1d7/0x890 fs/io_uring.c:1600
>  io_link_timeout_fn+0x5f7/0xb10 fs/io_uring.c:6369
>  __run_hrtimer kernel/time/hrtimer.c:1537 [inline]
>  __hrtimer_run_queues+0x609/0xe40 kernel/time/hrtimer.c:1601
>  hrtimer_interrupt+0x330/0xa00 kernel/time/hrtimer.c:1663
>  local_apic_timer_interrupt arch/x86/kernel/apic/apic.c:1089 [inline]
>  __sysvec_apic_timer_interrupt+0x146/0x540 arch/x86/kernel/apic/apic.c:1106
>  sysvec_apic_timer_interrupt+0x8e/0xc0 arch/x86/kernel/apic/apic.c:1100
>  asm_sysvec_apic_timer_interrupt+0x12/0x20 arch/x86/include/asm/idtentry.h:632
> 
> The buggy address belongs to the object at ffff8880129a6480
>  which belongs to the cache kmalloc-96 of size 96
> The buggy address is located 56 bytes inside of
>  96-byte region [ffff8880129a6480, ffff8880129a64e0)
> The buggy address belongs to the page:
> page:ffffea00004a6980 refcount:1 mapcount:0 mapping:0000000000000000 index:0x0 pfn:0x129a6
> flags: 0xfff00000000200(slab|node=0|zone=1|lastcpupid=0x7ff)
> raw: 00fff00000000200 dead000000000100 dead000000000122 ffff888010841780
> raw: 0000000000000000 0000000000200020 00000001ffffffff 0000000000000000
> page dumped because: kasan: bad access detected
> page_owner tracks the page as allocated
> page last allocated via order 0, migratetype Unmovable, gfp_mask 0x12cc0(GFP_KERNEL|__GFP_NOWARN|__GFP_NORETRY), pid 9928, ts 107924339577
>  set_page_owner include/linux/page_owner.h:31 [inline]
>  post_alloc_hook+0x161/0x1c0 mm/page_alloc.c:2302
>  prep_new_page mm/page_alloc.c:2311 [inline]
>  get_page_from_freelist+0x1c6f/0x3fb0 mm/page_alloc.c:3951
>  __alloc_pages_nodemask+0x2d6/0x730 mm/page_alloc.c:5001
>  alloc_pages_current+0x18c/0x2a0 mm/mempolicy.c:2277
>  alloc_pages include/linux/gfp.h:561 [inline]
>  alloc_slab_page mm/slub.c:1639 [inline]
>  allocate_slab+0x2c5/0x4c0 mm/slub.c:1779
>  new_slab mm/slub.c:1842 [inline]
>  new_slab_objects mm/slub.c:2588 [inline]
>  ___slab_alloc+0x44c/0x7a0 mm/slub.c:2751
>  __slab_alloc.constprop.0+0xa7/0xf0 mm/slub.c:2791
>  slab_alloc_node mm/slub.c:2872 [inline]
>  slab_alloc mm/slub.c:2916 [inline]
>  __kmalloc+0x2e5/0x300 mm/slub.c:4054
>  kmalloc include/linux/slab.h:561 [inline]
>  io_alloc_async_data fs/io_uring.c:3116 [inline]
>  io_timeout_prep+0x3d9/0x500 fs/io_uring.c:5637
>  io_req_prep fs/io_uring.c:5908 [inline]
>  io_submit_sqe fs/io_uring.c:6576 [inline]
>  io_submit_sqes+0x4e4c/0x6c50 fs/io_uring.c:6734
>  __do_sys_io_uring_enter+0xeaf/0x1d50 fs/io_uring.c:9319
>  do_syscall_64+0x3a/0xb0 arch/x86/entry/common.c:47
>  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:1271 [inline]
>  free_pcp_prepare+0x2cb/0x410 mm/page_alloc.c:1310
>  free_unref_page_prepare mm/page_alloc.c:3205 [inline]
>  free_unref_page+0x12/0x1d0 mm/page_alloc.c:3253
>  qlink_free mm/kasan/quarantine.c:146 [inline]
>  qlist_free_all+0x5a/0xc0 mm/kasan/quarantine.c:165
>  kasan_quarantine_reduce+0x180/0x200 mm/kasan/quarantine.c:272
>  __kasan_slab_alloc+0x7f/0x90 mm/kasan/common.c:437
>  kasan_slab_alloc include/linux/kasan.h:223 [inline]
>  slab_post_alloc_hook mm/slab.h:516 [inline]
>  slab_alloc_node mm/slub.c:2908 [inline]
>  slab_alloc mm/slub.c:2916 [inline]
>  kmem_cache_alloc+0x153/0x370 mm/slub.c:2921
>  getname_flags.part.0+0x50/0x4f0 fs/namei.c:138
>  getname_flags include/linux/audit.h:319 [inline]
>  getname+0x8e/0xd0 fs/namei.c:209
>  do_sys_openat2+0xf5/0x420 fs/open.c:1181
>  do_sys_open fs/open.c:1203 [inline]
>  __do_sys_open fs/open.c:1211 [inline]
>  __se_sys_open fs/open.c:1207 [inline]
>  __x64_sys_open+0x119/0x1c0 fs/open.c:1207
>  do_syscall_64+0x3a/0xb0 arch/x86/entry/common.c:47
>  entry_SYSCALL_64_after_hwframe+0x44/0xae
> 
> Memory state around the buggy address:
>  ffff8880129a6380: fa fb fb fb fb fb fb fb fb fb fb fb fc fc fc fc
>  ffff8880129a6400: 00 00 00 00 00 00 00 00 00 00 00 00 fc fc fc fc
>> ffff8880129a6480: fa fb fb fb fb fb fb fb fb fb fb fb fc fc fc fc
>                                         ^
>  ffff8880129a6500: fa fb fb fb fb fb fb fb fb fb fb fb fc fc fc fc
>  ffff8880129a6580: fa fb fb fb fb fb fb fb fb fb fb fb fc fc fc fc
> ==================================================================
> 
> 
> Tested on:
> 
> commit:         a519b86e io_uring: syz debug output
> git tree:       https://github.com/isilence/linux.git syz_test6
> console output: https://syzkaller.appspot.com/x/log.txt?x=14127779d00000
> kernel config:  https://syzkaller.appspot.com/x/.config?x=ae2e6c63d6410fd3
> dashboard link: https://syzkaller.appspot.com/bug?extid=5a864149dd970b546223
> compiler:       
> 

-- 
Pavel Begunkov