linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* kvm: use-after-free in irq_bypass_register_consumer
@ 2017-01-27 21:26 Dmitry Vyukov
  2017-03-04 16:24 ` Dmitry Vyukov
  0 siblings, 1 reply; 5+ messages in thread
From: Dmitry Vyukov @ 2017-01-27 21:26 UTC (permalink / raw)
  To: Paolo Bonzini, Radim Krčmář,
	KVM list, LKML, Steve Rutherford, James Mattson, Xiao Guangrong,
	haozhong.zhang
  Cc: syzkaller

Hello,

The following program triggers use-after-free in irq_bypass_register_consumer:
https://bit.googleplex.com/raw/5875575541465088/5629499534213120/

BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x48c/0x4c0
virt/lib/irqbypass.c:198 at addr ffff88006b74b438
Read of size 8 by task a.out/24723
CPU: 1 PID: 24723 Comm: a.out Not tainted 4.10.0-rc5+ #201
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
Call Trace:
 __dump_stack lib/dump_stack.c:15 [inline]
 dump_stack+0x2ee/0x3ef lib/dump_stack.c:51
 kasan_object_err+0x1c/0x70 mm/kasan/report.c:165
 print_address_description mm/kasan/report.c:203 [inline]
 kasan_report_error+0x17b/0x430 mm/kasan/report.c:287
 kasan_report mm/kasan/report.c:307 [inline]
 __asan_report_load8_noabort+0x3e/0x40 mm/kasan/report.c:328
 irq_bypass_register_consumer+0x48c/0x4c0 virt/lib/irqbypass.c:198
 kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:417 [inline]
 kvm_irqfd+0x130e/0x1e30 arch/x86/kvm/../../../virt/kvm/eventfd.c:578
 kvm_vm_ioctl+0x107a/0x1d10 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3007
 vfs_ioctl fs/ioctl.c:43 [inline]
 do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:683
 SYSC_ioctl fs/ioctl.c:698 [inline]
 SyS_ioctl+0x8f/0xc0 fs/ioctl.c:689
 entry_SYSCALL_64_fastpath+0x1f/0xc2
RIP: 0033:0x4402b9
RSP: 002b:00007f7f07514d98 EFLAGS: 00000297 ORIG_RAX: 0000000000000010
RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00000000004402b9
RDX: 0000000020ae3000 RSI: 000000004020ae76 RDI: 0000000000000007
RBP: 0000000000000000 R08: 0000000000000000 R09: 0000000000000000
R10: 0000000000000000 R11: 0000000000000297 R12: 0000000000000000
R13: 0000000000000000 R14: 00007f7f075159c0 R15: 00007f7f07515700
Object at ffff88006b74b2c0, in cache kmalloc-512 size: 512
Allocated:
PID = 24723
[<ffffffff812b1506>] save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
[<ffffffff81a0cdd3>] save_stack+0x43/0xd0 mm/kasan/kasan.c:502
[<ffffffff81a0d09a>] set_track mm/kasan/kasan.c:514 [inline]
[<ffffffff81a0d09a>] kasan_kmalloc+0xaa/0xd0 mm/kasan/kasan.c:605
[<ffffffff81a098eb>] kmem_cache_alloc_trace+0x10b/0x670 mm/slab.c:3629
[<ffffffff8108b95f>] kmalloc include/linux/slab.h:490 [inline]
[<ffffffff8108b95f>] kzalloc include/linux/slab.h:636 [inline]
[<ffffffff8108b95f>] kvm_irqfd_assign
arch/x86/kvm/../../../virt/kvm/eventfd.c:296 [inline]
[<ffffffff8108b95f>] kvm_irqfd+0x16f/0x1e30
arch/x86/kvm/../../../virt/kvm/eventfd.c:578
[<ffffffff81081b4a>] kvm_vm_ioctl+0x107a/0x1d10
arch/x86/kvm/../../../virt/kvm/kvm_main.c:3007
[<ffffffff81aa41cf>] vfs_ioctl fs/ioctl.c:43 [inline]
[<ffffffff81aa41cf>] do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:683
[<ffffffff81aa582f>] SYSC_ioctl fs/ioctl.c:698 [inline]
[<ffffffff81aa582f>] SyS_ioctl+0x8f/0xc0 fs/ioctl.c:689
[<ffffffff841c9c81>] entry_SYSCALL_64_fastpath+0x1f/0xc2
Freed:
PID = 1556
[<ffffffff812b1506>] save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
[<ffffffff81a0cdd3>] save_stack+0x43/0xd0 mm/kasan/kasan.c:502
[<ffffffff81a0d70f>] set_track mm/kasan/kasan.c:514 [inline]
[<ffffffff81a0d70f>] kasan_slab_free+0x6f/0xb0 mm/kasan/kasan.c:578
[<ffffffff81a0b5c3>] __cache_free mm/slab.c:3505 [inline]
[<ffffffff81a0b5c3>] kfree+0xd3/0x250 mm/slab.c:3822
[<ffffffff8108b4f0>] irqfd_shutdown+0x150/0x1c0
arch/x86/kvm/../../../virt/kvm/eventfd.c:148
[<ffffffff814b4920>] process_one_work+0xbd0/0x1c10 kernel/workqueue.c:2098
[<ffffffff814b5b83>] worker_thread+0x223/0x1990 kernel/workqueue.c:2232
[<ffffffff814ce546>] kthread+0x326/0x3f0 kernel/kthread.c:227
[<ffffffff841c9ef1>] ret_from_fork+0x31/0x40 arch/x86/entry/entry_64.S:430

On commit fd694aaa46c7ed811b72eb47d5eb11ce7ab3f7f1.

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

* Re: kvm: use-after-free in irq_bypass_register_consumer
  2017-01-27 21:26 kvm: use-after-free in irq_bypass_register_consumer Dmitry Vyukov
@ 2017-03-04 16:24 ` Dmitry Vyukov
  2017-03-24 13:58   ` Dmitry Vyukov
  0 siblings, 1 reply; 5+ messages in thread
From: Dmitry Vyukov @ 2017-03-04 16:24 UTC (permalink / raw)
  To: Paolo Bonzini, Radim Krčmář,
	KVM list, LKML, Steve Rutherford, James Mattson, Xiao Guangrong,
	Haozhong Zhang, Wanpeng Li
  Cc: syzkaller

On Fri, Jan 27, 2017 at 10:26 PM, Dmitry Vyukov <dvyukov@google.com> wrote:
> Hello,
>
> The following program triggers use-after-free in irq_bypass_register_consumer:
> https://bit.googleplex.com/raw/5875575541465088/5629499534213120/
>
> BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x48c/0x4c0
> virt/lib/irqbypass.c:198 at addr ffff88006b74b438
> Read of size 8 by task a.out/24723
> CPU: 1 PID: 24723 Comm: a.out Not tainted 4.10.0-rc5+ #201
> Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
> Call Trace:
>  __dump_stack lib/dump_stack.c:15 [inline]
>  dump_stack+0x2ee/0x3ef lib/dump_stack.c:51
>  kasan_object_err+0x1c/0x70 mm/kasan/report.c:165
>  print_address_description mm/kasan/report.c:203 [inline]
>  kasan_report_error+0x17b/0x430 mm/kasan/report.c:287
>  kasan_report mm/kasan/report.c:307 [inline]
>  __asan_report_load8_noabort+0x3e/0x40 mm/kasan/report.c:328
>  irq_bypass_register_consumer+0x48c/0x4c0 virt/lib/irqbypass.c:198
>  kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:417 [inline]
>  kvm_irqfd+0x130e/0x1e30 arch/x86/kvm/../../../virt/kvm/eventfd.c:578
>  kvm_vm_ioctl+0x107a/0x1d10 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3007
>  vfs_ioctl fs/ioctl.c:43 [inline]
>  do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:683
>  SYSC_ioctl fs/ioctl.c:698 [inline]
>  SyS_ioctl+0x8f/0xc0 fs/ioctl.c:689
>  entry_SYSCALL_64_fastpath+0x1f/0xc2
> RIP: 0033:0x4402b9
> RSP: 002b:00007f7f07514d98 EFLAGS: 00000297 ORIG_RAX: 0000000000000010
> RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00000000004402b9
> RDX: 0000000020ae3000 RSI: 000000004020ae76 RDI: 0000000000000007
> RBP: 0000000000000000 R08: 0000000000000000 R09: 0000000000000000
> R10: 0000000000000000 R11: 0000000000000297 R12: 0000000000000000
> R13: 0000000000000000 R14: 00007f7f075159c0 R15: 00007f7f07515700
> Object at ffff88006b74b2c0, in cache kmalloc-512 size: 512
> Allocated:
> PID = 24723
> [<ffffffff812b1506>] save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
> [<ffffffff81a0cdd3>] save_stack+0x43/0xd0 mm/kasan/kasan.c:502
> [<ffffffff81a0d09a>] set_track mm/kasan/kasan.c:514 [inline]
> [<ffffffff81a0d09a>] kasan_kmalloc+0xaa/0xd0 mm/kasan/kasan.c:605
> [<ffffffff81a098eb>] kmem_cache_alloc_trace+0x10b/0x670 mm/slab.c:3629
> [<ffffffff8108b95f>] kmalloc include/linux/slab.h:490 [inline]
> [<ffffffff8108b95f>] kzalloc include/linux/slab.h:636 [inline]
> [<ffffffff8108b95f>] kvm_irqfd_assign
> arch/x86/kvm/../../../virt/kvm/eventfd.c:296 [inline]
> [<ffffffff8108b95f>] kvm_irqfd+0x16f/0x1e30
> arch/x86/kvm/../../../virt/kvm/eventfd.c:578
> [<ffffffff81081b4a>] kvm_vm_ioctl+0x107a/0x1d10
> arch/x86/kvm/../../../virt/kvm/kvm_main.c:3007
> [<ffffffff81aa41cf>] vfs_ioctl fs/ioctl.c:43 [inline]
> [<ffffffff81aa41cf>] do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:683
> [<ffffffff81aa582f>] SYSC_ioctl fs/ioctl.c:698 [inline]
> [<ffffffff81aa582f>] SyS_ioctl+0x8f/0xc0 fs/ioctl.c:689
> [<ffffffff841c9c81>] entry_SYSCALL_64_fastpath+0x1f/0xc2
> Freed:
> PID = 1556
> [<ffffffff812b1506>] save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
> [<ffffffff81a0cdd3>] save_stack+0x43/0xd0 mm/kasan/kasan.c:502
> [<ffffffff81a0d70f>] set_track mm/kasan/kasan.c:514 [inline]
> [<ffffffff81a0d70f>] kasan_slab_free+0x6f/0xb0 mm/kasan/kasan.c:578
> [<ffffffff81a0b5c3>] __cache_free mm/slab.c:3505 [inline]
> [<ffffffff81a0b5c3>] kfree+0xd3/0x250 mm/slab.c:3822
> [<ffffffff8108b4f0>] irqfd_shutdown+0x150/0x1c0
> arch/x86/kvm/../../../virt/kvm/eventfd.c:148
> [<ffffffff814b4920>] process_one_work+0xbd0/0x1c10 kernel/workqueue.c:2098
> [<ffffffff814b5b83>] worker_thread+0x223/0x1990 kernel/workqueue.c:2232
> [<ffffffff814ce546>] kthread+0x326/0x3f0 kernel/kthread.c:227
> [<ffffffff841c9ef1>] ret_from_fork+0x31/0x40 arch/x86/entry/entry_64.S:430
>
> On commit fd694aaa46c7ed811b72eb47d5eb11ce7ab3f7f1.


Still happens on commit 0710f3ff91ecc4a715db6e4d0690472b13c4dac6:

BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x4cb/0x4e0
virt/lib/irqbypass.c:198 at addr ffff88006777ddf8
Read of size 8 by task a.out/15766
CPU: 2 PID: 15766 Comm: a.out Not tainted 4.10.0+ #285
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
Call Trace:
 __dump_stack lib/dump_stack.c:16 [inline]
 dump_stack+0x2fb/0x3fd lib/dump_stack.c:52
 kasan_object_err+0x1c/0x70 mm/kasan/report.c:166
 print_address_description mm/kasan/report.c:204 [inline]
 kasan_report_error mm/kasan/report.c:288 [inline]
 kasan_report.part.2+0x1b0/0x460 mm/kasan/report.c:310
 kasan_report mm/kasan/report.c:331 [inline]
 __asan_report_load8_noabort+0x29/0x30 mm/kasan/report.c:331
 irq_bypass_register_consumer+0x4cb/0x4e0 virt/lib/irqbypass.c:198
 kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:417 [inline]
 kvm_irqfd+0x137d/0x1e20 arch/x86/kvm/../../../virt/kvm/eventfd.c:578
 kvm_vm_ioctl+0x108c/0x1d70 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3011
 vfs_ioctl fs/ioctl.c:45 [inline]
 do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:685
 SYSC_ioctl fs/ioctl.c:700 [inline]
 SyS_ioctl+0x8f/0xc0 fs/ioctl.c:691
 entry_SYSCALL_64_fastpath+0x1f/0xc2
RIP: 0033:0x4402b9
RSP: 002b:00007fb73c4f9d98 EFLAGS: 00000297 ORIG_RAX: 0000000000000010
RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00000000004402b9
RDX: 0000000020ae3000 RSI: 000000004020ae76 RDI: 0000000000000008
RBP: 0000000000000000 R08: 0000000000000000 R09: 0000000000000000
R10: 0000000000000000 R11: 0000000000000297 R12: 0000000000000000
R13: 0000000000000000 R14: 00007fb73c4fa9c0 R15: 00007fb73c4fa700
Object at ffff88006777dc80, in cache kmalloc-512 size: 512
Allocated:
PID = 15766
 save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:59
 save_stack+0x43/0xd0 mm/kasan/kasan.c:513
 set_track mm/kasan/kasan.c:525 [inline]
 kasan_kmalloc+0xaa/0xd0 mm/kasan/kasan.c:616
 kmem_cache_alloc_trace+0x10b/0x6e0 mm/slab.c:3638
 kmalloc include/linux/slab.h:490 [inline]
 kzalloc include/linux/slab.h:663 [inline]
 kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:296 [inline]
 kvm_irqfd+0x16f/0x1e20 arch/x86/kvm/../../../virt/kvm/eventfd.c:578
 kvm_vm_ioctl+0x108c/0x1d70 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3011
 vfs_ioctl fs/ioctl.c:45 [inline]
 do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:685
 SYSC_ioctl fs/ioctl.c:700 [inline]
 SyS_ioctl+0x8f/0xc0 fs/ioctl.c:691
 entry_SYSCALL_64_fastpath+0x1f/0xc2
Freed:
PID = 670
 save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:59
 save_stack+0x43/0xd0 mm/kasan/kasan.c:513
 set_track mm/kasan/kasan.c:525 [inline]
 kasan_slab_free+0x6f/0xb0 mm/kasan/kasan.c:589
 __cache_free mm/slab.c:3514 [inline]
 kfree+0xd3/0x250 mm/slab.c:3831
 irqfd_shutdown+0x150/0x1c0 arch/x86/kvm/../../../virt/kvm/eventfd.c:148
 process_one_work+0xc06/0x1c40 kernel/workqueue.c:2096
 worker_thread+0x223/0x19f0 kernel/workqueue.c:2230
 kthread+0x334/0x400 kernel/kthread.c:229
 ret_from_fork+0x31/0x40 arch/x86/entry/entry_64.S:430
Memory state around the buggy address:
 ffff88006777dc80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
 ffff88006777dd00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>ffff88006777dd80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
                                                                ^
 ffff88006777de00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
 ffff88006777de80: fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc
==================================================================


Here is correct link for reproducer program:
https://gist.githubusercontent.com/dvyukov/34114444518fa22baff19ae204cc46a6/raw/7826cbcd1cbc472dfa4972fe56371df3c94b70c7/gistfile1.txt

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

* Re: kvm: use-after-free in irq_bypass_register_consumer
  2017-03-04 16:24 ` Dmitry Vyukov
