Thread (1 message) 1 message, 1 author, 2023-06-13

Re: [cocci] [PATCH] pinctrl: nxp: Adjust error predicates for improved exception handling in s32_dt_node_to_map()?

From: Markus Elfring <hidden>
Date: 2023-06-13 09:16:09

quoted
quoted
Ensure child node references are decremented properly in the error path.
quoted
+++ b/drivers/pinctrl/nxp/pinctrl-s32cc.c
@@ -279,8 +279,10 @@ static int s32_dt_node_to_map(struct pinctrl_dev *pctldev,
 		ret = s32_dt_group_node_to_map(pctldev, np, map,
 					       &reserved_maps, num_maps,
 					       np_config->name);
-		if (ret < 0)
+		if (ret < 0) {
+			of_node_put(np);
 			break;
+		}
 	}

 	if (ret)
May the same error predicate be applied at two places here?
I think these two error predicates can be combined together in another patch.
Are you going to reduce such error predicates so that exception handling
can be improved another bit?

Regards,
Markus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help