Hello,
syzbot found the following issue on:
HEAD commit: 1c7873e33645 mm: lock newly mapped VMA with corrected orde..
git tree: upstream
console output: https://syzkaller.appspot.com/x/log.txt?x=1517663ca80000
kernel config: https://syzkaller.appspot.com/x/.config?x=832b404e095b70c0
dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
compiler: gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2
syz repro: https://syzkaller.appspot.com/x/repro.syz?x=1776519ca80000
C reproducer: https://syzkaller.appspot.com/x/repro.c?x=117f9778a80000
Downloadable assets:
disk image: https://storage.googleapis.com/syzbot-assets/43b42bd2cf70/disk-1c7873e3.raw.xz
vmlinux: https://storage.googleapis.com/syzbot-assets/011ba1b9e8c1/vmlinux-1c7873e3.xz
kernel image: https://storage.googleapis.com/syzbot-assets/18b349342595/bzImage-1c7873e3.xz
IMPORTANT: if you fix the issue, please add the following tag to the commit:
Reported-by: [email protected]
BUG: memory leak
unreferenced object 0xffff88810b172000 (size 4096):
comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
hex dump (first 32 bytes):
01 00 00 00 00 00 00 00 00 bc 96 0e 81 88 ff ff ................
5d b6 9e 85 ff ff ff ff 03 00 00 00 00 00 00 00 ]...............
backtrace:
[<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
[<ffffffff8347eb55>] kmalloc include/linux/slab.h:582 [inline]
[<ffffffff8347eb55>] kzalloc include/linux/slab.h:703 [inline]
[<ffffffff8347eb55>] dev_new drivers/usb/gadget/legacy/raw_gadget.c:191 [inline]
[<ffffffff8347eb55>] raw_open+0x45/0x110 drivers/usb/gadget/legacy/raw_gadget.c:385
[<ffffffff827d1d09>] misc_open+0x1a9/0x1f0 drivers/char/misc.c:165
[<ffffffff8166763b>] chrdev_open+0x11b/0x340 fs/char_dev.c:414
[<ffffffff8165573f>] do_dentry_open+0x30f/0x990 fs/open.c:914
[<ffffffff8167cabb>] do_open fs/namei.c:3636 [inline]
[<ffffffff8167cabb>] path_openat+0x178b/0x1b20 fs/namei.c:3793
[<ffffffff8167e995>] do_filp_open+0xc5/0x1b0 fs/namei.c:3820
[<ffffffff81659453>] do_sys_openat2+0xe3/0x140 fs/open.c:1407
[<ffffffff81659de3>] do_sys_open fs/open.c:1422 [inline]
[<ffffffff81659de3>] __do_sys_openat fs/open.c:1438 [inline]
[<ffffffff81659de3>] __se_sys_openat fs/open.c:1433 [inline]
[<ffffffff81659de3>] __x64_sys_openat+0x83/0xe0 fs/open.c:1433
[<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
[<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
[<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
BUG: memory leak
unreferenced object 0xffff88810ad20d60 (size 32):
comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
hex dump (first 32 bytes):
72 61 77 2d 67 61 64 67 65 74 2e 30 00 00 00 00 raw-gadget.0....
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
backtrace:
[<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
[<ffffffff8347cd2f>] kmalloc include/linux/slab.h:582 [inline]
[<ffffffff8347cd2f>] raw_ioctl_init+0xdf/0x410 drivers/usb/gadget/legacy/raw_gadget.c:460
[<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
[<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
[<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
[<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
[<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
[<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
[<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
[<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
BUG: memory leak
unreferenced object 0xffff88810e96bc00 (size 128):
comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
hex dump (first 32 bytes):
64 75 6d 6d 79 5f 75 64 63 00 00 00 00 00 00 00 dummy_udc.......
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
backtrace:
[<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
[<ffffffff8347cd6d>] kmalloc include/linux/slab.h:582 [inline]
[<ffffffff8347cd6d>] raw_ioctl_init+0x11d/0x410 drivers/usb/gadget/legacy/raw_gadget.c:468
[<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
[<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
[<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
[<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
[<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
[<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
[<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
[<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
BUG: memory leak
unreferenced object 0xffff88810e96ba80 (size 128):
comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
hex dump (first 32 bytes):
64 75 6d 6d 79 5f 75 64 63 2e 30 00 00 00 00 00 dummy_udc.0.....
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
backtrace:
[<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
[<ffffffff8347cdfc>] kmalloc include/linux/slab.h:582 [inline]
[<ffffffff8347cdfc>] raw_ioctl_init+0x1ac/0x410 drivers/usb/gadget/legacy/raw_gadget.c:479
[<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
[<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
[<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
[<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
[<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
[<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
[<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
[<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
BUG: memory leak
unreferenced object 0xffff88810e96bd00 (size 128):
comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
hex dump (first 32 bytes):
00 bd 96 0e 81 88 ff ff 00 bd 96 0e 81 88 ff ff ................
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
backtrace:
[<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
[<ffffffff833ecc6a>] kmalloc include/linux/slab.h:582 [inline]
[<ffffffff833ecc6a>] kzalloc include/linux/slab.h:703 [inline]
[<ffffffff833ecc6a>] dummy_alloc_request+0x5a/0xe0 drivers/usb/gadget/udc/dummy_hcd.c:665
[<ffffffff833e9132>] usb_ep_alloc_request+0x22/0xd0 drivers/usb/gadget/udc/core.c:196
[<ffffffff8347f13d>] gadget_bind+0x6d/0x370 drivers/usb/gadget/legacy/raw_gadget.c:292
[<ffffffff833e9e83>] gadget_bind_driver+0xe3/0x2e0 drivers/usb/gadget/udc/core.c:1591
[<ffffffff82ba069d>] call_driver_probe drivers/base/dd.c:579 [inline]
[<ffffffff82ba069d>] really_probe+0x12d/0x430 drivers/base/dd.c:658
[<ffffffff82ba0a61>] __driver_probe_device+0xc1/0x1a0 drivers/base/dd.c:798
[<ffffffff82ba0b6a>] driver_probe_device+0x2a/0x120 drivers/base/dd.c:828
[<ffffffff82ba0eae>] __driver_attach drivers/base/dd.c:1214 [inline]
[<ffffffff82ba0eae>] __driver_attach+0xfe/0x1f0 drivers/base/dd.c:1154
[<ffffffff82b9d985>] bus_for_each_dev+0xa5/0x110 drivers/base/bus.c:368
[<ffffffff82b9f486>] bus_add_driver+0x126/0x2a0 drivers/base/bus.c:673
[<ffffffff82ba25da>] driver_register+0x8a/0x190 drivers/base/driver.c:246
[<ffffffff833e7524>] usb_gadget_register_driver_owner+0x64/0x160 drivers/usb/gadget/udc/core.c:1665
[<ffffffff8347e0e6>] raw_ioctl_run drivers/usb/gadget/legacy/raw_gadget.c:546 [inline]
[<ffffffff8347e0e6>] raw_ioctl+0x6f6/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1253
[<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
[<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
[<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
[<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
[<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
[<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
---
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 [email protected].
syzbot will keep track of this issue. See:
https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
If the bug is already fixed, let syzbot know by replying with:
#syz fix: exact-commit-title
If you want syzbot to run the reproducer, reply with:
#syz test: git://repo/address.git branch-or-commit-hash
If you attach or paste a git patch, syzbot will apply it before testing.
If you want to change bug's subsystems, reply with:
#syz set subsystems: new-subsystem
(See the list of subsystem names on the web dashboard)
If the bug is a duplicate of another bug, reply with:
#syz dup: exact-subject-of-another-report
If you want to undo deduplication, reply with:
#syz undup
>
> Hello,
>
> syzbot found the following issue on:
>
> HEAD commit: 1c7873e33645 mm: lock newly mapped VMA with corrected orde..
> git tree: upstream
> console output: https://syzkaller.appspot.com/x/log.txt?x=1517663ca80000
> kernel config: https://syzkaller.appspot.com/x/.config?x=832b404e095b70c0
> dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
> compiler: gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2
> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=1776519ca80000
> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=117f9778a80000
>
> Downloadable assets:
> disk image: https://storage.googleapis.com/syzbot-assets/43b42bd2cf70/disk-1c7873e3.raw.xz
> vmlinux: https://storage.googleapis.com/syzbot-assets/011ba1b9e8c1/vmlinux-1c7873e3.xz
> kernel image: https://storage.googleapis.com/syzbot-assets/18b349342595/bzImage-1c7873e3.xz
>
> IMPORTANT: if you fix the issue, please add the following tag to the commit:
> Reported-by: [email protected]
>
> BUG: memory leak
> unreferenced object 0xffff88810b172000 (size 4096):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 01 00 00 00 00 00 00 00 00 bc 96 0e 81 88 ff ff ................
> 5d b6 9e 85 ff ff ff ff 03 00 00 00 00 00 00 00 ]...............
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347eb55>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347eb55>] kzalloc include/linux/slab.h:703 [inline]
> [<ffffffff8347eb55>] dev_new drivers/usb/gadget/legacy/raw_gadget.c:191 [inline]
> [<ffffffff8347eb55>] raw_open+0x45/0x110 drivers/usb/gadget/legacy/raw_gadget.c:385
> [<ffffffff827d1d09>] misc_open+0x1a9/0x1f0 drivers/char/misc.c:165
> [<ffffffff8166763b>] chrdev_open+0x11b/0x340 fs/char_dev.c:414
> [<ffffffff8165573f>] do_dentry_open+0x30f/0x990 fs/open.c:914
> [<ffffffff8167cabb>] do_open fs/namei.c:3636 [inline]
> [<ffffffff8167cabb>] path_openat+0x178b/0x1b20 fs/namei.c:3793
> [<ffffffff8167e995>] do_filp_open+0xc5/0x1b0 fs/namei.c:3820
> [<ffffffff81659453>] do_sys_openat2+0xe3/0x140 fs/open.c:1407
> [<ffffffff81659de3>] do_sys_open fs/open.c:1422 [inline]
> [<ffffffff81659de3>] __do_sys_openat fs/open.c:1438 [inline]
> [<ffffffff81659de3>] __se_sys_openat fs/open.c:1433 [inline]
> [<ffffffff81659de3>] __x64_sys_openat+0x83/0xe0 fs/open.c:1433
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810ad20d60 (size 32):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 72 61 77 2d 67 61 64 67 65 74 2e 30 00 00 00 00 raw-gadget.0....
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347cd2f>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347cd2f>] raw_ioctl_init+0xdf/0x410 drivers/usb/gadget/legacy/raw_gadget.c:460
> [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810e96bc00 (size 128):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 64 75 6d 6d 79 5f 75 64 63 00 00 00 00 00 00 00 dummy_udc.......
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347cd6d>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347cd6d>] raw_ioctl_init+0x11d/0x410 drivers/usb/gadget/legacy/raw_gadget.c:468
> [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810e96ba80 (size 128):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 64 75 6d 6d 79 5f 75 64 63 2e 30 00 00 00 00 00 dummy_udc.0.....
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347cdfc>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347cdfc>] raw_ioctl_init+0x1ac/0x410 drivers/usb/gadget/legacy/raw_gadget.c:479
> [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810e96bd00 (size 128):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 00 bd 96 0e 81 88 ff ff 00 bd 96 0e 81 88 ff ff ................
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff833ecc6a>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff833ecc6a>] kzalloc include/linux/slab.h:703 [inline]
> [<ffffffff833ecc6a>] dummy_alloc_request+0x5a/0xe0 drivers/usb/gadget/udc/dummy_hcd.c:665
> [<ffffffff833e9132>] usb_ep_alloc_request+0x22/0xd0 drivers/usb/gadget/udc/core.c:196
> [<ffffffff8347f13d>] gadget_bind+0x6d/0x370 drivers/usb/gadget/legacy/raw_gadget.c:292
> [<ffffffff833e9e83>] gadget_bind_driver+0xe3/0x2e0 drivers/usb/gadget/udc/core.c:1591
> [<ffffffff82ba069d>] call_driver_probe drivers/base/dd.c:579 [inline]
> [<ffffffff82ba069d>] really_probe+0x12d/0x430 drivers/base/dd.c:658
> [<ffffffff82ba0a61>] __driver_probe_device+0xc1/0x1a0 drivers/base/dd.c:798
> [<ffffffff82ba0b6a>] driver_probe_device+0x2a/0x120 drivers/base/dd.c:828
> [<ffffffff82ba0eae>] __driver_attach drivers/base/dd.c:1214 [inline]
> [<ffffffff82ba0eae>] __driver_attach+0xfe/0x1f0 drivers/base/dd.c:1154
> [<ffffffff82b9d985>] bus_for_each_dev+0xa5/0x110 drivers/base/bus.c:368
> [<ffffffff82b9f486>] bus_add_driver+0x126/0x2a0 drivers/base/bus.c:673
> [<ffffffff82ba25da>] driver_register+0x8a/0x190 drivers/base/driver.c:246
> [<ffffffff833e7524>] usb_gadget_register_driver_owner+0x64/0x160 drivers/usb/gadget/udc/core.c:1665
> [<ffffffff8347e0e6>] raw_ioctl_run drivers/usb/gadget/legacy/raw_gadget.c:546 [inline]
> [<ffffffff8347e0e6>] raw_ioctl+0x6f6/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1253
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
>
#syz test: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
master
diff --git a/drivers/usb/gadget/legacy/raw_gadget.c
b/drivers/usb/gadget/legacy/raw_gadget.c
index 2acece16b890..c00f93b630ad 100644
--- a/drivers/usb/gadget/legacy/raw_gadget.c
+++ b/drivers/usb/gadget/legacy/raw_gadget.c
@@ -310,13 +310,12 @@ static int gadget_bind(struct usb_gadget *gadget,
dev->eps_num = i;
spin_unlock_irqrestore(&dev->lock, flags);
- /* Matches kref_put() in gadget_unbind(). */
- kref_get(&dev->count);
-
ret = raw_queue_event(dev, USB_RAW_EVENT_CONNECT, 0, NULL);
if (ret < 0)
dev_err(&gadget->dev, "failed to queue event\n");
+ /* Matches kref_put() in gadget_unbind(). */
+ kref_get(&dev->count);
return ret;
}
>
>
> ---
> 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 [email protected].
>
> syzbot will keep track of this issue. See:
> https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
>
> If the bug is already fixed, let syzbot know by replying with:
> #syz fix: exact-commit-title
>
> If you want syzbot to run the reproducer, reply with:
> #syz test: git://repo/address.git branch-or-commit-hash
> If you attach or paste a git patch, syzbot will apply it before testing.
>
> If you want to change bug's subsystems, reply with:
> #syz set subsystems: new-subsystem
> (See the list of subsystem names on the web dashboard)
>
> If the bug is a duplicate of another bug, reply with:
> #syz dup: exact-subject-of-another-report
>
> If you want to undo deduplication, reply with:
> #syz undup
On Fri, Jul 14, 2023 at 2:21 PM Z qiang <[email protected]> wrote:
>
> >
> > Hello,
> >
> > syzbot found the following issue on:
> >
> > HEAD commit: 1c7873e33645 mm: lock newly mapped VMA with corrected orde..
> > git tree: upstream
> > console output: https://syzkaller.appspot.com/x/log.txt?x=1517663ca80000
> > kernel config: https://syzkaller.appspot.com/x/.config?x=832b404e095b70c0
> > dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
> > compiler: gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2
> > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=1776519ca80000
> > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=117f9778a80000
> >
> > Downloadable assets:
> > disk image: https://storage.googleapis.com/syzbot-assets/43b42bd2cf70/disk-1c7873e3.raw.xz
> > vmlinux: https://storage.googleapis.com/syzbot-assets/011ba1b9e8c1/vmlinux-1c7873e3.xz
> > kernel image: https://storage.googleapis.com/syzbot-assets/18b349342595/bzImage-1c7873e3.xz
> >
> > IMPORTANT: if you fix the issue, please add the following tag to the commit:
> > Reported-by: [email protected]
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810b172000 (size 4096):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 01 00 00 00 00 00 00 00 00 bc 96 0e 81 88 ff ff ................
> > 5d b6 9e 85 ff ff ff ff 03 00 00 00 00 00 00 00 ]...............
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347eb55>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347eb55>] kzalloc include/linux/slab.h:703 [inline]
> > [<ffffffff8347eb55>] dev_new drivers/usb/gadget/legacy/raw_gadget.c:191 [inline]
> > [<ffffffff8347eb55>] raw_open+0x45/0x110 drivers/usb/gadget/legacy/raw_gadget.c:385
> > [<ffffffff827d1d09>] misc_open+0x1a9/0x1f0 drivers/char/misc.c:165
> > [<ffffffff8166763b>] chrdev_open+0x11b/0x340 fs/char_dev.c:414
> > [<ffffffff8165573f>] do_dentry_open+0x30f/0x990 fs/open.c:914
> > [<ffffffff8167cabb>] do_open fs/namei.c:3636 [inline]
> > [<ffffffff8167cabb>] path_openat+0x178b/0x1b20 fs/namei.c:3793
> > [<ffffffff8167e995>] do_filp_open+0xc5/0x1b0 fs/namei.c:3820
> > [<ffffffff81659453>] do_sys_openat2+0xe3/0x140 fs/open.c:1407
> > [<ffffffff81659de3>] do_sys_open fs/open.c:1422 [inline]
> > [<ffffffff81659de3>] __do_sys_openat fs/open.c:1438 [inline]
> > [<ffffffff81659de3>] __se_sys_openat fs/open.c:1433 [inline]
> > [<ffffffff81659de3>] __x64_sys_openat+0x83/0xe0 fs/open.c:1433
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810ad20d60 (size 32):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 72 61 77 2d 67 61 64 67 65 74 2e 30 00 00 00 00 raw-gadget.0....
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347cd2f>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347cd2f>] raw_ioctl_init+0xdf/0x410 drivers/usb/gadget/legacy/raw_gadget.c:460
> > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810e96bc00 (size 128):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 64 75 6d 6d 79 5f 75 64 63 00 00 00 00 00 00 00 dummy_udc.......
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347cd6d>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347cd6d>] raw_ioctl_init+0x11d/0x410 drivers/usb/gadget/legacy/raw_gadget.c:468
> > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810e96ba80 (size 128):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 64 75 6d 6d 79 5f 75 64 63 2e 30 00 00 00 00 00 dummy_udc.0.....
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347cdfc>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347cdfc>] raw_ioctl_init+0x1ac/0x410 drivers/usb/gadget/legacy/raw_gadget.c:479
> > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810e96bd00 (size 128):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 00 bd 96 0e 81 88 ff ff 00 bd 96 0e 81 88 ff ff ................
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff833ecc6a>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff833ecc6a>] kzalloc include/linux/slab.h:703 [inline]
> > [<ffffffff833ecc6a>] dummy_alloc_request+0x5a/0xe0 drivers/usb/gadget/udc/dummy_hcd.c:665
> > [<ffffffff833e9132>] usb_ep_alloc_request+0x22/0xd0 drivers/usb/gadget/udc/core.c:196
> > [<ffffffff8347f13d>] gadget_bind+0x6d/0x370 drivers/usb/gadget/legacy/raw_gadget.c:292
> > [<ffffffff833e9e83>] gadget_bind_driver+0xe3/0x2e0 drivers/usb/gadget/udc/core.c:1591
> > [<ffffffff82ba069d>] call_driver_probe drivers/base/dd.c:579 [inline]
> > [<ffffffff82ba069d>] really_probe+0x12d/0x430 drivers/base/dd.c:658
> > [<ffffffff82ba0a61>] __driver_probe_device+0xc1/0x1a0 drivers/base/dd.c:798
> > [<ffffffff82ba0b6a>] driver_probe_device+0x2a/0x120 drivers/base/dd.c:828
> > [<ffffffff82ba0eae>] __driver_attach drivers/base/dd.c:1214 [inline]
> > [<ffffffff82ba0eae>] __driver_attach+0xfe/0x1f0 drivers/base/dd.c:1154
> > [<ffffffff82b9d985>] bus_for_each_dev+0xa5/0x110 drivers/base/bus.c:368
> > [<ffffffff82b9f486>] bus_add_driver+0x126/0x2a0 drivers/base/bus.c:673
> > [<ffffffff82ba25da>] driver_register+0x8a/0x190 drivers/base/driver.c:246
> > [<ffffffff833e7524>] usb_gadget_register_driver_owner+0x64/0x160 drivers/usb/gadget/udc/core.c:1665
> > [<ffffffff8347e0e6>] raw_ioctl_run drivers/usb/gadget/legacy/raw_gadget.c:546 [inline]
> > [<ffffffff8347e0e6>] raw_ioctl+0x6f6/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1253
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> >
>
> #syz test: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> master
#syz test: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
master
--- a/drivers/usb/gadget/legacy/raw_gadget.c
+++ b/drivers/usb/gadget/legacy/raw_gadget.c
@@ -310,13 +310,14 @@ static int gadget_bind(struct usb_gadget *gadget,
dev->eps_num = i;
spin_unlock_irqrestore(&dev->lock, flags);
- /* Matches kref_put() in gadget_unbind(). */
- kref_get(&dev->count);
-
ret = raw_queue_event(dev, USB_RAW_EVENT_CONNECT, 0, NULL);
- if (ret < 0)
+ if (ret < 0) {
dev_err(&gadget->dev, "failed to queue event\n");
+ return ret;
+ }
+ /* Matches kref_put() in gadget_unbind(). */
+ kref_get(&dev->count);
return ret;
}
>
> diff --git a/drivers/usb/gadget/legacy/raw_gadget.c
> b/drivers/usb/gadget/legacy/raw_gadget.c
> index 2acece16b890..c00f93b630ad 100644
> --- a/drivers/usb/gadget/legacy/raw_gadget.c
> +++ b/drivers/usb/gadget/legacy/raw_gadget.c
> @@ -310,13 +310,12 @@ static int gadget_bind(struct usb_gadget *gadget,
> dev->eps_num = i;
> spin_unlock_irqrestore(&dev->lock, flags);
>
> - /* Matches kref_put() in gadget_unbind(). */
> - kref_get(&dev->count);
> -
> ret = raw_queue_event(dev, USB_RAW_EVENT_CONNECT, 0, NULL);
> if (ret < 0)
> dev_err(&gadget->dev, "failed to queue event\n");
>
> + /* Matches kref_put() in gadget_unbind(). */
> + kref_get(&dev->count);
> return ret;
> }
>
>
>
> >
> >
> > ---
> > 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 [email protected].
> >
> > syzbot will keep track of this issue. See:
> > https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
> >
> > If the bug is already fixed, let syzbot know by replying with:
> > #syz fix: exact-commit-title
> >
> > If you want syzbot to run the reproducer, reply with:
> > #syz test: git://repo/address.git branch-or-commit-hash
> > If you attach or paste a git patch, syzbot will apply it before testing.
> >
> > If you want to change bug's subsystems, reply with:
> > #syz set subsystems: new-subsystem
> > (See the list of subsystem names on the web dashboard)
> >
> > If the bug is a duplicate of another bug, reply with:
> > #syz dup: exact-subject-of-another-report
> >
> > If you want to undo deduplication, reply with:
> > #syz undup
>
> Hello,
>
> syzbot found the following issue on:
>
> HEAD commit: 1c7873e33645 mm: lock newly mapped VMA with corrected orde..
> git tree: upstream
> console output: https://syzkaller.appspot.com/x/log.txt?x=1517663ca80000
> kernel config: https://syzkaller.appspot.com/x/.config?x=832b404e095b70c0
> dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
> compiler: gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2
> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=1776519ca80000
> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=117f9778a80000
>
> Downloadable assets:
> disk image: https://storage.googleapis.com/syzbot-assets/43b42bd2cf70/disk-1c7873e3.raw.xz
> vmlinux: https://storage.googleapis.com/syzbot-assets/011ba1b9e8c1/vmlinux-1c7873e3.xz
> kernel image: https://storage.googleapis.com/syzbot-assets/18b349342595/bzImage-1c7873e3.xz
>
> IMPORTANT: if you fix the issue, please add the following tag to the commit:
> Reported-by: [email protected]
>
> BUG: memory leak
> unreferenced object 0xffff88810b172000 (size 4096):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 01 00 00 00 00 00 00 00 00 bc 96 0e 81 88 ff ff ................
> 5d b6 9e 85 ff ff ff ff 03 00 00 00 00 00 00 00 ]...............
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347eb55>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347eb55>] kzalloc include/linux/slab.h:703 [inline]
> [<ffffffff8347eb55>] dev_new drivers/usb/gadget/legacy/raw_gadget.c:191 [inline]
> [<ffffffff8347eb55>] raw_open+0x45/0x110 drivers/usb/gadget/legacy/raw_gadget.c:385
> [<ffffffff827d1d09>] misc_open+0x1a9/0x1f0 drivers/char/misc.c:165
> [<ffffffff8166763b>] chrdev_open+0x11b/0x340 fs/char_dev.c:414
> [<ffffffff8165573f>] do_dentry_open+0x30f/0x990 fs/open.c:914
> [<ffffffff8167cabb>] do_open fs/namei.c:3636 [inline]
> [<ffffffff8167cabb>] path_openat+0x178b/0x1b20 fs/namei.c:3793
> [<ffffffff8167e995>] do_filp_open+0xc5/0x1b0 fs/namei.c:3820
> [<ffffffff81659453>] do_sys_openat2+0xe3/0x140 fs/open.c:1407
> [<ffffffff81659de3>] do_sys_open fs/open.c:1422 [inline]
> [<ffffffff81659de3>] __do_sys_openat fs/open.c:1438 [inline]
> [<ffffffff81659de3>] __se_sys_openat fs/open.c:1433 [inline]
> [<ffffffff81659de3>] __x64_sys_openat+0x83/0xe0 fs/open.c:1433
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810ad20d60 (size 32):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 72 61 77 2d 67 61 64 67 65 74 2e 30 00 00 00 00 raw-gadget.0....
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347cd2f>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347cd2f>] raw_ioctl_init+0xdf/0x410 drivers/usb/gadget/legacy/raw_gadget.c:460
> [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810e96bc00 (size 128):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 64 75 6d 6d 79 5f 75 64 63 00 00 00 00 00 00 00 dummy_udc.......
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347cd6d>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347cd6d>] raw_ioctl_init+0x11d/0x410 drivers/usb/gadget/legacy/raw_gadget.c:468
> [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810e96ba80 (size 128):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 64 75 6d 6d 79 5f 75 64 63 2e 30 00 00 00 00 00 dummy_udc.0.....
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff8347cdfc>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff8347cdfc>] raw_ioctl_init+0x1ac/0x410 drivers/usb/gadget/legacy/raw_gadget.c:479
> [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> BUG: memory leak
> unreferenced object 0xffff88810e96bd00 (size 128):
> comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> hex dump (first 32 bytes):
> 00 bd 96 0e 81 88 ff ff 00 bd 96 0e 81 88 ff ff ................
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> backtrace:
> [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> [<ffffffff833ecc6a>] kmalloc include/linux/slab.h:582 [inline]
> [<ffffffff833ecc6a>] kzalloc include/linux/slab.h:703 [inline]
> [<ffffffff833ecc6a>] dummy_alloc_request+0x5a/0xe0 drivers/usb/gadget/udc/dummy_hcd.c:665
> [<ffffffff833e9132>] usb_ep_alloc_request+0x22/0xd0 drivers/usb/gadget/udc/core.c:196
> [<ffffffff8347f13d>] gadget_bind+0x6d/0x370 drivers/usb/gadget/legacy/raw_gadget.c:292
> [<ffffffff833e9e83>] gadget_bind_driver+0xe3/0x2e0 drivers/usb/gadget/udc/core.c:1591
> [<ffffffff82ba069d>] call_driver_probe drivers/base/dd.c:579 [inline]
> [<ffffffff82ba069d>] really_probe+0x12d/0x430 drivers/base/dd.c:658
> [<ffffffff82ba0a61>] __driver_probe_device+0xc1/0x1a0 drivers/base/dd.c:798
> [<ffffffff82ba0b6a>] driver_probe_device+0x2a/0x120 drivers/base/dd.c:828
> [<ffffffff82ba0eae>] __driver_attach drivers/base/dd.c:1214 [inline]
> [<ffffffff82ba0eae>] __driver_attach+0xfe/0x1f0 drivers/base/dd.c:1154
> [<ffffffff82b9d985>] bus_for_each_dev+0xa5/0x110 drivers/base/bus.c:368
> [<ffffffff82b9f486>] bus_add_driver+0x126/0x2a0 drivers/base/bus.c:673
> [<ffffffff82ba25da>] driver_register+0x8a/0x190 drivers/base/driver.c:246
> [<ffffffff833e7524>] usb_gadget_register_driver_owner+0x64/0x160 drivers/usb/gadget/udc/core.c:1665
> [<ffffffff8347e0e6>] raw_ioctl_run drivers/usb/gadget/legacy/raw_gadget.c:546 [inline]
> [<ffffffff8347e0e6>] raw_ioctl+0x6f6/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1253
> [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
>
>
#syz test: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
master
--- a/drivers/usb/gadget/legacy/raw_gadget.c
+++ b/drivers/usb/gadget/legacy/raw_gadget.c
@@ -310,12 +310,15 @@ static int gadget_bind(struct usb_gadget *gadget,
dev->eps_num = i;
spin_unlock_irqrestore(&dev->lock, flags);
- /* Matches kref_put() in gadget_unbind(). */
- kref_get(&dev->count);
-
ret = raw_queue_event(dev, USB_RAW_EVENT_CONNECT, 0, NULL);
- if (ret < 0)
+ if (ret < 0) {
dev_err(&gadget->dev, "failed to queue event\n");
+ set_gadget_data(gadget, NULL);
+ return ret;
+ }
+
+ /* Matches kref_put() in gadget_unbind(). */
+ kref_get(&dev->count);
return ret;
}
Hello,
syzbot tried to test the proposed patch but the build/boot failed:
failed to apply patch:
checking file drivers/usb/gadget/legacy/raw_gadget.c
patch: **** unexpected end of file in patch
Tested on:
commit: 7c2878be Add linux-next specific files for 20230714
git tree: linux-next
dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
compiler:
patch: https://syzkaller.appspot.com/x/patch.diff?x=14266bcca80000
Hello,
syzbot tried to test the proposed patch but the build/boot failed:
failed to apply patch:
checking file drivers/usb/gadget/legacy/raw_gadget.c
patch: **** unexpected end of file in patch
Tested on:
commit: 7c2878be Add linux-next specific files for 20230714
git tree: linux-next
dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
compiler:
patch: https://syzkaller.appspot.com/x/patch.diff?x=118488f8a80000
Hello,
syzbot tried to test the proposed patch but the build/boot failed:
failed to apply patch:
checking file drivers/usb/gadget/legacy/raw_gadget.c
patch: **** unexpected end of file in patch
Tested on:
commit: 7c2878be Add linux-next specific files for 20230714
git tree: linux-next
dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
compiler:
patch: https://syzkaller.appspot.com/x/patch.diff?x=1310af86a80000
Hi,
On Fri, Jul 14, 2023 at 9:12 AM Z qiang <[email protected]> wrote:
>
> >
> > Hello,
> >
> > syzbot found the following issue on:
> >
> > HEAD commit: 1c7873e33645 mm: lock newly mapped VMA with corrected orde..
> > git tree: upstream
> > console output: https://syzkaller.appspot.com/x/log.txt?x=1517663ca80000
> > kernel config: https://syzkaller.appspot.com/x/.config?x=832b404e095b70c0
> > dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
> > compiler: gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2
> > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=1776519ca80000
> > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=117f9778a80000
> >
> > Downloadable assets:
> > disk image: https://storage.googleapis.com/syzbot-assets/43b42bd2cf70/disk-1c7873e3.raw.xz
> > vmlinux: https://storage.googleapis.com/syzbot-assets/011ba1b9e8c1/vmlinux-1c7873e3.xz
> > kernel image: https://storage.googleapis.com/syzbot-assets/18b349342595/bzImage-1c7873e3.xz
> >
> > IMPORTANT: if you fix the issue, please add the following tag to the commit:
> > Reported-by: [email protected]
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810b172000 (size 4096):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 01 00 00 00 00 00 00 00 00 bc 96 0e 81 88 ff ff ................
> > 5d b6 9e 85 ff ff ff ff 03 00 00 00 00 00 00 00 ]...............
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347eb55>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347eb55>] kzalloc include/linux/slab.h:703 [inline]
> > [<ffffffff8347eb55>] dev_new drivers/usb/gadget/legacy/raw_gadget.c:191 [inline]
> > [<ffffffff8347eb55>] raw_open+0x45/0x110 drivers/usb/gadget/legacy/raw_gadget.c:385
> > [<ffffffff827d1d09>] misc_open+0x1a9/0x1f0 drivers/char/misc.c:165
> > [<ffffffff8166763b>] chrdev_open+0x11b/0x340 fs/char_dev.c:414
> > [<ffffffff8165573f>] do_dentry_open+0x30f/0x990 fs/open.c:914
> > [<ffffffff8167cabb>] do_open fs/namei.c:3636 [inline]
> > [<ffffffff8167cabb>] path_openat+0x178b/0x1b20 fs/namei.c:3793
> > [<ffffffff8167e995>] do_filp_open+0xc5/0x1b0 fs/namei.c:3820
> > [<ffffffff81659453>] do_sys_openat2+0xe3/0x140 fs/open.c:1407
> > [<ffffffff81659de3>] do_sys_open fs/open.c:1422 [inline]
> > [<ffffffff81659de3>] __do_sys_openat fs/open.c:1438 [inline]
> > [<ffffffff81659de3>] __se_sys_openat fs/open.c:1433 [inline]
> > [<ffffffff81659de3>] __x64_sys_openat+0x83/0xe0 fs/open.c:1433
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810ad20d60 (size 32):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 72 61 77 2d 67 61 64 67 65 74 2e 30 00 00 00 00 raw-gadget.0....
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347cd2f>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347cd2f>] raw_ioctl_init+0xdf/0x410 drivers/usb/gadget/legacy/raw_gadget.c:460
> > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810e96bc00 (size 128):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 64 75 6d 6d 79 5f 75 64 63 00 00 00 00 00 00 00 dummy_udc.......
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347cd6d>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347cd6d>] raw_ioctl_init+0x11d/0x410 drivers/usb/gadget/legacy/raw_gadget.c:468
> > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810e96ba80 (size 128):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 64 75 6d 6d 79 5f 75 64 63 2e 30 00 00 00 00 00 dummy_udc.0.....
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff8347cdfc>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff8347cdfc>] raw_ioctl_init+0x1ac/0x410 drivers/usb/gadget/legacy/raw_gadget.c:479
> > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> >
> > BUG: memory leak
> > unreferenced object 0xffff88810e96bd00 (size 128):
> > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > hex dump (first 32 bytes):
> > 00 bd 96 0e 81 88 ff ff 00 bd 96 0e 81 88 ff ff ................
> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > backtrace:
> > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > [<ffffffff833ecc6a>] kmalloc include/linux/slab.h:582 [inline]
> > [<ffffffff833ecc6a>] kzalloc include/linux/slab.h:703 [inline]
> > [<ffffffff833ecc6a>] dummy_alloc_request+0x5a/0xe0 drivers/usb/gadget/udc/dummy_hcd.c:665
> > [<ffffffff833e9132>] usb_ep_alloc_request+0x22/0xd0 drivers/usb/gadget/udc/core.c:196
> > [<ffffffff8347f13d>] gadget_bind+0x6d/0x370 drivers/usb/gadget/legacy/raw_gadget.c:292
> > [<ffffffff833e9e83>] gadget_bind_driver+0xe3/0x2e0 drivers/usb/gadget/udc/core.c:1591
> > [<ffffffff82ba069d>] call_driver_probe drivers/base/dd.c:579 [inline]
> > [<ffffffff82ba069d>] really_probe+0x12d/0x430 drivers/base/dd.c:658
> > [<ffffffff82ba0a61>] __driver_probe_device+0xc1/0x1a0 drivers/base/dd.c:798
> > [<ffffffff82ba0b6a>] driver_probe_device+0x2a/0x120 drivers/base/dd.c:828
> > [<ffffffff82ba0eae>] __driver_attach drivers/base/dd.c:1214 [inline]
> > [<ffffffff82ba0eae>] __driver_attach+0xfe/0x1f0 drivers/base/dd.c:1154
> > [<ffffffff82b9d985>] bus_for_each_dev+0xa5/0x110 drivers/base/bus.c:368
> > [<ffffffff82b9f486>] bus_add_driver+0x126/0x2a0 drivers/base/bus.c:673
> > [<ffffffff82ba25da>] driver_register+0x8a/0x190 drivers/base/driver.c:246
> > [<ffffffff833e7524>] usb_gadget_register_driver_owner+0x64/0x160 drivers/usb/gadget/udc/core.c:1665
> > [<ffffffff8347e0e6>] raw_ioctl_run drivers/usb/gadget/legacy/raw_gadget.c:546 [inline]
> > [<ffffffff8347e0e6>] raw_ioctl+0x6f6/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1253
> > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> >
> >
>
>
> #syz test: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> master
>
>
> --- a/drivers/usb/gadget/legacy/raw_gadget.c
> +++ b/drivers/usb/gadget/legacy/raw_gadget.c
> @@ -310,12 +310,15 @@ static int gadget_bind(struct usb_gadget *gadget,
> dev->eps_num = i;
> spin_unlock_irqrestore(&dev->lock, flags);
>
syzbot expects that every line of the git diff patch begins with some
character (unchanged lines with space, added/deleted with + and -
correspondingly). Here you have an empty line and syzbot thinks it's
already the end of the patch. Therefore the patch testing fails.
> - /* Matches kref_put() in gadget_unbind(). */
> - kref_get(&dev->count);
> -
> ret = raw_queue_event(dev, USB_RAW_EVENT_CONNECT, 0, NULL);
> - if (ret < 0)
> + if (ret < 0) {
> dev_err(&gadget->dev, "failed to queue event\n");
> + set_gadget_data(gadget, NULL);
> + return ret;
> + }
> +
> + /* Matches kref_put() in gadget_unbind(). */
> + kref_get(&dev->count);
>
> return ret;
> }
>
> --
> You received this message because you are subscribed to the Google Groups "syzkaller-bugs" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
> To view this discussion on the web visit https://groups.google.com/d/msgid/syzkaller-bugs/CALm%2B0cXi-YsnJBdBt38v4gEoR7oEeaoj3wXjEw3m%3D25RSLEs8g%40mail.gmail.com.
On Fri, Jul 14, 2023 at 5:47 PM Aleksandr Nogikh <[email protected]> wrote:
>
> Hi,
>
> On Fri, Jul 14, 2023 at 9:12 AM Z qiang <[email protected]> wrote:
> >
> > >
> > > Hello,
> > >
> > > syzbot found the following issue on:
> > >
> > > HEAD commit: 1c7873e33645 mm: lock newly mapped VMA with corrected orde..
> > > git tree: upstream
> > > console output: https://syzkaller.appspot.com/x/log.txt?x=1517663ca80000
> > > kernel config: https://syzkaller.appspot.com/x/.config?x=832b404e095b70c0
> > > dashboard link: https://syzkaller.appspot.com/bug?extid=feb045d335c1fdde5bf7
> > > compiler: gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU Binutils for Debian) 2.35.2
> > > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=1776519ca80000
> > > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=117f9778a80000
> > >
> > > Downloadable assets:
> > > disk image: https://storage.googleapis.com/syzbot-assets/43b42bd2cf70/disk-1c7873e3.raw.xz
> > > vmlinux: https://storage.googleapis.com/syzbot-assets/011ba1b9e8c1/vmlinux-1c7873e3.xz
> > > kernel image: https://storage.googleapis.com/syzbot-assets/18b349342595/bzImage-1c7873e3.xz
> > >
> > > IMPORTANT: if you fix the issue, please add the following tag to the commit:
> > > Reported-by: [email protected]
> > >
> > > BUG: memory leak
> > > unreferenced object 0xffff88810b172000 (size 4096):
> > > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > > hex dump (first 32 bytes):
> > > 01 00 00 00 00 00 00 00 00 bc 96 0e 81 88 ff ff ................
> > > 5d b6 9e 85 ff ff ff ff 03 00 00 00 00 00 00 00 ]...............
> > > backtrace:
> > > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > > [<ffffffff8347eb55>] kmalloc include/linux/slab.h:582 [inline]
> > > [<ffffffff8347eb55>] kzalloc include/linux/slab.h:703 [inline]
> > > [<ffffffff8347eb55>] dev_new drivers/usb/gadget/legacy/raw_gadget.c:191 [inline]
> > > [<ffffffff8347eb55>] raw_open+0x45/0x110 drivers/usb/gadget/legacy/raw_gadget.c:385
> > > [<ffffffff827d1d09>] misc_open+0x1a9/0x1f0 drivers/char/misc.c:165
> > > [<ffffffff8166763b>] chrdev_open+0x11b/0x340 fs/char_dev.c:414
> > > [<ffffffff8165573f>] do_dentry_open+0x30f/0x990 fs/open.c:914
> > > [<ffffffff8167cabb>] do_open fs/namei.c:3636 [inline]
> > > [<ffffffff8167cabb>] path_openat+0x178b/0x1b20 fs/namei.c:3793
> > > [<ffffffff8167e995>] do_filp_open+0xc5/0x1b0 fs/namei.c:3820
> > > [<ffffffff81659453>] do_sys_openat2+0xe3/0x140 fs/open.c:1407
> > > [<ffffffff81659de3>] do_sys_open fs/open.c:1422 [inline]
> > > [<ffffffff81659de3>] __do_sys_openat fs/open.c:1438 [inline]
> > > [<ffffffff81659de3>] __se_sys_openat fs/open.c:1433 [inline]
> > > [<ffffffff81659de3>] __x64_sys_openat+0x83/0xe0 fs/open.c:1433
> > > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> > >
> > > BUG: memory leak
> > > unreferenced object 0xffff88810ad20d60 (size 32):
> > > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > > hex dump (first 32 bytes):
> > > 72 61 77 2d 67 61 64 67 65 74 2e 30 00 00 00 00 raw-gadget.0....
> > > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > > backtrace:
> > > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > > [<ffffffff8347cd2f>] kmalloc include/linux/slab.h:582 [inline]
> > > [<ffffffff8347cd2f>] raw_ioctl_init+0xdf/0x410 drivers/usb/gadget/legacy/raw_gadget.c:460
> > > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> > >
> > > BUG: memory leak
> > > unreferenced object 0xffff88810e96bc00 (size 128):
> > > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > > hex dump (first 32 bytes):
> > > 64 75 6d 6d 79 5f 75 64 63 00 00 00 00 00 00 00 dummy_udc.......
> > > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > > backtrace:
> > > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > > [<ffffffff8347cd6d>] kmalloc include/linux/slab.h:582 [inline]
> > > [<ffffffff8347cd6d>] raw_ioctl_init+0x11d/0x410 drivers/usb/gadget/legacy/raw_gadget.c:468
> > > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> > >
> > > BUG: memory leak
> > > unreferenced object 0xffff88810e96ba80 (size 128):
> > > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > > hex dump (first 32 bytes):
> > > 64 75 6d 6d 79 5f 75 64 63 2e 30 00 00 00 00 00 dummy_udc.0.....
> > > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > > backtrace:
> > > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > > [<ffffffff8347cdfc>] kmalloc include/linux/slab.h:582 [inline]
> > > [<ffffffff8347cdfc>] raw_ioctl_init+0x1ac/0x410 drivers/usb/gadget/legacy/raw_gadget.c:479
> > > [<ffffffff8347dfe9>] raw_ioctl+0x5f9/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1250
> > > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > > [<ffffffff84c0008b>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
> > >
> > > BUG: memory leak
> > > unreferenced object 0xffff88810e96bd00 (size 128):
> > > comm "syz-executor103", pid 5067, jiffies 4294964128 (age 13.070s)
> > > hex dump (first 32 bytes):
> > > 00 bd 96 0e 81 88 ff ff 00 bd 96 0e 81 88 ff ff ................
> > > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
> > > backtrace:
> > > [<ffffffff8154bf94>] kmalloc_trace+0x24/0x90 mm/slab_common.c:1076
> > > [<ffffffff833ecc6a>] kmalloc include/linux/slab.h:582 [inline]
> > > [<ffffffff833ecc6a>] kzalloc include/linux/slab.h:703 [inline]
> > > [<ffffffff833ecc6a>] dummy_alloc_request+0x5a/0xe0 drivers/usb/gadget/udc/dummy_hcd.c:665
> > > [<ffffffff833e9132>] usb_ep_alloc_request+0x22/0xd0 drivers/usb/gadget/udc/core.c:196
> > > [<ffffffff8347f13d>] gadget_bind+0x6d/0x370 drivers/usb/gadget/legacy/raw_gadget.c:292
> > > [<ffffffff833e9e83>] gadget_bind_driver+0xe3/0x2e0 drivers/usb/gadget/udc/core.c:1591
> > > [<ffffffff82ba069d>] call_driver_probe drivers/base/dd.c:579 [inline]
> > > [<ffffffff82ba069d>] really_probe+0x12d/0x430 drivers/base/dd.c:658
> > > [<ffffffff82ba0a61>] __driver_probe_device+0xc1/0x1a0 drivers/base/dd.c:798
> > > [<ffffffff82ba0b6a>] driver_probe_device+0x2a/0x120 drivers/base/dd.c:828
> > > [<ffffffff82ba0eae>] __driver_attach drivers/base/dd.c:1214 [inline]
> > > [<ffffffff82ba0eae>] __driver_attach+0xfe/0x1f0 drivers/base/dd.c:1154
> > > [<ffffffff82b9d985>] bus_for_each_dev+0xa5/0x110 drivers/base/bus.c:368
> > > [<ffffffff82b9f486>] bus_add_driver+0x126/0x2a0 drivers/base/bus.c:673
> > > [<ffffffff82ba25da>] driver_register+0x8a/0x190 drivers/base/driver.c:246
> > > [<ffffffff833e7524>] usb_gadget_register_driver_owner+0x64/0x160 drivers/usb/gadget/udc/core.c:1665
> > > [<ffffffff8347e0e6>] raw_ioctl_run drivers/usb/gadget/legacy/raw_gadget.c:546 [inline]
> > > [<ffffffff8347e0e6>] raw_ioctl+0x6f6/0x1120 drivers/usb/gadget/legacy/raw_gadget.c:1253
> > > [<ffffffff81685173>] vfs_ioctl fs/ioctl.c:51 [inline]
> > > [<ffffffff81685173>] __do_sys_ioctl fs/ioctl.c:870 [inline]
> > > [<ffffffff81685173>] __se_sys_ioctl fs/ioctl.c:856 [inline]
> > > [<ffffffff81685173>] __x64_sys_ioctl+0x103/0x140 fs/ioctl.c:856
> > > [<ffffffff84a77ff9>] do_syscall_x64 arch/x86/entry/common.c:50 [inline]
> > > [<ffffffff84a77ff9>] do_syscall_64+0x39/0xb0 arch/x86/entry/common.c:80
> > >
> > >
> >
> >
> > #syz test: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> > master
> >
> >
> > --- a/drivers/usb/gadget/legacy/raw_gadget.c
> > +++ b/drivers/usb/gadget/legacy/raw_gadget.c
> > @@ -310,12 +310,15 @@ static int gadget_bind(struct usb_gadget *gadget,
> > dev->eps_num = i;
> > spin_unlock_irqrestore(&dev->lock, flags);
> >
>
> syzbot expects that every line of the git diff patch begins with some
> character (unchanged lines with space, added/deleted with + and -
> correspondingly). Here you have an empty line and syzbot thinks it's
> already the end of the patch. Therefore the patch testing fails.
>
Thanks for your remind and I also sent a patch :)
Thanks
Zqiang
>
> > - /* Matches kref_put() in gadget_unbind(). */
> > - kref_get(&dev->count);
> > -
> > ret = raw_queue_event(dev, USB_RAW_EVENT_CONNECT, 0, NULL);
> > - if (ret < 0)
> > + if (ret < 0) {
> > dev_err(&gadget->dev, "failed to queue event\n");
> > + set_gadget_data(gadget, NULL);
> > + return ret;
> > + }
> > +
> > + /* Matches kref_put() in gadget_unbind(). */
> > + kref_get(&dev->count);
> >
> > return ret;
> > }
> >
> > --
> > You received this message because you are subscribed to the Google Groups "syzkaller-bugs" group.
> > To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
> > To view this discussion on the web visit https://groups.google.com/d/msgid/syzkaller-bugs/CALm%2B0cXi-YsnJBdBt38v4gEoR7oEeaoj3wXjEw3m%3D25RSLEs8g%40mail.gmail.com.