All of lore.kernel.org
 help / color / mirror / Atom feed
* [MPTCP] [mptcp:export 1339/1352] net/mptcp/options.c:687 mptcp_established_options_rm_addr() error: uninitialized symbol 'align'.
@ 2021-02-15 12:21 ` Dan Carpenter
  0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-02-15 12:21 UTC (permalink / raw)
  To: mptcp 

[-- Attachment #1: Type: text/plain, Size: 3236 bytes --]

tree:   https://github.com/multipath-tcp/mptcp_net-next.git export
head:   d15532665bd1e4a0e210110b275658ba4fcb5b87
commit: 7619397044315f5289200ea122ce4aac316d30da [1339/1352] mptcp: add rm_list in mptcp_out_options
config: i386-randconfig-m021-20210209 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
Reported-by: Dan Carpenter <dan.carpenter(a)oracle.com>

smatch warnings:
net/mptcp/options.c:687 mptcp_established_options_rm_addr() error: uninitialized symbol 'align'.

vim +/align +687 net/mptcp/options.c

5cb104ae55e9db Geliang Tang  2020-09-24  668  static bool mptcp_established_options_rm_addr(struct sock *sk,
5cb104ae55e9db Geliang Tang  2020-09-24  669  					      unsigned int *size,
5cb104ae55e9db Geliang Tang  2020-09-24  670  					      unsigned int remaining,
5cb104ae55e9db Geliang Tang  2020-09-24  671  					      struct mptcp_out_options *opts)
5cb104ae55e9db Geliang Tang  2020-09-24  672  {
5cb104ae55e9db Geliang Tang  2020-09-24  673  	struct mptcp_subflow_context *subflow = mptcp_subflow_ctx(sk);
5cb104ae55e9db Geliang Tang  2020-09-24  674  	struct mptcp_sock *msk = mptcp_sk(subflow->conn);
7619397044315f Geliang Tang  2021-02-12  675  	struct mptcp_rm_list rm_list;
7619397044315f Geliang Tang  2021-02-12  676  	u8 i, align;
5cb104ae55e9db Geliang Tang  2020-09-24  677  
5cb104ae55e9db Geliang Tang  2020-09-24  678  	if (!mptcp_pm_should_rm_signal(msk) ||
7619397044315f Geliang Tang  2021-02-12  679  	    !(mptcp_pm_rm_addr_signal(msk, remaining, &rm_list)))
5cb104ae55e9db Geliang Tang  2020-09-24  680  		return false;
5cb104ae55e9db Geliang Tang  2020-09-24  681  
7619397044315f Geliang Tang  2021-02-12  682  	if (rm_list.nr > 1)
7619397044315f Geliang Tang  2021-02-12  683  		align = 5;
7619397044315f Geliang Tang  2021-02-12  684  	if (rm_list.nr > 5)
7619397044315f Geliang Tang  2021-02-12  685  		align = 9;

Can rm_list.nr be zero?

Smatch will hopefully know the answer if you have the cross function DB
built, but hopefully eventually we will turn these warnings on for GCC
again.

7619397044315f Geliang Tang  2021-02-12  686  
7619397044315f Geliang Tang  2021-02-12 @687  	if (remaining < TCPOLEN_MPTCP_RM_ADDR_BASE + align)
5cb104ae55e9db Geliang Tang  2020-09-24  688  		return false;
5cb104ae55e9db Geliang Tang  2020-09-24  689  
7619397044315f Geliang Tang  2021-02-12  690  	*size = TCPOLEN_MPTCP_RM_ADDR_BASE + align;
5cb104ae55e9db Geliang Tang  2020-09-24  691  	opts->suboptions |= OPTION_MPTCP_RM_ADDR;
7619397044315f Geliang Tang  2021-02-12  692  	opts->rm_list = rm_list;
5cb104ae55e9db Geliang Tang  2020-09-24  693  
7619397044315f Geliang Tang  2021-02-12  694  	for (i = 0; i < opts->rm_list.nr; i++)
7619397044315f Geliang Tang  2021-02-12  695  		pr_debug("rm_list_ids[%d]=%d", i, opts->rm_list.ids[i]);
5cb104ae55e9db Geliang Tang  2020-09-24  696  
5cb104ae55e9db Geliang Tang  2020-09-24  697  	return true;
5cb104ae55e9db Geliang Tang  2020-09-24  698  }

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 37196 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [mptcp:export 1339/1352] net/mptcp/options.c:687 mptcp_established_options_rm_addr() error: uninitialized symbol 'align'.
@ 2021-02-15 12:21 ` Dan Carpenter
  0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-02-15 12:21 UTC (permalink / raw)
  To: kbuild

