[bug report] [media] davinci: vpif_capture: get subdevs from DT when available

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

 



Hello Kevin Hilman,

The patch 4a5f8ae50b66: "[media] davinci: vpif_capture: get subdevs
from DT when available" from Jun 6, 2017, leads to the following
static checker warning:

	drivers/media/platform/davinci/vpif_capture.c:1596 vpif_capture_get_pdata()
	error: potential NULL dereference 'pdata'.

drivers/media/platform/davinci/vpif_capture.c
  1576  
  1577                  dev_dbg(&pdev->dev, "Remote device %s, %s found\n",
  1578                          rem->name, rem->full_name);
  1579                  sdinfo->name = rem->full_name;
  1580  
  1581                  pdata->asd[i] = devm_kzalloc(&pdev->dev,
  1582                                               sizeof(struct v4l2_async_subdev),
  1583                                               GFP_KERNEL);
  1584                  if (!pdata->asd[i]) {
  1585                          of_node_put(rem);
  1586                          pdata = NULL;
                                ^^^^^^^^^^^^
Set to NULL

  1587                          goto done;
  1588                  }
  1589  
  1590                  pdata->asd[i]->match_type = V4L2_ASYNC_MATCH_FWNODE;
  1591                  pdata->asd[i]->match.fwnode.fwnode = of_fwnode_handle(rem);
  1592                  of_node_put(rem);
  1593          }
  1594  
  1595  done:
  1596          pdata->asd_sizes[0] = i;
                ^^^^^^^^^^^^^^^^
Dereference.

  1597          pdata->subdev_count = i;
  1598          pdata->card_name = "DA850/OMAP-L138 Video Capture";
  1599  
  1600          return pdata;
  1601  }

regards,
dan carpenter



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux