usb: typec: Port mapping utility
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
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:168typec_link_port()
warn: missing error code 'ret'
drivers/usb/typec/port-mapper.c
156 inttypec_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