[-- Attachment #1: Type: text/plain, Size: 3232 bytes --]

tree:   https://github.com/multipath-tcp/mptcp_net-next.git export
head:   d15532665bd1e4a0e210110b275658ba4fcb5b87
commit: 7619397044315f5289200ea122ce4aac316d30da [1339/1352] mptcp: add rm_list in mptcp_out_options
config: i386-randconfig-m021-20210209 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

smatch warnings:
net/mptcp/options.c:687 mptcp_established_options_rm_addr() error: uninitialized symbol 'align'.

vim +/align +687 net/mptcp/options.c

5cb104ae55e9db Geliang Tang  2020-09-24  668  static bool mptcp_established_options_rm_addr(struct sock *sk,
5cb104ae55e9db Geliang Tang  2020-09-24  669  					      unsigned int *size,
5cb104ae55e9db Geliang Tang  2020-09-24  670  					      unsigned int remaining,
5cb104ae55e9db Geliang Tang  2020-09-24  671  					      struct mptcp_out_options *opts)
5cb104ae55e9db Geliang Tang  2020-09-24  672  {
5cb104ae55e9db Geliang Tang  2020-09-24  673  	struct mptcp_subflow_context *subflow = mptcp_subflow_ctx(sk);
5cb104ae55e9db Geliang Tang  2020-09-24  674  	struct mptcp_sock *msk = mptcp_sk(subflow->conn);
7619397044315f Geliang Tang  2021-02-12  675  	struct mptcp_rm_list rm_list;
7619397044315f Geliang Tang  2021-02-12  676  	u8 i, align;
5cb104ae55e9db Geliang Tang  2020-09-24  677  
5cb104ae55e9db Geliang Tang  2020-09-24  678  	if (!mptcp_pm_should_rm_signal(msk) ||
7619397044315f Geliang Tang  2021-02-12  679  	    !(mptcp_pm_rm_addr_signal(msk, remaining, &rm_list)))
5cb104ae55e9db Geliang Tang  2020-09-24  680  		return false;
5cb104ae55e9db Geliang Tang  2020-09-24  681  
7619397044315f Geliang Tang  2021-02-12  682  	if (rm_list.nr > 1)
7619397044315f Geliang Tang  2021-02-12  683  		align = 5;
7619397044315f Geliang Tang  2021-02-12  684  	if (rm_list.nr > 5)
7619397044315f Geliang Tang  2021-02-12  685  		align = 9;

Can rm_list.nr be zero?

Smatch will hopefully know the answer if you have the cross function DB
built, but hopefully eventually we will turn these warnings on for GCC
again.

7619397044315f Geliang Tang  2021-02-12  686  
7619397044315f Geliang Tang  2021-02-12 @687  	if (remaining < TCPOLEN_MPTCP_RM_ADDR_BASE + align)
5cb104ae55e9db Geliang Tang  2020-09-24  688  		return false;
5cb104ae55e9db Geliang Tang  2020-09-24  689  
7619397044315f Geliang Tang  2021-02-12  690  	*size = TCPOLEN_MPTCP_RM_ADDR_BASE + align;
5cb104ae55e9db Geliang Tang  2020-09-24  691  	opts->suboptions |= OPTION_MPTCP_RM_ADDR;
7619397044315f Geliang Tang  2021-02-12  692  	opts->rm_list = rm_list;
5cb104ae55e9db Geliang Tang  2020-09-24  693  
7619397044315f Geliang Tang  2021-02-12  694  	for (i = 0; i < opts->rm_list.nr; i++)
7619397044315f Geliang Tang  2021-02-12  695  		pr_debug("rm_list_ids[%d]=%d", i, opts->rm_list.ids[i]);
5cb104ae55e9db Geliang Tang  2020-09-24  696  
5cb104ae55e9db Geliang Tang  2020-09-24  697  	return true;
5cb104ae55e9db Geliang Tang  2020-09-24  698  }

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 37196 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [mptcp:export 1339/1352] net/mptcp/options.c:687 mptcp_established_options_rm_addr() error: uninitialized symbol 'align'.
@ 2021-02-15 12:21 ` Dan Carpenter
  0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-02-15 12:21 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 3232 bytes --]

tree:   https://github.com/multipath-tcp/mptcp_net-next.git export
head:   d15532665bd1e4a0e210110b275658ba4fcb5b87
commit: 7619397044315f5289200ea122ce4aac316d30da [1339/1352] mptcp: add rm_list in mptcp_out_options
config: i386-randconfig-m021-20210209 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

smatch warnings:
net/mptcp/options.c:687 mptcp_established_options_rm_addr() error: uninitialized symbol 'align'.

vim +/align +687 net/mptcp/options.c

5cb104ae55e9db Geliang Tang  2020-09-24  668  static bool mptcp_established_options_rm_addr(struct sock *sk,
5cb104ae55e9db Geliang Tang  2020-09-24  669  					      unsigned int *size,
5cb104ae55e9db Geliang Tang  2020-09-24  670  					      unsigned int remaining,
5cb104ae55e9db Geliang Tang  2020-09-24  671  					      struct mptcp_out_options *opts)
5cb104ae55e9db Geliang Tang  2020-09-24  672  {
5cb104ae55e9db Geliang Tang  2020-09-24  673  	struct mptcp_subflow_context *subflow = mptcp_subflow_ctx(sk);
5cb104ae55e9db Geliang Tang  2020-09-24  674  	struct mptcp_sock *msk = mptcp_sk(subflow->conn);
7619397044315f Geliang Tang  2021-02-12  675  	struct mptcp_rm_list rm_list;
7619397044315f Geliang Tang  2021-02-12  676  	u8 i, align;
5cb104ae55e9db Geliang Tang  2020-09-24  677  
5cb104ae55e9db Geliang Tang  2020-09-24  678  	if (!mptcp_pm_should_rm_signal(msk) ||
7619397044315f Geliang Tang  2021-02-12  679  	    !(mptcp_pm_rm_addr_signal(msk, remaining, &rm_list)))
5cb104ae55e9db Geliang Tang  2020-09-24  680  		return false;
5cb104ae55e9db Geliang Tang  2020-09-24  681  
7619397044315f Geliang Tang  2021-02-12  682  	if (rm_list.nr > 1)
7619397044315f Geliang Tang  2021-02-12  683  		align = 5;
7619397044315f Geliang Tang  2021-02-12  684  	if (rm_list.nr > 5)
7619397044315f Geliang Tang  2021-02-12  685  		align = 9;

Can rm_list.nr be zero?

Smatch will hopefully know the answer if you have the cross function DB
built, but hopefully eventually we will turn these warnings on for GCC
again.

7619397044315f Geliang Tang  2021-02-12  686  
7619397044315f Geliang Tang  2021-02-12 @687  	if (remaining < TCPOLEN_MPTCP_RM_ADDR_BASE + align)
5cb104ae55e9db Geliang Tang  2020-09-24  688  		return false;
5cb104ae55e9db Geliang Tang  2020-09-24  689  
7619397044315f Geliang Tang  2021-02-12  690  	*size = TCPOLEN_MPTCP_RM_ADDR_BASE + align;
5cb104ae55e9db Geliang Tang  2020-09-24  691  	opts->suboptions |= OPTION_MPTCP_RM_ADDR;
7619397044315f Geliang Tang  2021-02-12  692  	opts->rm_list = rm_list;
5cb104ae55e9db Geliang Tang  2020-09-24  693  
7619397044315f Geliang Tang  2021-02-12  694  	for (i = 0; i < opts->rm_list.nr; i++)
7619397044315f Geliang Tang  2021-02-12  695  		pr_debug("rm_list_ids[%d]=%d", i, opts->rm_list.ids[i]);
5cb104ae55e9db Geliang Tang  2020-09-24  696  
5cb104ae55e9db Geliang Tang  2020-09-24  697  	return true;
5cb104ae55e9db Geliang Tang  2020-09-24  698  }

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 37196 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-02-15 12:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-02-15 12:21 [MPTCP] [mptcp:export 1339/1352] net/mptcp/options.c:687 mptcp_established_options_rm_addr() error: uninitialized symbol 'align' Dan Carpenter
2021-02-15 12:21 ` Dan Carpenter
2021-02-15 12:21 ` Dan Carpenter

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.