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

Reply via email to