mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-10-06 00:39:48 +00:00
drm/of: free the iterator object on failure
When bailing out due to the sanity check the iterator value needs to be
freed because the early return prevents for_each_child_of_node() from
doing the dereference itself.
Fixes: 6529007522
("drm: of: Add drm_of_lvds_get_dual_link_pixel_order")
Signed-off-by: Steven Price <steven.price@arm.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210714143300.20632-1-steven.price@arm.com
This commit is contained in:
parent
311ac6f3e5
commit
6f9223a56f
1 changed files with 3 additions and 1 deletions
|
@ -331,9 +331,11 @@ static int drm_of_lvds_get_remote_pixels_type(
|
||||||
* configurations by passing the endpoints explicitly to
|
* configurations by passing the endpoints explicitly to
|
||||||
* drm_of_lvds_get_dual_link_pixel_order().
|
* drm_of_lvds_get_dual_link_pixel_order().
|
||||||
*/
|
*/
|
||||||
if (!current_pt || pixels_type != current_pt)
|
if (!current_pt || pixels_type != current_pt) {
|
||||||
|
of_node_put(endpoint);
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
return pixels_type;
|
return pixels_type;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue