From: Eric Biggers <ebiggers@kernel.org>
To: Boris Pismenny <borisp@nvidia.com>,
Aviad Yehezkel <aviadye@nvidia.com>,
John Fastabend <john.fastabend@gmail.com>,
Daniel Borkmann <daniel@iogearbox.net>,
Jakub Kicinski <kuba@kernel.org>,
netdev@vger.kernel.org
Cc: davem@davemloft.net, glider@google.com,
herbert@gondor.apana.org.au, linux-crypto@vger.kernel.org,
linux-kernel@vger.kernel.org, syzkaller-bugs@googlegroups.com,
syzbot <syzbot+828dfc12440b4f6f305d@syzkaller.appspotmail.com>
Subject: [net/tls] Re: KMSAN: uninit-value in aes_encrypt (4)
Date: Fri, 11 Sep 2020 10:01:50 -0700 [thread overview]
Message-ID: <20200911170150.GA889@sol.localdomain> (raw)
In-Reply-To: <0000000000008a7ae505aef61db1@google.com>
Looks like the tls subsystem is encrypting uninitialized memory again.
+tls maintainers and netdev.
On Thu, Sep 10, 2020 at 07:09:24AM -0700, syzbot wrote:
> Hello,
>
> syzbot found the following issue on:
>
> HEAD commit: 3b3ea602 x86: add failure injection to get/put/clear_user
> git tree: https://github.com/google/kmsan.git master
> console output: https://syzkaller.appspot.com/x/log.txt?x=1030dda5900000
> kernel config: https://syzkaller.appspot.com/x/.config?x=ee5f7a0b2e48ed66
> dashboard link: https://syzkaller.appspot.com/bug?extid=828dfc12440b4f6f305d
> compiler: clang version 10.0.0 (https://github.com/llvm/llvm-project/ c2443155a0fb245c8f17f2c1c72b6ea391e86e81)
> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=17b15055900000
> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=12931621900000
>
> IMPORTANT: if you fix the issue, please add the following tag to the commit:
> Reported-by: syzbot+828dfc12440b4f6f305d@syzkaller.appspotmail.com
>
> =====================================================
> BUG: KMSAN: uninit-value in subshift lib/crypto/aes.c:149 [inline]
> BUG: KMSAN: uninit-value in aes_encrypt+0x12c5/0x1bc0 lib/crypto/aes.c:282
> CPU: 0 PID: 8537 Comm: syz-executor872 Not tainted 5.8.0-rc5-syzkaller #0
> 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+0x21c/0x280 lib/dump_stack.c:118
> kmsan_report+0xf7/0x1e0 mm/kmsan/kmsan_report.c:121
> __msan_warning+0x58/0xa0 mm/kmsan/kmsan_instr.c:215
> subshift lib/crypto/aes.c:149 [inline]
> aes_encrypt+0x12c5/0x1bc0 lib/crypto/aes.c:282
> aesti_encrypt+0xe8/0x130 crypto/aes_ti.c:31
> cipher_crypt_one crypto/cipher.c:75 [inline]
> crypto_cipher_encrypt_one+0x1e2/0x3a0 crypto/cipher.c:82
> crypto_cbcmac_digest_update+0x3e3/0x560 crypto/ccm.c:830
> crypto_shash_update+0x455/0x5a0 crypto/shash.c:119
> shash_ahash_finup+0x20b/0x7a0 crypto/shash.c:291
> shash_async_finup+0xbb/0x110 crypto/shash.c:306
> crypto_ahash_op+0x1c3/0x770 crypto/ahash.c:370
> crypto_ahash_finup+0x8c/0xb0 crypto/ahash.c:395
> crypto_ccm_auth+0x16b6/0x1780 crypto/ccm.c:221
> crypto_ccm_encrypt+0x285/0x850 crypto/ccm.c:300
> crypto_aead_encrypt+0x107/0x190 crypto/aead.c:94
> tls_do_encryption net/tls/tls_sw.c:528 [inline]
> tls_push_record+0x3cb9/0x4fc0 net/tls/tls_sw.c:762
> bpf_exec_tx_verdict+0x195a/0x29e0 net/tls/tls_sw.c:802
> tls_sw_do_sendpage+0x138a/0x1e30 net/tls/tls_sw.c:1213
> tls_sw_sendpage+0x1da/0x250 net/tls/tls_sw.c:1277
> inet_sendpage+0x1dc/0x2f0 net/ipv4/af_inet.c:828
> kernel_sendpage net/socket.c:3642 [inline]
> sock_sendpage+0x1dc/0x2b0 net/socket.c:945
> pipe_to_sendpage+0x3f4/0x530 fs/splice.c:448
> splice_from_pipe_feed fs/splice.c:502 [inline]
> __splice_from_pipe+0x5e3/0xff0 fs/splice.c:626
> splice_from_pipe fs/splice.c:661 [inline]
> generic_splice_sendpage+0x1d5/0x2d0 fs/splice.c:834
> do_splice_from fs/splice.c:846 [inline]
> do_splice+0x2727/0x39e0 fs/splice.c:1144
> __do_sys_splice fs/splice.c:1419 [inline]
> __se_sys_splice+0x323/0x500 fs/splice.c:1401
> __x64_sys_splice+0x6e/0x90 fs/splice.c:1401
> do_syscall_64+0xad/0x160 arch/x86/entry/common.c:386
> entry_SYSCALL_64_after_hwframe+0x44/0xa9
> RIP: 0033:0x446a29
> Code: Bad RIP value.
> RSP: 002b:00007f91f8c0bd98 EFLAGS: 00000246 ORIG_RAX: 0000000000000113
> RAX: ffffffffffffffda RBX: 00000000006dbc48 RCX: 0000000000446a29
> RDX: 0000000000000004 RSI: 0000000000000000 RDI: 0000000000000003
> RBP: 00000000006dbc40 R08: 00080000fffffffc R09: 0000000000000000
> R10: 0000000000000000 R11: 0000000000000246 R12: 00000000006dbc4c
> R13: 0000000020000680 R14: 00000000004ae948 R15: 20c49ba5e353f7cf
>
> Uninit was stored to memory at:
> kmsan_save_stack_with_flags mm/kmsan/kmsan.c:144 [inline]
> kmsan_internal_chain_origin+0xad/0x130 mm/kmsan/kmsan.c:310
> __msan_chain_origin+0x50/0x90 mm/kmsan/kmsan_instr.c:165
> __crypto_xor+0x251/0x1610 crypto/algapi.c:1000
> crypto_xor include/crypto/algapi.h:152 [inline]
> crypto_cbcmac_digest_update+0x2b8/0x560 crypto/ccm.c:824
> crypto_shash_update+0x455/0x5a0 crypto/shash.c:119
> shash_ahash_finup+0x20b/0x7a0 crypto/shash.c:291
> shash_async_finup+0xbb/0x110 crypto/shash.c:306
> crypto_ahash_op+0x1c3/0x770 crypto/ahash.c:370
> crypto_ahash_finup+0x8c/0xb0 crypto/ahash.c:395
> crypto_ccm_auth+0x16b6/0x1780 crypto/ccm.c:221
> crypto_ccm_encrypt+0x285/0x850 crypto/ccm.c:300
> crypto_aead_encrypt+0x107/0x190 crypto/aead.c:94
> tls_do_encryption net/tls/tls_sw.c:528 [inline]
> tls_push_record+0x3cb9/0x4fc0 net/tls/tls_sw.c:762
> bpf_exec_tx_verdict+0x195a/0x29e0 net/tls/tls_sw.c:802
> tls_sw_do_sendpage+0x138a/0x1e30 net/tls/tls_sw.c:1213
> tls_sw_sendpage+0x1da/0x250 net/tls/tls_sw.c:1277
> inet_sendpage+0x1dc/0x2f0 net/ipv4/af_inet.c:828
> kernel_sendpage net/socket.c:3642 [inline]
> sock_sendpage+0x1dc/0x2b0 net/socket.c:945
> pipe_to_sendpage+0x3f4/0x530 fs/splice.c:448
> splice_from_pipe_feed fs/splice.c:502 [inline]
> __splice_from_pipe+0x5e3/0xff0 fs/splice.c:626
> splice_from_pipe fs/splice.c:661 [inline]
> generic_splice_sendpage+0x1d5/0x2d0 fs/splice.c:834
> do_splice_from fs/splice.c:846 [inline]
> do_splice+0x2727/0x39e0 fs/splice.c:1144
> __do_sys_splice fs/splice.c:1419 [inline]
> __se_sys_splice+0x323/0x500 fs/splice.c:1401
> __x64_sys_splice+0x6e/0x90 fs/splice.c:1401
> do_syscall_64+0xad/0x160 arch/x86/entry/common.c:386
> entry_SYSCALL_64_after_hwframe+0x44/0xa9
>
> Uninit was created at:
> kmsan_save_stack_with_flags+0x3c/0x90 mm/kmsan/kmsan.c:144
> kmsan_internal_alloc_meta_for_pages mm/kmsan/kmsan_shadow.c:269 [inline]
> kmsan_alloc_page+0xc5/0x1a0 mm/kmsan/kmsan_shadow.c:293
> __alloc_pages_nodemask+0xdf0/0x1030 mm/page_alloc.c:4889
> alloc_pages_current+0x685/0xb50 mm/mempolicy.c:2292
> alloc_pages include/linux/gfp.h:545 [inline]
> skb_page_frag_refill+0x540/0x780 net/core/sock.c:2487
> sk_page_frag_refill+0xa3/0x3b0 net/core/sock.c:2507
> sk_msg_alloc+0x26e/0x1340 net/core/skmsg.c:38
> tls_alloc_encrypted_msg net/tls/tls_sw.c:289 [inline]
> tls_sw_do_sendpage+0xb0d/0x1e30 net/tls/tls_sw.c:1191
> tls_sw_sendpage+0x1da/0x250 net/tls/tls_sw.c:1277
> inet_sendpage+0x1dc/0x2f0 net/ipv4/af_inet.c:828
> kernel_sendpage net/socket.c:3642 [inline]
> sock_sendpage+0x1dc/0x2b0 net/socket.c:945
> pipe_to_sendpage+0x3f4/0x530 fs/splice.c:448
> splice_from_pipe_feed fs/splice.c:502 [inline]
> __splice_from_pipe+0x5e3/0xff0 fs/splice.c:626
> splice_from_pipe fs/splice.c:661 [inline]
> generic_splice_sendpage+0x1d5/0x2d0 fs/splice.c:834
> do_splice_from fs/splice.c:846 [inline]
> do_splice+0x2727/0x39e0 fs/splice.c:1144
> __do_sys_splice fs/splice.c:1419 [inline]
> __se_sys_splice+0x323/0x500 fs/splice.c:1401
> __x64_sys_splice+0x6e/0x90 fs/splice.c:1401
> do_syscall_64+0xad/0x160 arch/x86/entry/common.c:386
> entry_SYSCALL_64_after_hwframe+0x44/0xa9
> =====================================================
>
>
> ---
> 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 syzkaller@googlegroups.com.
>
> syzbot will keep track of this issue. See:
> https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
> syzbot can test patches for this issue, 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/0000000000008a7ae505aef61db1%40google.com.
next parent reply other threads:[~2020-09-11 17:02 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <0000000000008a7ae505aef61db1@google.com>
2020-09-11 17:01 ` Eric Biggers [this message]
[not found] ` <c16e9ab9-13e0-b911-e33a-c9ae81e93a8d@I-love.SAKURA.ne.jp>
2023-06-28 21:03 ` [PATCH] net: tls: enable __GFP_ZERO upon tls_init() Jakub Kicinski
2023-06-28 22:15 ` Tetsuo Handa
2023-06-30 8:15 ` Eric Biggers
2023-06-30 9:36 ` Ard Biesheuvel
2023-06-30 9:52 ` Tetsuo Handa
2023-06-30 10:02 ` Ard Biesheuvel
2023-06-30 10:10 ` Alexander Potapenko
2023-06-30 10:18 ` Ard Biesheuvel
2023-06-30 11:11 ` Tetsuo Handa
2023-06-30 11:24 ` Eric Dumazet
2023-06-30 11:32 ` Ard Biesheuvel
2023-06-30 11:43 ` Alexander Potapenko
2023-06-30 11:37 ` Alexander Potapenko
2023-06-30 11:49 ` Ard Biesheuvel
2023-06-30 11:55 ` Alexander Potapenko
2023-06-30 15:16 ` Ard Biesheuvel
2023-06-30 15:27 ` Jakub Kicinski
2023-06-30 15:50 ` Ard Biesheuvel
2023-07-01 4:12 ` Tetsuo Handa
2023-07-04 13:32 ` Tetsuo Handa
2023-07-06 20:53 ` Jakub Kicinski
2023-07-07 9:41 ` Tetsuo Handa
2023-07-08 10:34 ` Tetsuo Handa
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200911170150.GA889@sol.localdomain \
--to=ebiggers@kernel.org \
--cc=aviadye@nvidia.com \
--cc=borisp@nvidia.com \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=glider@google.com \
--cc=herbert@gondor.apana.org.au \
--cc=john.fastabend@gmail.com \
--cc=kuba@kernel.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=syzbot+828dfc12440b4f6f305d@syzkaller.appspotmail.com \
--cc=syzkaller-bugs@googlegroups.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).