* BUG: corrupted list in io_file_get
@ 2021-01-28 16:58 syzbot
2021-01-28 17:12 ` Pavel Begunkov
0 siblings, 1 reply; 4+ messages in thread
From: syzbot @ 2021-01-28 16:58 UTC (permalink / raw)
To: axboe, io-uring, linux-fsdevel, linux-kernel, syzkaller-bugs,
viro
Hello,
syzbot found the following issue on:
HEAD commit: 76c057c8 Merge branch 'parisc-5.11-2' of git://git.kernel...
git tree: upstream
console output: https://syzkaller.appspot.com/x/log.txt?x=11959454d00000
kernel config: https://syzkaller.appspot.com/x/.config?x=96b123631a6700e9
dashboard link: https://syzkaller.appspot.com/bug?extid=6879187cf57845801267
compiler: gcc (GCC) 10.1.0-syz 20200507
syz repro: https://syzkaller.appspot.com/x/repro.syz?x=12a3872cd00000
C reproducer: https://syzkaller.appspot.com/x/repro.c?x=16ab17a4d00000
The issue was bisected to:
commit 02a13674fa0e8dd326de8b9f4514b41b03d99003
Author: Jens Axboe <[email protected]>
Date: Sat Jan 23 22:49:31 2021 +0000
io_uring: account io_uring internal files as REQ_F_INFLIGHT
bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=14d1bf44d00000
final oops: https://syzkaller.appspot.com/x/report.txt?x=16d1bf44d00000
console output: https://syzkaller.appspot.com/x/log.txt?x=12d1bf44d00000
IMPORTANT: if you fix the issue, please add the following tag to the commit:
Reported-by: [email protected]
Fixes: 02a13674fa0e ("io_uring: account io_uring internal files as REQ_F_INFLIGHT")
list_add double add: new=ffff888017eaa080, prev=ffff88801a9cb520, next=ffff888017eaa080.
------------[ cut here ]------------
kernel BUG at lib/list_debug.c:29!
invalid opcode: 0000 [#1] PREEMPT SMP KASAN
CPU: 0 PID: 8481 Comm: syz-executor556 Not tainted 5.11.0-rc5-syzkaller #0
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
Call Trace:
__list_add include/linux/list.h:67 [inline]
list_add include/linux/list.h:86 [inline]
io_file_get+0x8cc/0xdb0 fs/io_uring.c:6466
__io_splice_prep+0x1bc/0x530 fs/io_uring.c:3866
io_splice_prep fs/io_uring.c:3920 [inline]
io_req_prep+0x3546/0x4e80 fs/io_uring.c:6081
io_queue_sqe+0x609/0x10d0 fs/io_uring.c:6628
io_submit_sqe fs/io_uring.c:6705 [inline]
io_submit_sqes+0x1495/0x2720 fs/io_uring.c:6953
__do_sys_io_uring_enter+0x107d/0x1f30 fs/io_uring.c:9353
do_syscall_64+0x2d/0x70 arch/x86/entry/common.c:46
entry_SYSCALL_64_after_hwframe+0x44/0xa9
RIP: 0033:0x440569
Code: 18 89 d0 c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 0f 83 7b 13 fc ff c3 66 2e 0f 1f 84 00 00 00 00
RSP: 002b:00007ffe38c5c5a8 EFLAGS: 00000246 ORIG_RAX: 00000000000001aa
RAX: ffffffffffffffda RBX: 0000000000401e00 RCX: 0000000000440569
RDX: 0000000000000000 RSI: 000000000000450c RDI: 0000000000000004
RBP: 00000000006ca018 R08: 0000000000000000 R09: 0000000000000000
R10: 0000000000000002 R11: 0000000000000246 R12: 0000000000401d70
R13: 0000000000401e00 R14: 0000000000000000 R15: 0000000000000000
Modules linked in:
---[ end trace 3c68392a0f24e7a0 ]---
RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
---
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.
For information about bisection process see: https://goo.gl/tpsmEJ#bisection
syzbot can test patches for this issue, for details see:
https://goo.gl/tpsmEJ#testing-patches
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: BUG: corrupted list in io_file_get
2021-01-28 16:58 BUG: corrupted list in io_file_get syzbot
@ 2021-01-28 17:12 ` Pavel Begunkov
2021-01-28 17:25 ` Jens Axboe
0 siblings, 1 reply; 4+ messages in thread
From: Pavel Begunkov @ 2021-01-28 17:12 UTC (permalink / raw)
To: syzbot, axboe, io-uring, linux-fsdevel, linux-kernel,
syzkaller-bugs, viro
[-- Attachment #1.1: Type: text/plain, Size: 5303 bytes --]
On 28/01/2021 16:58, syzbot wrote:
> Hello,
>
> syzbot found the following issue on:
>
> HEAD commit: 76c057c8 Merge branch 'parisc-5.11-2' of git://git.kernel...
> git tree: upstream
> console output: https://syzkaller.appspot.com/x/log.txt?x=11959454d00000
> kernel config: https://syzkaller.appspot.com/x/.config?x=96b123631a6700e9
> dashboard link: https://syzkaller.appspot.com/bug?extid=6879187cf57845801267
> compiler: gcc (GCC) 10.1.0-syz 20200507
> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=12a3872cd00000
> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=16ab17a4d00000
>
> The issue was bisected to:
>
> commit 02a13674fa0e8dd326de8b9f4514b41b03d99003
> Author: Jens Axboe <[email protected]>
> Date: Sat Jan 23 22:49:31 2021 +0000
>
> io_uring: account io_uring internal files as REQ_F_INFLIGHT
>
> bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=14d1bf44d00000
> final oops: https://syzkaller.appspot.com/x/report.txt?x=16d1bf44d00000
> console output: https://syzkaller.appspot.com/x/log.txt?x=12d1bf44d00000
>
> IMPORTANT: if you fix the issue, please add the following tag to the commit:
> Reported-by: [email protected]
> Fixes: 02a13674fa0e ("io_uring: account io_uring internal files as REQ_F_INFLIGHT")
>
> list_add double add: new=ffff888017eaa080, prev=ffff88801a9cb520, next=ffff888017eaa080.
> ------------[ cut here ]------------
> kernel BUG at lib/list_debug.c:29!
> invalid opcode: 0000 [#1] PREEMPT SMP KASAN
> CPU: 0 PID: 8481 Comm: syz-executor556 Not tainted 5.11.0-rc5-syzkaller #0
> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
> RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
> Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
> RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
> RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
> RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
> RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
> R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
> R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
> FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
> Call Trace:
> __list_add include/linux/list.h:67 [inline]
> list_add include/linux/list.h:86 [inline]
> io_file_get+0x8cc/0xdb0 fs/io_uring.c:6466
> __io_splice_prep+0x1bc/0x530 fs/io_uring.c:3866
> io_splice_prep fs/io_uring.c:3920 [inline]
> io_req_prep+0x3546/0x4e80 fs/io_uring.c:6081
> io_queue_sqe+0x609/0x10d0 fs/io_uring.c:6628
> io_submit_sqe fs/io_uring.c:6705 [inline]
> io_submit_sqes+0x1495/0x2720 fs/io_uring.c:6953
> __do_sys_io_uring_enter+0x107d/0x1f30 fs/io_uring.c:9353
> do_syscall_64+0x2d/0x70 arch/x86/entry/common.c:46
> entry_SYSCALL_64_after_hwframe+0x44/0xa9
> RIP: 0033:0x440569
> Code: 18 89 d0 c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 0f 83 7b 13 fc ff c3 66 2e 0f 1f 84 00 00 00 00
> RSP: 002b:00007ffe38c5c5a8 EFLAGS: 00000246 ORIG_RAX: 00000000000001aa
> RAX: ffffffffffffffda RBX: 0000000000401e00 RCX: 0000000000440569
> RDX: 0000000000000000 RSI: 000000000000450c RDI: 0000000000000004
> RBP: 00000000006ca018 R08: 0000000000000000 R09: 0000000000000000
> R10: 0000000000000002 R11: 0000000000000246 R12: 0000000000401d70
> R13: 0000000000401e00 R14: 0000000000000000 R15: 0000000000000000
> Modules linked in:
> ---[ end trace 3c68392a0f24e7a0 ]---
> RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
> Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
> RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
> RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
> RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
> RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
> R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
> R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
> FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
This one is simple
diff --git a/fs/io_uring.c b/fs/io_uring.c
index ae388cc52843..39ae1f821cef 100644
--- a/fs/io_uring.c
+++ b/fs/io_uring.c
@@ -6460,7 +6460,8 @@ static struct file *io_file_get(struct io_submit_state *state,
file = __io_file_get(state, fd);
}
- if (file && file->f_op == &io_uring_fops) {
+ if (file && file->f_op == &io_uring_fops &&
+ !(req->flags & REQ_F_INFLIGHT)) {
io_req_init_async(req);
req->flags |= REQ_F_INFLIGHT;
--
Pavel Begunkov
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: BUG: corrupted list in io_file_get
2021-01-28 17:12 ` Pavel Begunkov
@ 2021-01-28 17:25 ` Jens Axboe
2021-01-28 17:28 ` Pavel Begunkov
0 siblings, 1 reply; 4+ messages in thread
From: Jens Axboe @ 2021-01-28 17:25 UTC (permalink / raw)
To: Pavel Begunkov, syzbot, io-uring, linux-fsdevel, linux-kernel,
syzkaller-bugs, viro
On 1/28/21 10:12 AM, Pavel Begunkov wrote:
> On 28/01/2021 16:58, syzbot wrote:
>> Hello,
>>
>> syzbot found the following issue on:
>>
>> HEAD commit: 76c057c8 Merge branch 'parisc-5.11-2' of git://git.kernel...
>> git tree: upstream
>> console output: https://syzkaller.appspot.com/x/log.txt?x=11959454d00000
>> kernel config: https://syzkaller.appspot.com/x/.config?x=96b123631a6700e9
>> dashboard link: https://syzkaller.appspot.com/bug?extid=6879187cf57845801267
>> compiler: gcc (GCC) 10.1.0-syz 20200507
>> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=12a3872cd00000
>> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=16ab17a4d00000
>>
>> The issue was bisected to:
>>
>> commit 02a13674fa0e8dd326de8b9f4514b41b03d99003
>> Author: Jens Axboe <[email protected]>
>> Date: Sat Jan 23 22:49:31 2021 +0000
>>
>> io_uring: account io_uring internal files as REQ_F_INFLIGHT
>>
>> bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=14d1bf44d00000
>> final oops: https://syzkaller.appspot.com/x/report.txt?x=16d1bf44d00000
>> console output: https://syzkaller.appspot.com/x/log.txt?x=12d1bf44d00000
>>
>> IMPORTANT: if you fix the issue, please add the following tag to the commit:
>> Reported-by: [email protected]
>> Fixes: 02a13674fa0e ("io_uring: account io_uring internal files as REQ_F_INFLIGHT")
>>
>> list_add double add: new=ffff888017eaa080, prev=ffff88801a9cb520, next=ffff888017eaa080.
>> ------------[ cut here ]------------
>> kernel BUG at lib/list_debug.c:29!
>> invalid opcode: 0000 [#1] PREEMPT SMP KASAN
>> CPU: 0 PID: 8481 Comm: syz-executor556 Not tainted 5.11.0-rc5-syzkaller #0
>> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
>> RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
>> Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
>> RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
>> RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
>> RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
>> RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
>> R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
>> R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
>> FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
>> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>> CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
>> Call Trace:
>> __list_add include/linux/list.h:67 [inline]
>> list_add include/linux/list.h:86 [inline]
>> io_file_get+0x8cc/0xdb0 fs/io_uring.c:6466
>> __io_splice_prep+0x1bc/0x530 fs/io_uring.c:3866
>> io_splice_prep fs/io_uring.c:3920 [inline]
>> io_req_prep+0x3546/0x4e80 fs/io_uring.c:6081
>> io_queue_sqe+0x609/0x10d0 fs/io_uring.c:6628
>> io_submit_sqe fs/io_uring.c:6705 [inline]
>> io_submit_sqes+0x1495/0x2720 fs/io_uring.c:6953
>> __do_sys_io_uring_enter+0x107d/0x1f30 fs/io_uring.c:9353
>> do_syscall_64+0x2d/0x70 arch/x86/entry/common.c:46
>> entry_SYSCALL_64_after_hwframe+0x44/0xa9
>> RIP: 0033:0x440569
>> Code: 18 89 d0 c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 0f 83 7b 13 fc ff c3 66 2e 0f 1f 84 00 00 00 00
>> RSP: 002b:00007ffe38c5c5a8 EFLAGS: 00000246 ORIG_RAX: 00000000000001aa
>> RAX: ffffffffffffffda RBX: 0000000000401e00 RCX: 0000000000440569
>> RDX: 0000000000000000 RSI: 000000000000450c RDI: 0000000000000004
>> RBP: 00000000006ca018 R08: 0000000000000000 R09: 0000000000000000
>> R10: 0000000000000002 R11: 0000000000000246 R12: 0000000000401d70
>> R13: 0000000000401e00 R14: 0000000000000000 R15: 0000000000000000
>> Modules linked in:
>> ---[ end trace 3c68392a0f24e7a0 ]---
>> RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
>> Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
>> RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
>> RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
>> RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
>> RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
>> R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
>> R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
>> FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
>> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>> CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
>
> This one is simple
>
> diff --git a/fs/io_uring.c b/fs/io_uring.c
> index ae388cc52843..39ae1f821cef 100644
> --- a/fs/io_uring.c
> +++ b/fs/io_uring.c
> @@ -6460,7 +6460,8 @@ static struct file *io_file_get(struct io_submit_state *state,
> file = __io_file_get(state, fd);
> }
>
> - if (file && file->f_op == &io_uring_fops) {
> + if (file && file->f_op == &io_uring_fops &&
> + !(req->flags & REQ_F_INFLIGHT)) {
> io_req_init_async(req);
> req->flags |= REQ_F_INFLIGHT;
Curious, how is it marked in-flight already? Ah it's splice... Pavel,
can you send the fix for this one?
--
Jens Axboe
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: BUG: corrupted list in io_file_get
2021-01-28 17:25 ` Jens Axboe
@ 2021-01-28 17:28 ` Pavel Begunkov
0 siblings, 0 replies; 4+ messages in thread
From: Pavel Begunkov @ 2021-01-28 17:28 UTC (permalink / raw)
To: Jens Axboe, syzbot, io-uring, linux-fsdevel, linux-kernel,
syzkaller-bugs, viro
On 28/01/2021 17:25, Jens Axboe wrote:
> On 1/28/21 10:12 AM, Pavel Begunkov wrote:
>> On 28/01/2021 16:58, syzbot wrote:
>>> Hello,
>>>
>>> syzbot found the following issue on:
>>>
>>> HEAD commit: 76c057c8 Merge branch 'parisc-5.11-2' of git://git.kernel...
>>> git tree: upstream
>>> console output: https://syzkaller.appspot.com/x/log.txt?x=11959454d00000
>>> kernel config: https://syzkaller.appspot.com/x/.config?x=96b123631a6700e9
>>> dashboard link: https://syzkaller.appspot.com/bug?extid=6879187cf57845801267
>>> compiler: gcc (GCC) 10.1.0-syz 20200507
>>> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=12a3872cd00000
>>> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=16ab17a4d00000
>>>
>>> The issue was bisected to:
>>>
>>> commit 02a13674fa0e8dd326de8b9f4514b41b03d99003
>>> Author: Jens Axboe <[email protected]>
>>> Date: Sat Jan 23 22:49:31 2021 +0000
>>>
>>> io_uring: account io_uring internal files as REQ_F_INFLIGHT
>>>
>>> bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=14d1bf44d00000
>>> final oops: https://syzkaller.appspot.com/x/report.txt?x=16d1bf44d00000
>>> console output: https://syzkaller.appspot.com/x/log.txt?x=12d1bf44d00000
>>>
>>> IMPORTANT: if you fix the issue, please add the following tag to the commit:
>>> Reported-by: [email protected]
>>> Fixes: 02a13674fa0e ("io_uring: account io_uring internal files as REQ_F_INFLIGHT")
>>>
>>> list_add double add: new=ffff888017eaa080, prev=ffff88801a9cb520, next=ffff888017eaa080.
>>> ------------[ cut here ]------------
>>> kernel BUG at lib/list_debug.c:29!
>>> invalid opcode: 0000 [#1] PREEMPT SMP KASAN
>>> CPU: 0 PID: 8481 Comm: syz-executor556 Not tainted 5.11.0-rc5-syzkaller #0
>>> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
>>> RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
>>> Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
>>> RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
>>> RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
>>> RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
>>> RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
>>> R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
>>> R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
>>> FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
>>> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>>> CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
>>> Call Trace:
>>> __list_add include/linux/list.h:67 [inline]
>>> list_add include/linux/list.h:86 [inline]
>>> io_file_get+0x8cc/0xdb0 fs/io_uring.c:6466
>>> __io_splice_prep+0x1bc/0x530 fs/io_uring.c:3866
>>> io_splice_prep fs/io_uring.c:3920 [inline]
>>> io_req_prep+0x3546/0x4e80 fs/io_uring.c:6081
>>> io_queue_sqe+0x609/0x10d0 fs/io_uring.c:6628
>>> io_submit_sqe fs/io_uring.c:6705 [inline]
>>> io_submit_sqes+0x1495/0x2720 fs/io_uring.c:6953
>>> __do_sys_io_uring_enter+0x107d/0x1f30 fs/io_uring.c:9353
>>> do_syscall_64+0x2d/0x70 arch/x86/entry/common.c:46
>>> entry_SYSCALL_64_after_hwframe+0x44/0xa9
>>> RIP: 0033:0x440569
>>> Code: 18 89 d0 c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 0f 83 7b 13 fc ff c3 66 2e 0f 1f 84 00 00 00 00
>>> RSP: 002b:00007ffe38c5c5a8 EFLAGS: 00000246 ORIG_RAX: 00000000000001aa
>>> RAX: ffffffffffffffda RBX: 0000000000401e00 RCX: 0000000000440569
>>> RDX: 0000000000000000 RSI: 000000000000450c RDI: 0000000000000004
>>> RBP: 00000000006ca018 R08: 0000000000000000 R09: 0000000000000000
>>> R10: 0000000000000002 R11: 0000000000000246 R12: 0000000000401d70
>>> R13: 0000000000401e00 R14: 0000000000000000 R15: 0000000000000000
>>> Modules linked in:
>>> ---[ end trace 3c68392a0f24e7a0 ]---
>>> RIP: 0010:__list_add_valid.cold+0x26/0x3c lib/list_debug.c:29
>>> Code: 04 c3 fb fa 4c 89 e1 48 c7 c7 e0 de 9e 89 e8 9e 43 f3 ff 0f 0b 48 89 f2 4c 89 e1 48 89 ee 48 c7 c7 20 e0 9e 89 e8 87 43 f3 ff <0f> 0b 48 89 f1 48 c7 c7 a0 df 9e 89 4c 89 e6 e8 73 43 f3 ff 0f 0b
>>> RSP: 0018:ffffc90000fef938 EFLAGS: 00010086
>>> RAX: 0000000000000058 RBX: ffff888017eaa000 RCX: 0000000000000000
>>> RDX: ffff88801f3ed340 RSI: ffffffff815b6285 RDI: fffff520001fdf19
>>> RBP: ffff888017eaa080 R08: 0000000000000058 R09: 0000000000000000
>>> R10: ffffffff815af45e R11: 0000000000000000 R12: ffff888017eaa080
>>> R13: ffff888014901900 R14: ffff88801a9cb000 R15: ffff88801a9cb520
>>> FS: 0000000002395880(0000) GS:ffff8880b9e00000(0000) knlGS:0000000000000000
>>> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>>> CR2: 00007ff04f95b6c0 CR3: 000000001a4f2000 CR4: 0000000000350ef0
>>
>> This one is simple
>>
>> diff --git a/fs/io_uring.c b/fs/io_uring.c
>> index ae388cc52843..39ae1f821cef 100644
>> --- a/fs/io_uring.c
>> +++ b/fs/io_uring.c
>> @@ -6460,7 +6460,8 @@ static struct file *io_file_get(struct io_submit_state *state,
>> file = __io_file_get(state, fd);
>> }
>>
>> - if (file && file->f_op == &io_uring_fops) {
>> + if (file && file->f_op == &io_uring_fops &&
>> + !(req->flags & REQ_F_INFLIGHT)) {
>> io_req_init_async(req);
>> req->flags |= REQ_F_INFLIGHT;
>
> Curious, how is it marked in-flight already? Ah it's splice... Pavel,
Yeah, splice was handled automatically in the end...
> can you send the fix for this one?
sure, in an hour
--
Pavel Begunkov
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-01-28 17:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-28 16:58 BUG: corrupted list in io_file_get syzbot
2021-01-28 17:12 ` Pavel Begunkov
2021-01-28 17:25 ` Jens Axboe
2021-01-28 17:28 ` Pavel Begunkov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox