re: mmc: rtsx: add support for pre_req and post_req

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Micky Ching,

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

The patch c42deffd5b53: "mmc: rtsx: add support for pre_req and 
post_req" from Feb 17, 2014, leads to the following Smatch complaint:

drivers/mmc/host/rtsx_pci_sdmmc.c:504 sd_get_rsp()
	 error: we previously assumed 'cmd' could be null (see line 434)

drivers/mmc/host/rtsx_pci_sdmmc.c
   433	
   434		if (!cmd) {
                     ^^^
Patch introduces check.

   435			dev_err(sdmmc_dev(host), "error: cmd not exist\n");
   436			goto out;
   437		}
   438	

[ snip ]

   497		if (cmd->data) {
   498			sd_start_multi_rw(host, host->mrq);
   499			spin_unlock_irqrestore(&host->lock, flags);
   500			return;
   501		}
   502	
   503	out:
   504		cmd->error = err;
                ^^^^^^^^^^
Dereference was already there.

   505	
   506		tasklet_schedule(&host->finish_tasklet);

regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux