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

[Alan Stern]

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,

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

Alan Stern