From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754542AbdASUJg (ORCPT ); Thu, 19 Jan 2017 15:09:36 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:52076 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751757AbdASUJf (ORCPT ); Thu, 19 Jan 2017 15:09:35 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 4A81460819 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=vivek.gautam@codeaurora.org MIME-Version: 1.0 In-Reply-To: <20170119104739.4376-1-bjorn.andersson@linaro.org> References: <20170119104739.4376-1-bjorn.andersson@linaro.org> From: Vivek Gautam Date: Fri, 20 Jan 2017 01:39:32 +0530 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 1/4] phy: qcom-ufs: Don't kfree devres resource To: Bjorn Andersson Cc: Kishon Vijay Abraham I , "linux-kernel@vger.kernel.org" , linux-arm-msm@vger.kernel.org, Subhash Jadavani Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jan 19, 2017 at 4:17 PM, Bjorn Andersson wrote: > Upon failing to acquire regulator supplies the qcom-ufs driver calls > kfree() on the devm allocated memory used to store the name of the > regulator, leading to devres corruption. > > Rather than switching to using the appropriate free function the patch > acknowledge the fact that "name" is always a constant string and we > don't actually need to create a local copy of it, but rather just > reference the constant string. > > Cc: Subhash Jadavani > Cc: Vivek Gautam > Signed-off-by: Bjorn Andersson > --- This patch fixes: add78fc05702 phy: qcom-ufs: Use devm sibling of kstrdup for regulator names Regards Vivek > drivers/phy/phy-qcom-ufs.c | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) > > diff --git a/drivers/phy/phy-qcom-ufs.c b/drivers/phy/phy-qcom-ufs.c > index c69568b8543d..4d7f3c018223 100644 > --- a/drivers/phy/phy-qcom-ufs.c > +++ b/drivers/phy/phy-qcom-ufs.c > @@ -217,12 +217,7 @@ static int __ufs_qcom_phy_init_vreg(struct device *dev, > > char prop_name[MAX_PROP_NAME]; > > - vreg->name = devm_kstrdup(dev, name, GFP_KERNEL); > - if (!vreg->name) { > - err = -ENOMEM; > - goto out; > - } > - > + vreg->name = name; > vreg->reg = devm_regulator_get(dev, name); > if (IS_ERR(vreg->reg)) { > err = PTR_ERR(vreg->reg); > @@ -265,8 +260,6 @@ static int __ufs_qcom_phy_init_vreg(struct device *dev, > } > > out: > - if (err) > - kfree(vreg->name); > return err; > } > > -- > 2.11.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project