Thread (3 messages) 3 messages, 3 authors, 2021-03-02

Re: [PATCH v3] usb: typec: tcpm: Export partner Source Capabilities

From: Dan Carpenter <hidden>
Date: 2021-02-15 17:42:58
Also in: lkml, oe-kbuild, oe-kbuild-all

Hi Kyle,

url:    https://github.com/0day-ci/linux/commits/Kyle-Tso/usb-typec-tcpm-Export-partner-Source-Capabilities/20210214-192125
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: i386-randconfig-m021-20210214 (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 <redacted>
Reported-by: Dan Carpenter <redacted>

smatch warnings:
drivers/usb/typec/tcpm/tcpm.c:5770 tcpm_get_partner_src_caps() warn: variable dereferenced before check 'src_pdo' (see line 5769)

vim +/src_pdo +5770 drivers/usb/typec/tcpm/tcpm.c

bfd31a180a3d55 Kyle Tso 2021-02-14  5759  int tcpm_get_partner_src_caps(struct tcpm_port *port, u32 **src_pdo)
bfd31a180a3d55 Kyle Tso 2021-02-14  5760  {
bfd31a180a3d55 Kyle Tso 2021-02-14  5761  	unsigned int nr_pdo;
bfd31a180a3d55 Kyle Tso 2021-02-14  5762  
bfd31a180a3d55 Kyle Tso 2021-02-14  5763  	mutex_lock(&port->lock);
bfd31a180a3d55 Kyle Tso 2021-02-14  5764  	if (port->nr_source_caps == 0) {
bfd31a180a3d55 Kyle Tso 2021-02-14  5765  		mutex_unlock(&port->lock);
bfd31a180a3d55 Kyle Tso 2021-02-14  5766  		return -ENODATA;
bfd31a180a3d55 Kyle Tso 2021-02-14  5767  	}
bfd31a180a3d55 Kyle Tso 2021-02-14  5768  
bfd31a180a3d55 Kyle Tso 2021-02-14 @5769  	*src_pdo = kcalloc(port->nr_source_caps, sizeof(u32), GFP_KERNEL);
bfd31a180a3d55 Kyle Tso 2021-02-14 @5770  	if (!src_pdo) {

Typo.  Missing * char.

bfd31a180a3d55 Kyle Tso 2021-02-14  5771  		mutex_unlock(&port->lock);
bfd31a180a3d55 Kyle Tso 2021-02-14  5772  		return -ENOMEM;
bfd31a180a3d55 Kyle Tso 2021-02-14  5773  	}
bfd31a180a3d55 Kyle Tso 2021-02-14  5774  
bfd31a180a3d55 Kyle Tso 2021-02-14  5775  	nr_pdo = tcpm_copy_pdos(*src_pdo, port->source_caps,
bfd31a180a3d55 Kyle Tso 2021-02-14  5776  				port->nr_source_caps);
bfd31a180a3d55 Kyle Tso 2021-02-14  5777  	mutex_unlock(&port->lock);
bfd31a180a3d55 Kyle Tso 2021-02-14  5778  	return nr_pdo;
bfd31a180a3d55 Kyle Tso 2021-02-14  5779  }

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

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help