re: drm/tilcdc: Add support for external tda998x encoder

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

 



Hello Jyri Sarha,

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

The patch 103cd8bc1c80: "drm/tilcdc: Add support for external tda998x
encoder" from Feb 10, 2015, leads to the following Smatch complaint:

drivers/gpu/drm/tilcdc/tilcdc_external.c:153 tilcdc_get_external_components()
	 error: we previously assumed 'node' could be null (see line 148)

drivers/gpu/drm/tilcdc/tilcdc_external.c
   147			node = of_graph_get_remote_port_parent(ep);
   148			if (!node && !of_device_is_available(node)) {
                             ^^^^

This test is probably reversed?

   149				of_node_put(node);
                                            ^^^^
NULL.

   150				continue;
   151			}
   152	
   153			dev_dbg(dev, "Subdevice node '%s' found\n", node->name);
                                                                    ^^^^^^^^^^^
Unchecked dereference.

   154			if (match)
   155				component_match_add(dev, match, dev_match_of, node);

regards,
dan carpenter
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux