* KMSAN: uninit-value in gf128mul_4k_lle (3)
@ 2018-11-23 22:50 syzbot
2018-11-23 23:01 ` Ard Biesheuvel
0 siblings, 1 reply; 4+ messages in thread
From: syzbot @ 2018-11-23 22:50 UTC (permalink / raw)
To: davem, herbert, linux-crypto, linux-kernel, syzkaller-bugs
Hello,
syzbot found the following crash on:
HEAD commit: cddc52641fd2 kmsan: use __memmove() in fixup_bad_iret()
git tree: https://github.com/google/kmsan.git/master
console output: https://syzkaller.appspot.com/x/log.txt?x=111c426d400000
kernel config: https://syzkaller.appspot.com/x/.config?x=e2808e34f8becb71
dashboard link: https://syzkaller.appspot.com/bug?extid=f8495bff23a879a6d0bd
compiler: clang version 8.0.0 (trunk 343298)
syz repro: https://syzkaller.appspot.com/x/repro.syz?x=17c0326d400000
C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17e4d45d400000
IMPORTANT: if you fix the bug, please add the following tag to the commit:
Reported-by: syzbot+f8495bff23a879a6d0bd@syzkaller.appspotmail.com
Local variable description: ----walk@crypto_ctr_crypt
Variable was created at:
crypto_ctr_crypt+0xae/0xc30 crypto/ctr.c:129
skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
==================================================================
BUG: KMSAN: uninit-value in gf128mul_4k_lle+0x29e/0x310
crypto/gf128mul.c:391
CPU: 0 PID: 8470 Comm: syz-executor696 Not tainted 4.20.0-rc2+ #88
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS
Google 01/01/2011
Call Trace:
__dump_stack lib/dump_stack.c:77 [inline]
dump_stack+0x32d/0x480 lib/dump_stack.c:113
kmsan_report+0x19f/0x300 mm/kmsan/kmsan.c:911
__msan_warning+0x76/0xc0 mm/kmsan/kmsan_instr.c:415
gf128mul_4k_lle+0x29e/0x310 crypto/gf128mul.c:391
ghash_update+0x9d3/0x10e0 crypto/ghash-generic.c:75
crypto_shash_update crypto/shash.c:103 [inline]
shash_ahash_update+0x4de/0x600 crypto/shash.c:244
shash_async_update+0x50/0x60 crypto/shash.c:252
crypto_ahash_update include/crypto/hash.h:557 [inline]
gcm_hash_update crypto/gcm.c:235 [inline]
gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
gcm_hash_init_continue crypto/gcm.c:400 [inline]
gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
gcm_encrypt_continue crypto/gcm.c:455 [inline]
crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
crypto_aead_encrypt include/crypto/aead.h:364 [inline]
tls_do_encryption net/tls/tls_sw.c:460 [inline]
tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
sock_sendmsg_nosec net/socket.c:621 [inline]
sock_sendmsg net/socket.c:631 [inline]
__sys_sendto+0x940/0xb80 net/socket.c:1788
__do_sys_sendto net/socket.c:1800 [inline]
__se_sys_sendto+0x107/0x130 net/socket.c:1796
__x64_sys_sendto+0x6e/0x90 net/socket.c:1796
do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
entry_SYSCALL_64_after_hwframe+0x63/0xe7
RIP: 0033:0x448509
Code: e8 fc e5 ff ff 48 83 c4 18 c3 0f 1f 80 00 00 00 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 0b 01 fc ff c3 66 2e 0f 1f 84 00 00 00 00
RSP: 002b:00007f9ff6aa3cd8 EFLAGS: 00000216 ORIG_RAX: 000000000000002c
RAX: ffffffffffffffda RBX: 00000000006f0038 RCX: 0000000000448509
RDX: 000000000039a191 RSI: 00000000200005c0 RDI: 0000000000000003
RBP: 00000000006f0030 R08: 0000000020000000 R09: 000000000000001c
R10: 0000000000000000 R11: 0000000000000216 R12: 00000000006f003c
R13: 00000000007ffc6f R14: 00007f9ff6aa49c0 R15: 00000000000003e8
Uninit was stored to memory at:
kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
__msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
__crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
crypto_xor include/crypto/algapi.h:219 [inline]
ghash_update+0x991/0x10e0 crypto/ghash-generic.c:74
crypto_shash_update crypto/shash.c:103 [inline]
shash_ahash_update+0x4de/0x600 crypto/shash.c:244
shash_async_update+0x50/0x60 crypto/shash.c:252
crypto_ahash_update include/crypto/hash.h:557 [inline]
gcm_hash_update crypto/gcm.c:235 [inline]
gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
gcm_hash_init_continue crypto/gcm.c:400 [inline]
gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
gcm_encrypt_continue crypto/gcm.c:455 [inline]
crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
crypto_aead_encrypt include/crypto/aead.h:364 [inline]
tls_do_encryption net/tls/tls_sw.c:460 [inline]
tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
sock_sendmsg_nosec net/socket.c:621 [inline]
sock_sendmsg net/socket.c:631 [inline]
__sys_sendto+0x940/0xb80 net/socket.c:1788
__do_sys_sendto net/socket.c:1800 [inline]
__se_sys_sendto+0x107/0x130 net/socket.c:1796
__x64_sys_sendto+0x6e/0x90 net/socket.c:1796
do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
entry_SYSCALL_64_after_hwframe+0x63/0xe7
Uninit was stored to memory at:
kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
__msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
__crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
crypto_xor include/crypto/algapi.h:219 [inline]
crypto_ctr_crypt_inplace crypto/ctr.c:115 [inline]
crypto_ctr_crypt+0x776/0xc30 crypto/ctr.c:142
skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
crypto_skcipher_encrypt include/crypto/skcipher.h:534 [inline]
crypto_gcm_encrypt+0x512/0xaa0 crypto/gcm.c:483
crypto_aead_encrypt include/crypto/aead.h:364 [inline]
tls_do_encryption net/tls/tls_sw.c:460 [inline]
tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
sock_sendmsg_nosec net/socket.c:621 [inline]
sock_sendmsg net/socket.c:631 [inline]
__sys_sendto+0x940/0xb80 net/socket.c:1788
__do_sys_sendto net/socket.c:1800 [inline]
__se_sys_sendto+0x107/0x130 net/socket.c:1796
__x64_sys_sendto+0x6e/0x90 net/socket.c:1796
do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
entry_SYSCALL_64_after_hwframe+0x63/0xe7
Uninit was created at:
kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
kmsan_internal_alloc_meta_for_pages+0x155/0x740 mm/kmsan/kmsan.c:689
kmsan_alloc_page+0x77/0xc0 mm/kmsan/kmsan_hooks.c:320
__alloc_pages_nodemask+0x12ac/0x64d0 mm/page_alloc.c:4421
alloc_pages_current+0x55d/0x7d0 mm/mempolicy.c:2080
alloc_pages include/linux/gfp.h:511 [inline]
skb_page_frag_refill+0x48e/0x7a0 net/core/sock.c:2213
sk_page_frag_refill+0xa4/0x330 net/core/sock.c:2233
sk_msg_alloc+0x22f/0x11a0 net/core/skmsg.c:37
tls_alloc_encrypted_msg net/tls/tls_sw.c:236 [inline]
tls_sw_sendmsg+0xd0c/0x2a30 net/tls/tls_sw.c:871
inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
sock_sendmsg_nosec net/socket.c:621 [inline]
sock_sendmsg net/socket.c:631 [inline]
__sys_sendto+0x940/0xb80 net/socket.c:1788
__do_sys_sendto net/socket.c:1800 [inline]
__se_sys_sendto+0x107/0x130 net/socket.c:1796
__x64_sys_sendto+0x6e/0x90 net/socket.c:1796
do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
entry_SYSCALL_64_after_hwframe+0x63/0xe7
==================================================================
---
This bug 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 syzkaller@googlegroups.com.
syzbot will keep track of this bug report. See:
https://goo.gl/tpsmEJ#bug-status-tracking for how to communicate with
syzbot.
syzbot can test patches for this bug, for details see:
https://goo.gl/tpsmEJ#testing-patches
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: KMSAN: uninit-value in gf128mul_4k_lle (3)
2018-11-23 22:50 KMSAN: uninit-value in gf128mul_4k_lle (3) syzbot
@ 2018-11-23 23:01 ` Ard Biesheuvel
2018-11-28 10:58 ` Alexander Potapenko
0 siblings, 1 reply; 4+ messages in thread
From: Ard Biesheuvel @ 2018-11-23 23:01 UTC (permalink / raw)
To: syzbot+f8495bff23a879a6d0bd, Boris Pismenny, Aviad Yehezkel,
Dave Watson
Cc: David S. Miller, Herbert Xu,
open list:HARDWARE RANDOM NUMBER GENERATOR CORE,
Linux Kernel Mailing List, syzkaller-bugs
(+ TLS maintainers)
On Fri, 23 Nov 2018 at 23:51, syzbot
<syzbot+f8495bff23a879a6d0bd@syzkaller.appspotmail.com> wrote:
>
> Hello,
>
> syzbot found the following crash on:
>
> HEAD commit: cddc52641fd2 kmsan: use __memmove() in fixup_bad_iret()
> git tree: https://github.com/google/kmsan.git/master
> console output: https://syzkaller.appspot.com/x/log.txt?x=111c426d400000
> kernel config: https://syzkaller.appspot.com/x/.config?x=e2808e34f8becb71
> dashboard link: https://syzkaller.appspot.com/bug?extid=f8495bff23a879a6d0bd
> compiler: clang version 8.0.0 (trunk 343298)
> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=17c0326d400000
> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17e4d45d400000
>
> IMPORTANT: if you fix the bug, please add the following tag to the commit:
> Reported-by: syzbot+f8495bff23a879a6d0bd@syzkaller.appspotmail.com
>
> Local variable description: ----walk@crypto_ctr_crypt
> Variable was created at:
> crypto_ctr_crypt+0xae/0xc30 crypto/ctr.c:129
> skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
> skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
> ==================================================================
> BUG: KMSAN: uninit-value in gf128mul_4k_lle+0x29e/0x310
> crypto/gf128mul.c:391
> CPU: 0 PID: 8470 Comm: syz-executor696 Not tainted 4.20.0-rc2+ #88
> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS
> Google 01/01/2011
> Call Trace:
> __dump_stack lib/dump_stack.c:77 [inline]
> dump_stack+0x32d/0x480 lib/dump_stack.c:113
> kmsan_report+0x19f/0x300 mm/kmsan/kmsan.c:911
> __msan_warning+0x76/0xc0 mm/kmsan/kmsan_instr.c:415
> gf128mul_4k_lle+0x29e/0x310 crypto/gf128mul.c:391
> ghash_update+0x9d3/0x10e0 crypto/ghash-generic.c:75
> crypto_shash_update crypto/shash.c:103 [inline]
> shash_ahash_update+0x4de/0x600 crypto/shash.c:244
> shash_async_update+0x50/0x60 crypto/shash.c:252
> crypto_ahash_update include/crypto/hash.h:557 [inline]
> gcm_hash_update crypto/gcm.c:235 [inline]
> gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
> gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
> gcm_hash_init_continue crypto/gcm.c:400 [inline]
> gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
> gcm_encrypt_continue crypto/gcm.c:455 [inline]
> crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
> crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> tls_do_encryption net/tls/tls_sw.c:460 [inline]
> tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> sock_sendmsg_nosec net/socket.c:621 [inline]
> sock_sendmsg net/socket.c:631 [inline]
> __sys_sendto+0x940/0xb80 net/socket.c:1788
> __do_sys_sendto net/socket.c:1800 [inline]
> __se_sys_sendto+0x107/0x130 net/socket.c:1796
> __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> entry_SYSCALL_64_after_hwframe+0x63/0xe7
> RIP: 0033:0x448509
> Code: e8 fc e5 ff ff 48 83 c4 18 c3 0f 1f 80 00 00 00 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 0b 01 fc ff c3 66 2e 0f 1f 84 00 00 00 00
> RSP: 002b:00007f9ff6aa3cd8 EFLAGS: 00000216 ORIG_RAX: 000000000000002c
> RAX: ffffffffffffffda RBX: 00000000006f0038 RCX: 0000000000448509
> RDX: 000000000039a191 RSI: 00000000200005c0 RDI: 0000000000000003
> RBP: 00000000006f0030 R08: 0000000020000000 R09: 000000000000001c
> R10: 0000000000000000 R11: 0000000000000216 R12: 00000000006f003c
> R13: 00000000007ffc6f R14: 00007f9ff6aa49c0 R15: 00000000000003e8
>
> Uninit was stored to memory at:
> kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
> kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
> __msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
> __crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
> crypto_xor include/crypto/algapi.h:219 [inline]
> ghash_update+0x991/0x10e0 crypto/ghash-generic.c:74
> crypto_shash_update crypto/shash.c:103 [inline]
> shash_ahash_update+0x4de/0x600 crypto/shash.c:244
> shash_async_update+0x50/0x60 crypto/shash.c:252
> crypto_ahash_update include/crypto/hash.h:557 [inline]
> gcm_hash_update crypto/gcm.c:235 [inline]
> gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
> gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
> gcm_hash_init_continue crypto/gcm.c:400 [inline]
> gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
> gcm_encrypt_continue crypto/gcm.c:455 [inline]
> crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
> crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> tls_do_encryption net/tls/tls_sw.c:460 [inline]
> tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> sock_sendmsg_nosec net/socket.c:621 [inline]
> sock_sendmsg net/socket.c:631 [inline]
> __sys_sendto+0x940/0xb80 net/socket.c:1788
> __do_sys_sendto net/socket.c:1800 [inline]
> __se_sys_sendto+0x107/0x130 net/socket.c:1796
> __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> entry_SYSCALL_64_after_hwframe+0x63/0xe7
>
> Uninit was stored to memory at:
> kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
> kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
> __msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
> __crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
> crypto_xor include/crypto/algapi.h:219 [inline]
> crypto_ctr_crypt_inplace crypto/ctr.c:115 [inline]
> crypto_ctr_crypt+0x776/0xc30 crypto/ctr.c:142
> skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
> skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
> crypto_skcipher_encrypt include/crypto/skcipher.h:534 [inline]
> crypto_gcm_encrypt+0x512/0xaa0 crypto/gcm.c:483
> crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> tls_do_encryption net/tls/tls_sw.c:460 [inline]
> tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> sock_sendmsg_nosec net/socket.c:621 [inline]
> sock_sendmsg net/socket.c:631 [inline]
> __sys_sendto+0x940/0xb80 net/socket.c:1788
> __do_sys_sendto net/socket.c:1800 [inline]
> __se_sys_sendto+0x107/0x130 net/socket.c:1796
> __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> entry_SYSCALL_64_after_hwframe+0x63/0xe7
>
> Uninit was created at:
> kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> kmsan_internal_alloc_meta_for_pages+0x155/0x740 mm/kmsan/kmsan.c:689
> kmsan_alloc_page+0x77/0xc0 mm/kmsan/kmsan_hooks.c:320
> __alloc_pages_nodemask+0x12ac/0x64d0 mm/page_alloc.c:4421
> alloc_pages_current+0x55d/0x7d0 mm/mempolicy.c:2080
> alloc_pages include/linux/gfp.h:511 [inline]
> skb_page_frag_refill+0x48e/0x7a0 net/core/sock.c:2213
> sk_page_frag_refill+0xa4/0x330 net/core/sock.c:2233
> sk_msg_alloc+0x22f/0x11a0 net/core/skmsg.c:37
> tls_alloc_encrypted_msg net/tls/tls_sw.c:236 [inline]
> tls_sw_sendmsg+0xd0c/0x2a30 net/tls/tls_sw.c:871
> inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> sock_sendmsg_nosec net/socket.c:621 [inline]
> sock_sendmsg net/socket.c:631 [inline]
> __sys_sendto+0x940/0xb80 net/socket.c:1788
> __do_sys_sendto net/socket.c:1800 [inline]
> __se_sys_sendto+0x107/0x130 net/socket.c:1796
> __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> entry_SYSCALL_64_after_hwframe+0x63/0xe7
> ==================================================================
>
>
> ---
> This bug 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 syzkaller@googlegroups.com.
>
> syzbot will keep track of this bug report. See:
> https://goo.gl/tpsmEJ#bug-status-tracking for how to communicate with
> syzbot.
> syzbot can test patches for this bug, for details see:
> https://goo.gl/tpsmEJ#testing-patches
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: KMSAN: uninit-value in gf128mul_4k_lle (3)
2018-11-23 23:01 ` Ard Biesheuvel
@ 2018-11-28 10:58 ` Alexander Potapenko
2019-06-10 22:25 ` [net/tls] " Eric Biggers
0 siblings, 1 reply; 4+ messages in thread
From: Alexander Potapenko @ 2018-11-28 10:58 UTC (permalink / raw)
To: Ard Biesheuvel
Cc: syzbot+f8495bff23a879a6d0bd, borisp, aviadye, davejwatson,
David Miller, Herbert Xu, linux-crypto, LKML, syzkaller-bugs
On Sat, Nov 24, 2018 at 12:02 AM Ard Biesheuvel
<ard.biesheuvel@linaro.org> wrote:
>
> (+ TLS maintainers)
This bug is also reproducible without KMSAN: if we fill the newly
allocated skb fragment with "AAAA" (see the patch below) we can see
these bytes being passed into the crypto functions.
Note that KMSAN enables CONFIG_GENERIC_CSUM, so one might need
something along the lines of
https://github.com/google/kmsan/commit/fffec98ae2a605a3b8a6b3518e3d61d66c1bfd0a
to reproduce this.
=======================================
diff --git a/crypto/algapi.c b/crypto/algapi.c
index 2545c5f89c4c..654b3e9877a5 100644
--- a/crypto/algapi.c
+++ b/crypto/algapi.c
@@ -1005,6 +1005,25 @@ void __crypto_xor(u8 *dst, const u8 *src1,
const u8 *src2, unsigned int len)
{
int relalign = 0;
+ char saved;
+ char *s1 = (char*)src1, *s2 = (char*)src2;
+ if (len) {
+ saved = s1[len-1];
+ s1[len-1] = 0;
+ if (strstr(s1, "AAAA"))
+ pr_err("src1: %s\n", s1);
+ s1[len-1] = saved;
+ }
+ if (len) {
+ saved = s2[len-1];
+ s2[len-1] = 0;
+ if (strstr(s2, "AAAA")) {
+ pr_err("src2: %s\n", s2);
+ if (s1 == s2)
+ pr_err("s1 == s2\n");
+ }
+ s2[len-1] = saved;
+ }
if (!IS_ENABLED(CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS)) {
int size = sizeof(unsigned long);
int d = (((unsigned long)dst ^ (unsigned long)src1) |
diff --git a/net/core/sock.c b/net/core/sock.c
index 080a880a1761..83f15668c3e8 100644
--- a/net/core/sock.c
+++ b/net/core/sock.c
@@ -2216,6 +2216,7 @@ bool skb_page_frag_refill(unsigned int sz,
struct page_frag *pfrag, gfp_t gfp)
SKB_FRAG_PAGE_ORDER);
if (likely(pfrag->page)) {
pfrag->size = PAGE_SIZE << SKB_FRAG_PAGE_ORDER;
+ memset(page_address(pfrag->page), 'A', pfrag->size);
return true;
}
}
The bug itself seems to be a data race, it only worked for me with the
multithreaded C reproducer that collided the syscalls.
> On Fri, 23 Nov 2018 at 23:51, syzbot
> <syzbot+f8495bff23a879a6d0bd@syzkaller.appspotmail.com> wrote:
> >
> > Hello,
> >
> > syzbot found the following crash on:
> >
> > HEAD commit: cddc52641fd2 kmsan: use __memmove() in fixup_bad_iret()
> > git tree: https://github.com/google/kmsan.git/master
> > console output: https://syzkaller.appspot.com/x/log.txt?x=111c426d400000
> > kernel config: https://syzkaller.appspot.com/x/.config?x=e2808e34f8becb71
> > dashboard link: https://syzkaller.appspot.com/bug?extid=f8495bff23a879a6d0bd
> > compiler: clang version 8.0.0 (trunk 343298)
> > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=17c0326d400000
> > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17e4d45d400000
> >
> > IMPORTANT: if you fix the bug, please add the following tag to the commit:
> > Reported-by: syzbot+f8495bff23a879a6d0bd@syzkaller.appspotmail.com
> >
> > Local variable description: ----walk@crypto_ctr_crypt
> > Variable was created at:
> > crypto_ctr_crypt+0xae/0xc30 crypto/ctr.c:129
> > skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
> > skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
> > ==================================================================
> > BUG: KMSAN: uninit-value in gf128mul_4k_lle+0x29e/0x310
> > crypto/gf128mul.c:391
> > CPU: 0 PID: 8470 Comm: syz-executor696 Not tainted 4.20.0-rc2+ #88
> > Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS
> > Google 01/01/2011
> > Call Trace:
> > __dump_stack lib/dump_stack.c:77 [inline]
> > dump_stack+0x32d/0x480 lib/dump_stack.c:113
> > kmsan_report+0x19f/0x300 mm/kmsan/kmsan.c:911
> > __msan_warning+0x76/0xc0 mm/kmsan/kmsan_instr.c:415
> > gf128mul_4k_lle+0x29e/0x310 crypto/gf128mul.c:391
> > ghash_update+0x9d3/0x10e0 crypto/ghash-generic.c:75
> > crypto_shash_update crypto/shash.c:103 [inline]
> > shash_ahash_update+0x4de/0x600 crypto/shash.c:244
> > shash_async_update+0x50/0x60 crypto/shash.c:252
> > crypto_ahash_update include/crypto/hash.h:557 [inline]
> > gcm_hash_update crypto/gcm.c:235 [inline]
> > gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
> > gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
> > gcm_hash_init_continue crypto/gcm.c:400 [inline]
> > gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
> > gcm_encrypt_continue crypto/gcm.c:455 [inline]
> > crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
> > crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> > tls_do_encryption net/tls/tls_sw.c:460 [inline]
> > tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> > bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> > tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > sock_sendmsg_nosec net/socket.c:621 [inline]
> > sock_sendmsg net/socket.c:631 [inline]
> > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > __do_sys_sendto net/socket.c:1800 [inline]
> > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> > RIP: 0033:0x448509
> > Code: e8 fc e5 ff ff 48 83 c4 18 c3 0f 1f 80 00 00 00 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 0b 01 fc ff c3 66 2e 0f 1f 84 00 00 00 00
> > RSP: 002b:00007f9ff6aa3cd8 EFLAGS: 00000216 ORIG_RAX: 000000000000002c
> > RAX: ffffffffffffffda RBX: 00000000006f0038 RCX: 0000000000448509
> > RDX: 000000000039a191 RSI: 00000000200005c0 RDI: 0000000000000003
> > RBP: 00000000006f0030 R08: 0000000020000000 R09: 000000000000001c
> > R10: 0000000000000000 R11: 0000000000000216 R12: 00000000006f003c
> > R13: 00000000007ffc6f R14: 00007f9ff6aa49c0 R15: 00000000000003e8
> >
> > Uninit was stored to memory at:
> > kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> > kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
> > kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
> > __msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
> > __crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
> > crypto_xor include/crypto/algapi.h:219 [inline]
> > ghash_update+0x991/0x10e0 crypto/ghash-generic.c:74
> > crypto_shash_update crypto/shash.c:103 [inline]
> > shash_ahash_update+0x4de/0x600 crypto/shash.c:244
> > shash_async_update+0x50/0x60 crypto/shash.c:252
> > crypto_ahash_update include/crypto/hash.h:557 [inline]
> > gcm_hash_update crypto/gcm.c:235 [inline]
> > gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
> > gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
> > gcm_hash_init_continue crypto/gcm.c:400 [inline]
> > gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
> > gcm_encrypt_continue crypto/gcm.c:455 [inline]
> > crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
> > crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> > tls_do_encryption net/tls/tls_sw.c:460 [inline]
> > tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> > bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> > tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > sock_sendmsg_nosec net/socket.c:621 [inline]
> > sock_sendmsg net/socket.c:631 [inline]
> > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > __do_sys_sendto net/socket.c:1800 [inline]
> > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> >
> > Uninit was stored to memory at:
> > kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> > kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
> > kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
> > __msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
> > __crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
> > crypto_xor include/crypto/algapi.h:219 [inline]
> > crypto_ctr_crypt_inplace crypto/ctr.c:115 [inline]
> > crypto_ctr_crypt+0x776/0xc30 crypto/ctr.c:142
> > skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
> > skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
> > crypto_skcipher_encrypt include/crypto/skcipher.h:534 [inline]
> > crypto_gcm_encrypt+0x512/0xaa0 crypto/gcm.c:483
> > crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> > tls_do_encryption net/tls/tls_sw.c:460 [inline]
> > tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> > bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> > tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > sock_sendmsg_nosec net/socket.c:621 [inline]
> > sock_sendmsg net/socket.c:631 [inline]
> > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > __do_sys_sendto net/socket.c:1800 [inline]
> > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> >
> > Uninit was created at:
> > kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> > kmsan_internal_alloc_meta_for_pages+0x155/0x740 mm/kmsan/kmsan.c:689
> > kmsan_alloc_page+0x77/0xc0 mm/kmsan/kmsan_hooks.c:320
> > __alloc_pages_nodemask+0x12ac/0x64d0 mm/page_alloc.c:4421
> > alloc_pages_current+0x55d/0x7d0 mm/mempolicy.c:2080
> > alloc_pages include/linux/gfp.h:511 [inline]
> > skb_page_frag_refill+0x48e/0x7a0 net/core/sock.c:2213
> > sk_page_frag_refill+0xa4/0x330 net/core/sock.c:2233
> > sk_msg_alloc+0x22f/0x11a0 net/core/skmsg.c:37
> > tls_alloc_encrypted_msg net/tls/tls_sw.c:236 [inline]
> > tls_sw_sendmsg+0xd0c/0x2a30 net/tls/tls_sw.c:871
> > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > sock_sendmsg_nosec net/socket.c:621 [inline]
> > sock_sendmsg net/socket.c:631 [inline]
> > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > __do_sys_sendto net/socket.c:1800 [inline]
> > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> > ==================================================================
> >
> >
> > ---
> > This bug 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 syzkaller@googlegroups.com.
> >
> > syzbot will keep track of this bug report. See:
> > https://goo.gl/tpsmEJ#bug-status-tracking for how to communicate with
> > syzbot.
> > syzbot can test patches for this bug, for details see:
> > https://goo.gl/tpsmEJ#testing-patches
>
> --
> 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 syzkaller-bugs+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/syzkaller-bugs/CAKv%2BGu8nZhmFhSy_FJLA2HkO4O56N3BdowNKmBcmCRoH%2BiNx%2BA%40mail.gmail.com.
> For more options, visit https://groups.google.com/d/optout.
--
Alexander Potapenko
Software Engineer
Google Germany GmbH
Erika-Mann-Straße, 33
80636 München
Geschäftsführer: Paul Manicle, Halimah DeLaine Prado
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [net/tls] Re: KMSAN: uninit-value in gf128mul_4k_lle (3)
2018-11-28 10:58 ` Alexander Potapenko
@ 2019-06-10 22:25 ` Eric Biggers
0 siblings, 0 replies; 4+ messages in thread
From: Eric Biggers @ 2019-06-10 22:25 UTC (permalink / raw)
To: Boris Pismenny, Aviad Yehezkel, Dave Watson
Cc: Alexander Potapenko, David Miller, netdev,
syzbot+f8495bff23a879a6d0bd, syzkaller-bugs
This TLS bug is still present. See more recent report here:
https://syzkaller.appspot.com/text?tag=CrashReport&x=10613446a00000
On Wed, Nov 28, 2018 at 11:58:37AM +0100, 'Alexander Potapenko' via syzkaller-bugs wrote:
> On Sat, Nov 24, 2018 at 12:02 AM Ard Biesheuvel
> <ard.biesheuvel@linaro.org> wrote:
> >
> > (+ TLS maintainers)
>
> This bug is also reproducible without KMSAN: if we fill the newly
> allocated skb fragment with "AAAA" (see the patch below) we can see
> these bytes being passed into the crypto functions.
> Note that KMSAN enables CONFIG_GENERIC_CSUM, so one might need
> something along the lines of
> https://github.com/google/kmsan/commit/fffec98ae2a605a3b8a6b3518e3d61d66c1bfd0a
> to reproduce this.
>
> =======================================
> diff --git a/crypto/algapi.c b/crypto/algapi.c
> index 2545c5f89c4c..654b3e9877a5 100644
> --- a/crypto/algapi.c
> +++ b/crypto/algapi.c
> @@ -1005,6 +1005,25 @@ void __crypto_xor(u8 *dst, const u8 *src1,
> const u8 *src2, unsigned int len)
> {
> int relalign = 0;
>
> + char saved;
> + char *s1 = (char*)src1, *s2 = (char*)src2;
> + if (len) {
> + saved = s1[len-1];
> + s1[len-1] = 0;
> + if (strstr(s1, "AAAA"))
> + pr_err("src1: %s\n", s1);
> + s1[len-1] = saved;
> + }
> + if (len) {
> + saved = s2[len-1];
> + s2[len-1] = 0;
> + if (strstr(s2, "AAAA")) {
> + pr_err("src2: %s\n", s2);
> + if (s1 == s2)
> + pr_err("s1 == s2\n");
> + }
> + s2[len-1] = saved;
> + }
> if (!IS_ENABLED(CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS)) {
> int size = sizeof(unsigned long);
> int d = (((unsigned long)dst ^ (unsigned long)src1) |
> diff --git a/net/core/sock.c b/net/core/sock.c
> index 080a880a1761..83f15668c3e8 100644
> --- a/net/core/sock.c
> +++ b/net/core/sock.c
> @@ -2216,6 +2216,7 @@ bool skb_page_frag_refill(unsigned int sz,
> struct page_frag *pfrag, gfp_t gfp)
> SKB_FRAG_PAGE_ORDER);
> if (likely(pfrag->page)) {
> pfrag->size = PAGE_SIZE << SKB_FRAG_PAGE_ORDER;
> + memset(page_address(pfrag->page), 'A', pfrag->size);
> return true;
> }
> }
>
> The bug itself seems to be a data race, it only worked for me with the
> multithreaded C reproducer that collided the syscalls.
> > On Fri, 23 Nov 2018 at 23:51, syzbot
> > <syzbot+f8495bff23a879a6d0bd@syzkaller.appspotmail.com> wrote:
> > >
> > > Hello,
> > >
> > > syzbot found the following crash on:
> > >
> > > HEAD commit: cddc52641fd2 kmsan: use __memmove() in fixup_bad_iret()
> > > git tree: https://github.com/google/kmsan.git/master
> > > console output: https://syzkaller.appspot.com/x/log.txt?x=111c426d400000
> > > kernel config: https://syzkaller.appspot.com/x/.config?x=e2808e34f8becb71
> > > dashboard link: https://syzkaller.appspot.com/bug?extid=f8495bff23a879a6d0bd
> > > compiler: clang version 8.0.0 (trunk 343298)
> > > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=17c0326d400000
> > > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17e4d45d400000
> > >
> > > IMPORTANT: if you fix the bug, please add the following tag to the commit:
> > > Reported-by: syzbot+f8495bff23a879a6d0bd@syzkaller.appspotmail.com
> > >
> > > Local variable description: ----walk@crypto_ctr_crypt
> > > Variable was created at:
> > > crypto_ctr_crypt+0xae/0xc30 crypto/ctr.c:129
> > > skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
> > > skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
> > > ==================================================================
> > > BUG: KMSAN: uninit-value in gf128mul_4k_lle+0x29e/0x310
> > > crypto/gf128mul.c:391
> > > CPU: 0 PID: 8470 Comm: syz-executor696 Not tainted 4.20.0-rc2+ #88
> > > Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS
> > > Google 01/01/2011
> > > Call Trace:
> > > __dump_stack lib/dump_stack.c:77 [inline]
> > > dump_stack+0x32d/0x480 lib/dump_stack.c:113
> > > kmsan_report+0x19f/0x300 mm/kmsan/kmsan.c:911
> > > __msan_warning+0x76/0xc0 mm/kmsan/kmsan_instr.c:415
> > > gf128mul_4k_lle+0x29e/0x310 crypto/gf128mul.c:391
> > > ghash_update+0x9d3/0x10e0 crypto/ghash-generic.c:75
> > > crypto_shash_update crypto/shash.c:103 [inline]
> > > shash_ahash_update+0x4de/0x600 crypto/shash.c:244
> > > shash_async_update+0x50/0x60 crypto/shash.c:252
> > > crypto_ahash_update include/crypto/hash.h:557 [inline]
> > > gcm_hash_update crypto/gcm.c:235 [inline]
> > > gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
> > > gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
> > > gcm_hash_init_continue crypto/gcm.c:400 [inline]
> > > gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
> > > gcm_encrypt_continue crypto/gcm.c:455 [inline]
> > > crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
> > > crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> > > tls_do_encryption net/tls/tls_sw.c:460 [inline]
> > > tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> > > bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> > > tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> > > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > > sock_sendmsg_nosec net/socket.c:621 [inline]
> > > sock_sendmsg net/socket.c:631 [inline]
> > > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > > __do_sys_sendto net/socket.c:1800 [inline]
> > > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> > > RIP: 0033:0x448509
> > > Code: e8 fc e5 ff ff 48 83 c4 18 c3 0f 1f 80 00 00 00 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 0b 01 fc ff c3 66 2e 0f 1f 84 00 00 00 00
> > > RSP: 002b:00007f9ff6aa3cd8 EFLAGS: 00000216 ORIG_RAX: 000000000000002c
> > > RAX: ffffffffffffffda RBX: 00000000006f0038 RCX: 0000000000448509
> > > RDX: 000000000039a191 RSI: 00000000200005c0 RDI: 0000000000000003
> > > RBP: 00000000006f0030 R08: 0000000020000000 R09: 000000000000001c
> > > R10: 0000000000000000 R11: 0000000000000216 R12: 00000000006f003c
> > > R13: 00000000007ffc6f R14: 00007f9ff6aa49c0 R15: 00000000000003e8
> > >
> > > Uninit was stored to memory at:
> > > kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> > > kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
> > > kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
> > > __msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
> > > __crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
> > > crypto_xor include/crypto/algapi.h:219 [inline]
> > > ghash_update+0x991/0x10e0 crypto/ghash-generic.c:74
> > > crypto_shash_update crypto/shash.c:103 [inline]
> > > shash_ahash_update+0x4de/0x600 crypto/shash.c:244
> > > shash_async_update+0x50/0x60 crypto/shash.c:252
> > > crypto_ahash_update include/crypto/hash.h:557 [inline]
> > > gcm_hash_update crypto/gcm.c:235 [inline]
> > > gcm_hash_assoc_remain_continue crypto/gcm.c:344 [inline]
> > > gcm_hash_assoc_continue crypto/gcm.c:375 [inline]
> > > gcm_hash_init_continue crypto/gcm.c:400 [inline]
> > > gcm_hash+0x1dbe/0x4870 crypto/gcm.c:430
> > > gcm_encrypt_continue crypto/gcm.c:455 [inline]
> > > crypto_gcm_encrypt+0x781/0xaa0 crypto/gcm.c:484
> > > crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> > > tls_do_encryption net/tls/tls_sw.c:460 [inline]
> > > tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> > > bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> > > tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> > > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > > sock_sendmsg_nosec net/socket.c:621 [inline]
> > > sock_sendmsg net/socket.c:631 [inline]
> > > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > > __do_sys_sendto net/socket.c:1800 [inline]
> > > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> > >
> > > Uninit was stored to memory at:
> > > kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> > > kmsan_save_stack mm/kmsan/kmsan.c:267 [inline]
> > > kmsan_internal_chain_origin+0x136/0x240 mm/kmsan/kmsan.c:569
> > > __msan_chain_origin+0x6d/0xb0 mm/kmsan/kmsan_instr.c:292
> > > __crypto_xor+0x224/0x15c0 crypto/algapi.c:1029
> > > crypto_xor include/crypto/algapi.h:219 [inline]
> > > crypto_ctr_crypt_inplace crypto/ctr.c:115 [inline]
> > > crypto_ctr_crypt+0x776/0xc30 crypto/ctr.c:142
> > > skcipher_crypt_blkcipher crypto/skcipher.c:622 [inline]
> > > skcipher_encrypt_blkcipher+0x232/0x340 crypto/skcipher.c:631
> > > crypto_skcipher_encrypt include/crypto/skcipher.h:534 [inline]
> > > crypto_gcm_encrypt+0x512/0xaa0 crypto/gcm.c:483
> > > crypto_aead_encrypt include/crypto/aead.h:364 [inline]
> > > tls_do_encryption net/tls/tls_sw.c:460 [inline]
> > > tls_push_record+0x2545/0x4290 net/tls/tls_sw.c:657
> > > bpf_exec_tx_verdict+0x16c0/0x1b40 net/tls/tls_sw.c:694
> > > tls_sw_sendmsg+0x136d/0x2a30 net/tls/tls_sw.c:949
> > > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > > sock_sendmsg_nosec net/socket.c:621 [inline]
> > > sock_sendmsg net/socket.c:631 [inline]
> > > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > > __do_sys_sendto net/socket.c:1800 [inline]
> > > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> > >
> > > Uninit was created at:
> > > kmsan_save_stack_with_flags mm/kmsan/kmsan.c:252 [inline]
> > > kmsan_internal_alloc_meta_for_pages+0x155/0x740 mm/kmsan/kmsan.c:689
> > > kmsan_alloc_page+0x77/0xc0 mm/kmsan/kmsan_hooks.c:320
> > > __alloc_pages_nodemask+0x12ac/0x64d0 mm/page_alloc.c:4421
> > > alloc_pages_current+0x55d/0x7d0 mm/mempolicy.c:2080
> > > alloc_pages include/linux/gfp.h:511 [inline]
> > > skb_page_frag_refill+0x48e/0x7a0 net/core/sock.c:2213
> > > sk_page_frag_refill+0xa4/0x330 net/core/sock.c:2233
> > > sk_msg_alloc+0x22f/0x11a0 net/core/skmsg.c:37
> > > tls_alloc_encrypted_msg net/tls/tls_sw.c:236 [inline]
> > > tls_sw_sendmsg+0xd0c/0x2a30 net/tls/tls_sw.c:871
> > > inet_sendmsg+0x4e9/0x800 net/ipv4/af_inet.c:798
> > > sock_sendmsg_nosec net/socket.c:621 [inline]
> > > sock_sendmsg net/socket.c:631 [inline]
> > > __sys_sendto+0x940/0xb80 net/socket.c:1788
> > > __do_sys_sendto net/socket.c:1800 [inline]
> > > __se_sys_sendto+0x107/0x130 net/socket.c:1796
> > > __x64_sys_sendto+0x6e/0x90 net/socket.c:1796
> > > do_syscall_64+0xcf/0x110 arch/x86/entry/common.c:291
> > > entry_SYSCALL_64_after_hwframe+0x63/0xe7
> > > ==================================================================
> > >
> > >
> > > ---
> > > This bug 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 syzkaller@googlegroups.com.
> > >
> > > syzbot will keep track of this bug report. See:
> > > https://goo.gl/tpsmEJ#bug-status-tracking for how to communicate with
> > > syzbot.
> > > syzbot can test patches for this bug, for details see:
> > > https://goo.gl/tpsmEJ#testing-patches
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-06-10 22:25 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-23 22:50 KMSAN: uninit-value in gf128mul_4k_lle (3) syzbot
2018-11-23 23:01 ` Ard Biesheuvel
2018-11-28 10:58 ` Alexander Potapenko
2019-06-10 22:25 ` [net/tls] " Eric Biggers
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.