public inbox for linux-mmc@vger.kernel.org
 help / color / mirror / Atom feed
* re: DaVinci: MMC: MMC/SD controller driver for DaVinci family
@ 2013-11-06 16:15 Dan Carpenter
  0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2013-11-06 16:15 UTC (permalink / raw)
  To: vipin.bhandari; +Cc: linux-mmc

Hello Vipin Bhandari,

This is a semi-automatic email about new static checker warnings.

The patch b4cff4549b7a: "DaVinci: MMC: MMC/SD controller driver for 
DaVinci family" from Dec 14, 2009, leads to the following Smatch 
complaint:

drivers/mmc/host/davinci_mmc.c:984 mmc_davinci_irq()
	 error: we previously assumed 'data' could be null (see line 967)

drivers/mmc/host/davinci_mmc.c
   966			/* All blocks sent/received, and CRC checks passed */
   967			if (data != NULL) {
                            ^^^^^^^^^^^^
Check.

   968				if ((host->do_dma == 0) && (host->bytes_left > 0)) {
   969					/* if datasize < rw_threshold
   970					 * no RX ints are generated
   971					 */
   972					davinci_fifo_data_trans(host, host->bytes_left);
   973				}
   974				end_transfer = 1;
   975				data->bytes_xfered = data->blocks * data->blksz;
   976			} else {
   977				dev_err(mmc_dev(host->mmc),
   978						"DATDNE with no host->data\n");
   979			}
   980		}
   981	
   982		if (qstatus & MMCST0_TOUTRD) {
   983			/* Read data timeout */
   984			data->error = -ETIMEDOUT;
                        ^^^^^^^^^^^^^^^^^^^^^^^^
Unchecked dereference.

   985			end_transfer = 1;
   986	

regards,
dan carpenter

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2013-11-06 16:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-06 16:15 DaVinci: MMC: MMC/SD controller driver for DaVinci family Dan Carpenter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox