| Message ID | 20250117-qcom-ice-fix-dev-leak-v2-3-1ffa5b6884cb@linaro.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | soc: qcom: ice: fix dev reference leaked through of_qcom_ice_get | expand |
On 25-01-17 14:18:52, Tudor Ambarus wrote: > The driver leaks the device reference taken with > of_find_device_by_node(). Fix the leak by using devm_of_qcom_ice_get(). > > Fixes: 56541c7c4468 ("scsi: ufs: ufs-qcom: Switch to the new ICE API") > Cc: stable@vger.kernel.org > Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > --- > drivers/ufs/host/ufs-qcom.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/ufs/host/ufs-qcom.c b/drivers/ufs/host/ufs-qcom.c > index 23b9f6efa047..a455a95f65fc 100644 > --- a/drivers/ufs/host/ufs-qcom.c > +++ b/drivers/ufs/host/ufs-qcom.c > @@ -125,7 +125,7 @@ static int ufs_qcom_ice_init(struct ufs_qcom_host *host) > int err; > int i; > > - ice = of_qcom_ice_get(dev); > + ice = devm_of_qcom_ice_get(dev); > if (ice == ERR_PTR(-EOPNOTSUPP)) { > dev_warn(dev, "Disabling inline encryption support\n"); > ice = NULL; > > -- > 2.48.0.rc2.279.g1de40edade-goog >
diff --git a/drivers/ufs/host/ufs-qcom.c b/drivers/ufs/host/ufs-qcom.c index 23b9f6efa047..a455a95f65fc 100644 --- a/drivers/ufs/host/ufs-qcom.c +++ b/drivers/ufs/host/ufs-qcom.c @@ -125,7 +125,7 @@ static int ufs_qcom_ice_init(struct ufs_qcom_host *host) int err; int i; - ice = of_qcom_ice_get(dev); + ice = devm_of_qcom_ice_get(dev); if (ice == ERR_PTR(-EOPNOTSUPP)) { dev_warn(dev, "Disabling inline encryption support\n"); ice = NULL;
The driver leaks the device reference taken with of_find_device_by_node(). Fix the leak by using devm_of_qcom_ice_get(). Fixes: 56541c7c4468 ("scsi: ufs: ufs-qcom: Switch to the new ICE API") Cc: stable@vger.kernel.org Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- drivers/ufs/host/ufs-qcom.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)