Skip to contentSkip to conversation

usb: typec: Port mapping utility

bug report Dan Carpenter 2 participants 2 messages Apr 13, 2021 – Apr 14, 2021 USB

lore
Share & tools
lore
#0 lore raw

Hello Heikki Krogerus,

The patch ae196ddb0d31: "usb: typec: Port mapping utility" from Apr 7, 2021, leads to the following static checker warning:

drivers/usb/typec/port-mapper.c:168 typec_link_port()
warn: missing error code 'ret'

drivers/usb/typec/port-mapper.c
156 int typec_link_port(struct device *port)
157 {

   158          struct device *connector;
   159          struct port_node *node;
   160          int ret = 0;

161
162 node = create_port_node(port);
163 if (IS_ERR(node))
164 return PTR_ERR(node);
165
166 connector = find_connector(node);
167 if (!connector)
168 goto remove_node;

Set error code?

   169  
   170          ret = link_port(to_typec_port(connector), node);
   171          if (ret)
   172                  goto put_connector;
   173  
   174          return 0;
   175  
   176  put_connector:
   177          put_device(connector);
   178  remove_node:
   179          remove_port_node(node);
   180  
   181          return ret;
   182  }

regards,
dan carpenter

#1 lore raw
Quoted 23 lines · Dan Carpenter

Hello Heikki Krogerus,

The patch ae196ddb0d31: "usb: typec: Port mapping utility" from Apr 7, 2021, leads to the following static checker warning:

drivers/usb/typec/port-mapper.c:168 typec_link_port()
warn: missing error code 'ret'

drivers/usb/typec/port-mapper.c
156 int typec_link_port(struct device *port)

   157  {
   158          struct device *connector;
   159          struct port_node *node;
   160          int ret = 0;
   161  
   162          node = create_port_node(port);
   163          if (IS_ERR(node))
   164                  return PTR_ERR(node);
   165  
   166          connector = find_connector(node);
   167          if (!connector)
   168                  goto remove_node;
Quoted 1 line

Set error code?

False positive in this case. Not an error.

thanks,

Quoted 14 lines
   169  
   170          ret = link_port(to_typec_port(connector), node);
   171          if (ret)
   172                  goto put_connector;
   173  
   174          return 0;
   175  
   176  put_connector:
   177          put_device(connector);
   178  remove_node:
   179          remove_port_node(node);
   180  
   181          return ret;
   182  }
Signature · 1 line
heikki