Message ID | tencent_B0E02F1D6C009450E8D6EC06CC6C7B5E6C0A@qq.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 237ff253f2d4f6307b7b20434d7cbcc67693298b |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | mptcp: fix uninit-value in mptcp_incoming_options | expand |
On Thu, 2023-11-23 at 09:23 +0800, Edward Adam Davis wrote: > Added initialization use_ack to mptcp_parse_option(). > > Reported-by: syzbot+b834a6b2decad004cfa1@syzkaller.appspotmail.com > Signed-off-by: Edward Adam Davis <eadavis@qq.com> > --- > net/mptcp/options.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/net/mptcp/options.c b/net/mptcp/options.c > index cd15ec73073e..c53914012d01 100644 > --- a/net/mptcp/options.c > +++ b/net/mptcp/options.c > @@ -108,6 +108,7 @@ static void mptcp_parse_option(const struct sk_buff *skb, > mp_opt->suboptions |= OPTION_MPTCP_DSS; > mp_opt->use_map = 1; > mp_opt->mpc_map = 1; > + mp_opt->use_ack = 0; > mp_opt->data_len = get_unaligned_be16(ptr); > ptr += 2; > } LGTM, and syzbot tested it. Acked-by: Paolo Abeni <pabeni@redhat.com> @Edward: for future similar patches, please add also the tested tag from syzbot, will make tracking easier. Thanks! Paolo
On Thu, 23 Nov 2023 17:41:31 +0100, Paolo Abeni wrote: > > Added initialization use_ack to mptcp_parse_option(). > > > > Reported-by: syzbot+b834a6b2decad004cfa1@syzkaller.appspotmail.com > > Signed-off-by: Edward Adam Davis <eadavis@qq.com> > > --- > > net/mptcp/options.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/net/mptcp/options.c b/net/mptcp/options.c > > index cd15ec73073e..c53914012d01 100644 > > --- a/net/mptcp/options.c > > +++ b/net/mptcp/options.c > > @@ -108,6 +108,7 @@ static void mptcp_parse_option(const struct sk_buff *skb, > > mp_opt->suboptions |= OPTION_MPTCP_DSS; > > mp_opt->use_map = 1; > > mp_opt->mpc_map = 1; > > + mp_opt->use_ack = 0; > > mp_opt->data_len = get_unaligned_be16(ptr); > > ptr += 2; > > } > > LGTM, and syzbot tested it. > > Acked-by: Paolo Abeni <pabeni@redhat.com> > > @Edward: for future similar patches, please add also the tested tag > from syzbot, will make tracking easier. OK, I will. Edward
Hello: This patch was applied to netdev/net.git (main) by David S. Miller <davem@davemloft.net>: On Thu, 23 Nov 2023 09:23:39 +0800 you wrote: > Added initialization use_ack to mptcp_parse_option(). > > Reported-by: syzbot+b834a6b2decad004cfa1@syzkaller.appspotmail.com > Signed-off-by: Edward Adam Davis <eadavis@qq.com> > --- > net/mptcp/options.c | 1 + > 1 file changed, 1 insertion(+) Here is the summary with links: - mptcp: fix uninit-value in mptcp_incoming_options https://git.kernel.org/netdev/net/c/237ff253f2d4 You are awesome, thank you!
diff --git a/net/mptcp/options.c b/net/mptcp/options.c index cd15ec73073e..c53914012d01 100644 --- a/net/mptcp/options.c +++ b/net/mptcp/options.c @@ -108,6 +108,7 @@ static void mptcp_parse_option(const struct sk_buff *skb, mp_opt->suboptions |= OPTION_MPTCP_DSS; mp_opt->use_map = 1; mp_opt->mpc_map = 1; + mp_opt->use_ack = 0; mp_opt->data_len = get_unaligned_be16(ptr); ptr += 2; }
Added initialization use_ack to mptcp_parse_option(). Reported-by: syzbot+b834a6b2decad004cfa1@syzkaller.appspotmail.com Signed-off-by: Edward Adam Davis <eadavis@qq.com> --- net/mptcp/options.c | 1 + 1 file changed, 1 insertion(+)