Re: [syzbot] [usb?] general protection fault in xpad_probe

[Dongliang Mu]

On 2023/4/30 02:09, Alan Stern wrote:
> Dmitry:
>
> On Sat, Apr 29, 2023 at 06:47:43AM -0700, syzbot wrote:
> > syzbot has found a reproducer for the following issue on:
> >
> > HEAD commit:    92e815cf07ed Add linux-next specific files for 20230428
> > git tree:       linux-next
> > console output: https://syzkaller.appspot.com/x/log.txt?x=104cb844280000
> > kernel config:  https://syzkaller.appspot.com/x/.config?x=c8c8ae4d47d23592
> > dashboard link: https://syzkaller.appspot.com/bug?extid=a3f758b8d8cb7e49afec
> > 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=14a5408c280000
> > C reproducer:   https://syzkaller.appspot.com/x/repro.c?x=10d3b0e4280000
> >
> > Downloadable assets:
> > disk image: https://storage.googleapis.com/syzbot-assets/c9e94856e6c9/disk-92e815cf.raw.xz
> > vmlinux: https://storage.googleapis.com/syzbot-assets/4c1c05a548a7/vmlinux-92e815cf.xz
> > kernel image: https://storage.googleapis.com/syzbot-assets/2a1bff6a133b/bzImage-92e815cf.xz
> >
> > IMPORTANT: if you fix the issue, please add the following tag to the commit:
> > Reported-by: syzbot+a3f758b8d8cb7e49afec@xxxxxxxxxxxxxxxxxxxxxxxxx
> >
> > usb 1-1: config 0 has no interface number 0
> > usb 1-1: New USB device found, idVendor=1949, idProduct=5e70, bcdDevice=d7.a2
> > usb 1-1: New USB device strings: Mfr=0, Product=0, SerialNumber=0
> > usb 1-1: config 0 descriptor??
> > usb 1-1: string descriptor 0 read error: -71
> > general protection fault, probably for non-canonical address 0xdffffc0000000068: 0000 [#1] PREEMPT SMP KASAN
> > KASAN: null-ptr-deref in range [0x0000000000000340-0x0000000000000347]
> > CPU: 0 PID: 9 Comm: kworker/0:1 Not tainted 6.3.0-next-20230428-syzkaller #0
> > Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 04/14/2023
> > Workqueue: usb_hub_wq hub_event
> > RIP: 0010:dev_name include/linux/device.h:706 [inline]
> > RIP: 0010:__dev_printk+0x3b/0x270 drivers/base/core.c:4863
> > Code: f5 53 e8 c8 c3 6b fc 48 85 ed 0f 84 cb 01 00 00 e8 ba c3 6b fc 48 8d 7d 50 48 b8 00 00 00 00 00 fc ff df 48 89 fa 48 c1 ea 03 <80> 3c 02 00 0f 85 d7 01 00 00 48 8b 5d 50 48 85 db 0f 84 b5 00 00
> > RSP: 0018:ffffc900000e6f70 EFLAGS: 00010202
> > RAX: dffffc0000000000 RBX: fffff5200001cdf5 RCX: 0000000000000000
> > RDX: 0000000000000068 RSI: ffffffff85188a06 RDI: 0000000000000340
> > RBP: 00000000000002f0 R08: 0000000000000005 R09: 0000000000000000
> > R10: 00000000ffffffb9 R11: ffffffff81d6ff05 R12: ffffffff8ace98e0
> > R13: ffffc900000e6fc8 R14: ffff88801ed25a64 R15: 0000000000000000
> > FS:  0000000000000000(0000) GS:ffff8880b9800000(0000) knlGS:0000000000000000
> > CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > CR2: 00007f1361dc1111 CR3: 00000000210fe000 CR4: 00000000003506f0
> > DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> > DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> > Call Trace:
> >   <TASK>
> >   _dev_warn+0xdc/0x120 drivers/base/core.c:4907
> >   xpad_probe+0x197e/0x2020 drivers/input/joystick/xpad.c:2053
> >   usb_probe_interface+0x30f/0x960 drivers/usb/core/driver.c:396
> It looks like the problem is caused by commit db7220c48d8d ("Input:
> xpad - fix support for some third-party controllers").  In the line
>
> 			dev_warn(&xpad->dev->dev,

Hi Alan,

Vicki had prepared and sent a patch to fix this kernel bug. Please take 
a look at the following threads[1][2].


[1] https://www.spinics.net/lists/linux-input/msg84905.html

[2] https://groups.google.com/g/hust-os-kernel-patches/c/CQY-rdr4kqE

> the xpad->dev ought to be xpad->udev.
>
> Alan Stern