@ 2017-03-24 13:58   ` Dmitry Vyukov
  0 siblings, 0 replies; 5+ messages in thread
From: Dmitry Vyukov @ 2017-03-24 13:58 UTC (permalink / raw)
  To: Paolo Bonzini, Radim Krčmář,
	KVM list, LKML, Steve Rutherford, James Mattson, Xiao Guangrong,
	Haozhong Zhang, Wanpeng Li, David Hildenbrand, Cornelia Huck
  Cc: syzkaller

On Sat, Mar 4, 2017 at 5:24 PM, Dmitry Vyukov <dvyukov@google.com> wrote:
> On Fri, Jan 27, 2017 at 10:26 PM, Dmitry Vyukov <dvyukov@google.com> wrote:
>> Hello,
>>
>> The following program triggers use-after-free in irq_bypass_register_consumer:
>> https://bit.googleplex.com/raw/5875575541465088/5629499534213120/
>>
>> BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x48c/0x4c0
>> virt/lib/irqbypass.c:198 at addr ffff88006b74b438
>> Read of size 8 by task a.out/24723
>> CPU: 1 PID: 24723 Comm: a.out Not tainted 4.10.0-rc5+ #201
>> Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
>> Call Trace:
>>  __dump_stack lib/dump_stack.c:15 [inline]
>>  dump_stack+0x2ee/0x3ef lib/dump_stack.c:51
>>  kasan_object_err+0x1c/0x70 mm/kasan/report.c:165
>>  print_address_description mm/kasan/report.c:203 [inline]
>>  kasan_report_error+0x17b/0x430 mm/kasan/report.c:287
>>  kasan_report mm/kasan/report.c:307 [inline]
>>  __asan_report_load8_noabort+0x3e/0x40 mm/kasan/report.c:328
>>  irq_bypass_register_consumer+0x48c/0x4c0 virt/lib/irqbypass.c:198
>>  kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:417 [inline]
>>  kvm_irqfd+0x130e/0x1e30 arch/x86/kvm/../../../virt/kvm/eventfd.c:578
>>  kvm_vm_ioctl+0x107a/0x1d10 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3007
>>  vfs_ioctl fs/ioctl.c:43 [inline]
>>  do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:683
>>  SYSC_ioctl fs/ioctl.c:698 [inline]
>>  SyS_ioctl+0x8f/0xc0 fs/ioctl.c:689
>>  entry_SYSCALL_64_fastpath+0x1f/0xc2
>> RIP: 0033:0x4402b9
>> RSP: 002b:00007f7f07514d98 EFLAGS: 00000297 ORIG_RAX: 0000000000000010
>> RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00000000004402b9
>> RDX: 0000000020ae3000 RSI: 000000004020ae76 RDI: 0000000000000007
>> RBP: 0000000000000000 R08: 0000000000000000 R09: 0000000000000000
>> R10: 0000000000000000 R11: 0000000000000297 R12: 0000000000000000
>> R13: 0000000000000000 R14: 00007f7f075159c0 R15: 00007f7f07515700
>> Object at ffff88006b74b2c0, in cache kmalloc-512 size: 512
>> Allocated:
>> PID = 24723
>> [<ffffffff812b1506>] save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
>> [<ffffffff81a0cdd3>] save_stack+0x43/0xd0 mm/kasan/kasan.c:502
>> [<ffffffff81a0d09a>] set_track mm/kasan/kasan.c:514 [inline]
>> [<ffffffff81a0d09a>] kasan_kmalloc+0xaa/0xd0 mm/kasan/kasan.c:605
>> [<ffffffff81a098eb>] kmem_cache_alloc_trace+0x10b/0x670 mm/slab.c:3629
>> [<ffffffff8108b95f>] kmalloc include/linux/slab.h:490 [inline]
>> [<ffffffff8108b95f>] kzalloc include/linux/slab.h:636 [inline]
>> [<ffffffff8108b95f>] kvm_irqfd_assign
>> arch/x86/kvm/../../../virt/kvm/eventfd.c:296 [inline]
>> [<ffffffff8108b95f>] kvm_irqfd+0x16f/0x1e30
>> arch/x86/kvm/../../../virt/kvm/eventfd.c:578
>> [<ffffffff81081b4a>] kvm_vm_ioctl+0x107a/0x1d10
>> arch/x86/kvm/../../../virt/kvm/kvm_main.c:3007
>> [<ffffffff81aa41cf>] vfs_ioctl fs/ioctl.c:43 [inline]
>> [<ffffffff81aa41cf>] do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:683
>> [<ffffffff81aa582f>] SYSC_ioctl fs/ioctl.c:698 [inline]
>> [<ffffffff81aa582f>] SyS_ioctl+0x8f/0xc0 fs/ioctl.c:689
>> [<ffffffff841c9c81>] entry_SYSCALL_64_fastpath+0x1f/0xc2
>> Freed:
>> PID = 1556
>> [<ffffffff812b1506>] save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
>> [<ffffffff81a0cdd3>] save_stack+0x43/0xd0 mm/kasan/kasan.c:502
>> [<ffffffff81a0d70f>] set_track mm/kasan/kasan.c:514 [inline]
>> [<ffffffff81a0d70f>] kasan_slab_free+0x6f/0xb0 mm/kasan/kasan.c:578
>> [<ffffffff81a0b5c3>] __cache_free mm/slab.c:3505 [inline]
>> [<ffffffff81a0b5c3>] kfree+0xd3/0x250 mm/slab.c:3822
>> [<ffffffff8108b4f0>] irqfd_shutdown+0x150/0x1c0
>> arch/x86/kvm/../../../virt/kvm/eventfd.c:148
>> [<ffffffff814b4920>] process_one_work+0xbd0/0x1c10 kernel/workqueue.c:2098
>> [<ffffffff814b5b83>] worker_thread+0x223/0x1990 kernel/workqueue.c:2232
>> [<ffffffff814ce546>] kthread+0x326/0x3f0 kernel/kthread.c:227
>> [<ffffffff841c9ef1>] ret_from_fork+0x31/0x40 arch/x86/entry/entry_64.S:430
>>
>> On commit fd694aaa46c7ed811b72eb47d5eb11ce7ab3f7f1.
>
>
> Still happens on commit 0710f3ff91ecc4a715db6e4d0690472b13c4dac6:
>
> BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x4cb/0x4e0
> virt/lib/irqbypass.c:198 at addr ffff88006777ddf8
> Read of size 8 by task a.out/15766
> CPU: 2 PID: 15766 Comm: a.out Not tainted 4.10.0+ #285
> Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
> Call Trace:
>  __dump_stack lib/dump_stack.c:16 [inline]
>  dump_stack+0x2fb/0x3fd lib/dump_stack.c:52
>  kasan_object_err+0x1c/0x70 mm/kasan/report.c:166
>  print_address_description mm/kasan/report.c:204 [inline]
>  kasan_report_error mm/kasan/report.c:288 [inline]
>  kasan_report.part.2+0x1b0/0x460 mm/kasan/report.c:310
>  kasan_report mm/kasan/report.c:331 [inline]
>  __asan_report_load8_noabort+0x29/0x30 mm/kasan/report.c:331
>  irq_bypass_register_consumer+0x4cb/0x4e0 virt/lib/irqbypass.c:198
>  kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:417 [inline]
>  kvm_irqfd+0x137d/0x1e20 arch/x86/kvm/../../../virt/kvm/eventfd.c:578
>  kvm_vm_ioctl+0x108c/0x1d70 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3011
>  vfs_ioctl fs/ioctl.c:45 [inline]
>  do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:685
>  SYSC_ioctl fs/ioctl.c:700 [inline]
>  SyS_ioctl+0x8f/0xc0 fs/ioctl.c:691
>  entry_SYSCALL_64_fastpath+0x1f/0xc2
> RIP: 0033:0x4402b9
> RSP: 002b:00007fb73c4f9d98 EFLAGS: 00000297 ORIG_RAX: 0000000000000010
> RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00000000004402b9
> RDX: 0000000020ae3000 RSI: 000000004020ae76 RDI: 0000000000000008
> RBP: 0000000000000000 R08: 0000000000000000 R09: 0000000000000000
> R10: 0000000000000000 R11: 0000000000000297 R12: 0000000000000000
> R13: 0000000000000000 R14: 00007fb73c4fa9c0 R15: 00007fb73c4fa700
> Object at ffff88006777dc80, in cache kmalloc-512 size: 512
> Allocated:
> PID = 15766
>  save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:59
>  save_stack+0x43/0xd0 mm/kasan/kasan.c:513
>  set_track mm/kasan/kasan.c:525 [inline]
>  kasan_kmalloc+0xaa/0xd0 mm/kasan/kasan.c:616
>  kmem_cache_alloc_trace+0x10b/0x6e0 mm/slab.c:3638
>  kmalloc include/linux/slab.h:490 [inline]
>  kzalloc include/linux/slab.h:663 [inline]
>  kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:296 [inline]
>  kvm_irqfd+0x16f/0x1e20 arch/x86/kvm/../../../virt/kvm/eventfd.c:578
>  kvm_vm_ioctl+0x108c/0x1d70 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3011
>  vfs_ioctl fs/ioctl.c:45 [inline]
>  do_vfs_ioctl+0x1bf/0x1790 fs/ioctl.c:685
>  SYSC_ioctl fs/ioctl.c:700 [inline]
>  SyS_ioctl+0x8f/0xc0 fs/ioctl.c:691
>  entry_SYSCALL_64_fastpath+0x1f/0xc2
> Freed:
> PID = 670
>  save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:59
>  save_stack+0x43/0xd0 mm/kasan/kasan.c:513
>  set_track mm/kasan/kasan.c:525 [inline]
>  kasan_slab_free+0x6f/0xb0 mm/kasan/kasan.c:589
>  __cache_free mm/slab.c:3514 [inline]
>  kfree+0xd3/0x250 mm/slab.c:3831
>  irqfd_shutdown+0x150/0x1c0 arch/x86/kvm/../../../virt/kvm/eventfd.c:148
>  process_one_work+0xc06/0x1c40 kernel/workqueue.c:2096
>  worker_thread+0x223/0x19f0 kernel/workqueue.c:2230
>  kthread+0x334/0x400 kernel/kthread.c:229
>  ret_from_fork+0x31/0x40 arch/x86/entry/entry_64.S:430
> Memory state around the buggy address:
>  ffff88006777dc80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>  ffff88006777dd00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>>ffff88006777dd80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>                                                                 ^
>  ffff88006777de00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>  ffff88006777de80: fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc
> ==================================================================
>
>
> Here is correct link for reproducer program:
> https://gist.githubusercontent.com/dvyukov/34114444518fa22baff19ae204cc46a6/raw/7826cbcd1cbc472dfa4972fe56371df3c94b70c7/gistfile1.txt


Still happens regularly.

On 093b995e3b55a0ae0670226ddfcb05bfbf0099ae (Mar 20):

==================================================================
BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x4f0/0x500
virt/lib/irqbypass.c:198 at addr ffff88006648ec38
Read of size 8 by task a.out/6857
CPU: 3 PID: 6857 Comm: a.out Not tainted 4.11.0-rc3+ #366
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
Call Trace:
 __dump_stack lib/dump_stack.c:16 [inline]
 dump_stack+0x1b8/0x28d lib/dump_stack.c:52
 kasan_object_err+0x1c/0x70 mm/kasan/report.c:166
 print_address_description mm/kasan/report.c:210 [inline]
 kasan_report_error mm/kasan/report.c:294 [inline]
 kasan_report.part.2+0x1be/0x480 mm/kasan/report.c:316
 kasan_report mm/kasan/report.c:337 [inline]
 __asan_report_load8_noabort+0x29/0x30 mm/kasan/report.c:337
 irq_bypass_register_consumer+0x4f0/0x500 virt/lib/irqbypass.c:198
 kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:417 [inline]
 kvm_irqfd+0x1391/0x1d70 arch/x86/kvm/../../../virt/kvm/eventfd.c:572
 kvm_vm_ioctl+0xe9c/0x1c60 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3012
 vfs_ioctl fs/ioctl.c:45 [inline]
 do_vfs_ioctl+0x1af/0x16d0 fs/ioctl.c:685
 SYSC_ioctl fs/ioctl.c:700 [inline]
 SyS_ioctl+0x8f/0xc0 fs/ioctl.c:691
 entry_SYSCALL_64_fastpath+0x1f/0xc2
RIP: 0033:0x4402b9
RSP: 002b:00007f0564cc2d98 EFLAGS: 00000297 ORIG_RAX: 0000000000000010
RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00000000004402b9
RDX: 0000000020ae3000 RSI: 000000004020ae76 RDI: 0000000000000007
RBP: 0000000000000086 R08: 0000000000000000 R09: 0000000000000000
R10: 0000000000000000 R11: 0000000000000297 R12: 0000000000000000
R13: 0000000000000000 R14: 00007f0564cc39c0 R15: 00007f0564cc3700
Object at ffff88006648eac0, in cache kmalloc-512 size: 512
Allocated:
PID = 6857
 save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:59
 save_stack+0x43/0xd0 mm/kasan/kasan.c:517
 set_track mm/kasan/kasan.c:529 [inline]
 kasan_kmalloc+0xbc/0xf0 mm/kasan/kasan.c:620
 kmem_cache_alloc_trace+0x11a/0x720 mm/slab.c:3638
 kmalloc include/linux/slab.h:490 [inline]
 kzalloc include/linux/slab.h:663 [inline]
 kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:296 [inline]
 kvm_irqfd+0x16c/0x1d70 arch/x86/kvm/../../../virt/kvm/eventfd.c:572
 kvm_vm_ioctl+0xe9c/0x1c60 arch/x86/kvm/../../../virt/kvm/kvm_main.c:3012
 vfs_ioctl fs/ioctl.c:45 [inline]
 do_vfs_ioctl+0x1af/0x16d0 fs/ioctl.c:685
 SYSC_ioctl fs/ioctl.c:700 [inline]
 SyS_ioctl+0x8f/0xc0 fs/ioctl.c:691
 entry_SYSCALL_64_fastpath+0x1f/0xc2
Freed:
PID = 1252
 save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:59
 save_stack+0x43/0xd0 mm/kasan/kasan.c:517
 set_track mm/kasan/kasan.c:529 [inline]
 kasan_slab_free+0x81/0xc0 mm/kasan/kasan.c:593
 __cache_free mm/slab.c:3514 [inline]
 kfree+0xd7/0x250 mm/slab.c:3831
 irqfd_shutdown+0x13c/0x1a0 arch/x86/kvm/../../../virt/kvm/eventfd.c:148
 process_one_work+0xb20/0x1b40 kernel/workqueue.c:2097
 worker_thread+0x1b4/0x1340 kernel/workqueue.c:2231
 kthread+0x359/0x420 kernel/kthread.c:229
 ret_from_fork+0x31/0x40 arch/x86/entry/entry_64.S:430
Memory state around the buggy address:
 ffff88006648eb00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
 ffff88006648eb80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>ffff88006648ec00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
                                        ^
 ffff88006648ec80: fb fb fb fb fb fb fb fb fc fc fc fc fc fc fc fc
 ffff88006648ed00: fc fc fc fc fc fc fc fc fb fb fb fb fb fb fb fb
==================================================================

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

* kvm: use-after-free in irq_bypass_register_consumer
@ 2017-09-06  3:39 idaifish
  0 siblings, 0 replies; 5+ messages in thread
From: idaifish @ 2017-09-06  3:39 UTC (permalink / raw)
  To: pbonzini, rkrcmar; +Cc: LKML, syzkaller, Dmitry Vyukov

Got the following report while fuzzing 4.9.47.

It seems that this bug has been reported by Dmitry Vyukov
[https://lkml.org/lkml/2017/1/27/828]
But I still can reproduce the bug  on latest Ubuntu1604 (4.4.0-94-generic)

PoC:  https://gist.githubusercontent.com/dvyukov/34114444518fa22baff19ae204cc46a6/raw/7826cbcd1cbc472dfa4972fe56371df3c94b70c7/gistfile1.txt


=======================================================================================

BUG: KASAN: use-after-free in __list_add include/linux/list.h:43
[inline] at addr ffff88007ab9dc80
BUG: KASAN: use-after-free in list_add include/linux/list.h:63
[inline] at addr ffff88007ab9dc80
BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x3a3/0x420
virt/lib/irqbypass.c:217 at addr ffff88007ab9dc80
Write of size 8 by task syz-executor4/2970
CPU: 0 PID: 2970 Comm: syz-executor4 Tainted: G    B           4.9.47 #1
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS
Ubuntu-1.8.2-1ubuntu1 04/01/2014
 ffff8800760df9c0 ffffffff81ad97d9 ffff88007f803080 ffff88007ab9db80
 ffff88007ab9dd80 ffff88007ab9da00 ffff8800760df9e8 ffffffff8153892c
 ffff8800760dfa78 ffff88007f803080 ffff8800776f7d20 ffff8800760dfa68
Call Trace:
 [<ffffffff81ad97d9>] __dump_stack lib/dump_stack.c:15 [inline]
 [<ffffffff81ad97d9>] dump_stack+0x83/0xba lib/dump_stack.c:51
 [<ffffffff8153892c>] kasan_object_err+0x1c/0x70 mm/kasan/report.c:160
 [<ffffffff81538bc0>] print_address_description mm/kasan/report.c:198 [inline]
 [<ffffffff81538bc0>] kasan_report_error+0x1f0/0x4e0 mm/kasan/report.c:287
 [<ffffffff815390ee>] kasan_report mm/kasan/report.c:309 [inline]
 [<ffffffff815390ee>] __asan_report_store8_noabort+0x3e/0x40
mm/kasan/report.c:335
 [<ffffffff82e70a43>] __list_add include/linux/list.h:43 [inline]
 [<ffffffff82e70a43>] list_add include/linux/list.h:63 [inline]
 [<ffffffff82e70a43>] irq_bypass_register_consumer+0x3a3/0x420
virt/lib/irqbypass.c:217
 [<ffffffff81063d25>] kvm_irqfd_assign
arch/x86/kvm/../../../virt/kvm/eventfd.c:417 [inline]
 [<ffffffff81063d25>] kvm_irqfd+0x1095/0x1840
arch/x86/kvm/../../../virt/kvm/eventfd.c:572
 [<ffffffff8105b19c>] kvm_vm_ioctl+0x2bc/0x1580
arch/x86/kvm/../../../virt/kvm/kvm_main.c:2999
 [<ffffffff815a1f2c>] vfs_ioctl fs/ioctl.c:43 [inline]
 [<ffffffff815a1f2c>] do_vfs_ioctl+0x18c/0xf80 fs/ioctl.c:679
 [<ffffffff815a2daf>] SYSC_ioctl fs/ioctl.c:694 [inline]
 [<ffffffff815a2daf>] SyS_ioctl+0x8f/0xc0 fs/ioctl.c:685
 [<ffffffff82e85577>] entry_SYSCALL_64_fastpath+0x1a/0xa9
Object at ffff88007ab9db80, in cache kmalloc-512 size: 512
Allocated:
PID = 2970
 save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
 save_stack+0x46/0xd0 mm/kasan/kasan.c:495
 set_track mm/kasan/kasan.c:507 [inline]
 kasan_kmalloc+0xad/0xe0 mm/kasan/kasan.c:598
 kmem_cache_alloc_trace include/linux/slab.h:391 [inline]
 kmalloc include/linux/slab.h:490 [inline]
 kzalloc include/linux/slab.h:636 [inline]
 kvm_irqfd_assign arch/x86/kvm/../../../virt/kvm/eventfd.c:296 [inline]
 kvm_irqfd+0xbd/0x1840 arch/x86/kvm/../../../virt/kvm/eventfd.c:572
 kvm_vm_ioctl+0x2bc/0x1580 arch/x86/kvm/../../../virt/kvm/kvm_main.c:2999
 vfs_ioctl fs/ioctl.c:43 [inline]
 do_vfs_ioctl+0x18c/0xf80 fs/ioctl.c:679
 SYSC_ioctl fs/ioctl.c:694 [inline]
 SyS_ioctl+0x8f/0xc0 fs/ioctl.c:685
 entry_SYSCALL_64_fastpath+0x1a/0xa9
Freed:
PID = 1098
 save_stack_trace+0x16/0x20 arch/x86/kernel/stacktrace.c:57
 save_stack+0x46/0xd0 mm/kasan/kasan.c:495
 set_track mm/kasan/kasan.c:507 [inline]
 kasan_slab_free+0x73/0xc0 mm/kasan/kasan.c:571
 slab_free_hook mm/slub.c:1355 [inline]
 slab_free_freelist_hook mm/slub.c:1377 [inline]
 slab_free mm/slub.c:2958 [inline]
 kfree+0xa0/0x150 mm/slub.c:3878
 irqfd_shutdown+0x137/0x1a0 arch/x86/kvm/../../../virt/kvm/eventfd.c:148
 process_one_work+0x87c/0x1170 kernel/workqueue.c:2096
 worker_thread+0xed/0x14e0 kernel/workqueue.c:2230
 kthread+0x220/0x2a0 kernel/kthread.c:211
 ret_from_fork+0x25/0x30 arch/x86/entry/entry_64.S:433
Memory state around the buggy address:
 ffff88007ab9db80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
 ffff88007ab9dc00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>ffff88007ab9dc80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
                   ^
 ffff88007ab9dd00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
 ffff88007ab9dd80: fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc




-- 
Regards,
idaifish

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

* kvm: use-after-free in irq_bypass_register_consumer
@ 2016-11-12  5:12 Dmitry Vyukov
  0 siblings, 0 replies; 5+ messages in thread
From: Dmitry Vyukov @ 2016-11-12  5:12 UTC (permalink / raw)
  To: Alex Williamson, Paolo Bonzini, KVM list, LKML, Steve Rutherford
  Cc: syzkaller

Hello,

While running syzkaller fuzzer I got the following use-after-free report.
On commit 015ed9433be2b476ec7e2e6a9a411a56e3b5b035 (Nov 11)


BUG: KASAN: use-after-free in irq_bypass_register_consumer+0x3d1/0x420
at addr ffff88003b5d8820
Write of size 8 by task syz-executor/25573
CPU: 0 PID: 25573 Comm: syz-executor Not tainted 4.9.0-rc4+ #46
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
 ffff88006caaf9e0 ffffffff81c2d79b ffff88003e80ccc0 ffff88003b5d86b8
 ffff88003b5d89f0 0000000000000001 ffff88006caafa08 ffffffff8165ab9c
 ffffed00076bb104 ffffed00076bb104 ffff88003e80ccc0 ffff88006caafa88
Call Trace:
 [<ffffffff8165b2b7>] __asan_report_store8_noabort+0x17/0x20
mm/kasan/report.c:334
 [<     inline     >] list_add include/linux/list.h:43
 [<ffffffff831d16d1>] irq_bypass_register_consumer+0x3d1/0x420
virt/lib/irqbypass.c:217
 [<     inline     >] kvm_irqfd_assign
arch/x86/kvm/../../../virt/kvm/eventfd.c:417
 [<ffffffff8106e3ea>] kvm_irqfd+0x109a/0x18a0
arch/x86/kvm/../../../virt/kvm/eventfd.c:572
 [<ffffffff81065797>] kvm_vm_ioctl+0x2e7/0x1670
arch/x86/kvm/../../../virt/kvm/kvm_main.c:2996
 [<     inline     >] vfs_ioctl fs/ioctl.c:43
 [<ffffffff816af6fc>] do_vfs_ioctl+0x18c/0x1040 fs/ioctl.c:679
 [<     inline     >] SYSC_ioctl fs/ioctl.c:694
 [<ffffffff816b063f>] SyS_ioctl+0x8f/0xc0 fs/ioctl.c:685
 [<ffffffff831e9dc1>] entry_SYSCALL_64_fastpath+0x1f/0xc2

Object at ffff88003b5d86b8, in cache kmalloc-512 size: 512

Allocated:
PID = 25573
 [  359.255946] [<     inline     >] kzalloc include/linux/slab.h:636
 [  359.255946] [<     inline     >] kvm_irqfd_assign
arch/x86/kvm/../../../virt/kvm/eventfd.c:296
 [  359.255946] [<ffffffff8106d3f7>] kvm_irqfd+0xa7/0x18a0
arch/x86/kvm/../../../virt/kvm/eventfd.c:572
 [  359.255946] [<ffffffff81065797>] kvm_vm_ioctl+0x2e7/0x1670
arch/x86/kvm/../../../virt/kvm/kvm_main.c:2996
 [  359.255946] [<     inline     >] vfs_ioctl fs/ioctl.c:43
 [  359.255946] [<ffffffff816af6fc>] do_vfs_ioctl+0x18c/0x1040 fs/ioctl.c:679
 [  359.255946] [<     inline     >] SYSC_ioctl fs/ioctl.c:694
 [  359.255946] [<ffffffff816b063f>] SyS_ioctl+0x8f/0xc0 fs/ioctl.c:685
 [  359.255946] [<ffffffff831e9dc1>] entry_SYSCALL_64_fastpath+0x1f/0xc2

Freed:
PID = 1057
 [  359.255946] [<ffffffff8165696a>] kfree+0xea/0x2c0 mm/slub.c:3871
 [  359.255946] [<ffffffff8106d16d>] irqfd_shutdown+0x13d/0x1a0
arch/x86/kvm/../../../virt/kvm/eventfd.c:148
 [  359.255946] [<ffffffff8129175c>] process_one_work+0x9fc/0x1900
kernel/workqueue.c:2096
 [  359.255946] [<ffffffff8129274f>] worker_thread+0xef/0x1480
kernel/workqueue.c:2230
 [  359.386293] [<ffffffff812a5a94>] kthread+0x244/0x2d0 kernel/kthread.c:209
 [  359.387074] [<ffffffff831ea02a>] ret_from_fork+0x2a/0x40
arch/x86/entry/entry_64.S:433

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

end of thread, other threads:[~2017-09-06  3:40 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-27 21:26 kvm: use-after-free in irq_bypass_register_consumer Dmitry Vyukov
2017-03-04 16:24 ` Dmitry Vyukov
2017-03-24 13:58   ` Dmitry Vyukov
  -- strict thread matches above, loose matches on Subject: below --
2017-09-06  3:39 idaifish
2016-11-12  5:12 Dmitry Vyukov

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