rpmsg: glink: Release driver_override
[ Upstream commitfb80ef67e8
] Upon termination of the rpmsg_device, driver_override needs to be freed to avoid leaking the potentially assigned string. Fixes:42cd402b8f
("rpmsg: Fix kfree() of static memory on setting driver_override") Fixes:39e47767ec
("rpmsg: Add driver_override device attribute for rpmsg_device") Reviewed-by: Chris Lew <quic_clew@quicinc.com> Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com> Signed-off-by: Bjorn Andersson <andersson@kernel.org> Link: https://lore.kernel.org/r/20230109223931.1706429-1-quic_bjorande@quicinc.com Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
09699f0e5b
commit
4a46f52ae8
|
@ -1447,6 +1447,7 @@ static void qcom_glink_rpdev_release(struct device *dev)
|
|||
{
|
||||
struct rpmsg_device *rpdev = to_rpmsg_device(dev);
|
||||
|
||||
kfree(rpdev->driver_override);
|
||||
kfree(rpdev);
|
||||
}
|
||||
|
||||
|
@ -1690,6 +1691,7 @@ static void qcom_glink_device_release(struct device *dev)
|
|||
|
||||
/* Release qcom_glink_alloc_channel() reference */
|
||||
kref_put(&channel->refcount, qcom_glink_channel_release);
|
||||
kfree(rpdev->driver_override);
|
||||
kfree(rpdev);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue