Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp2163059imu; Sun, 18 Nov 2018 17:26:38 -0800 (PST) X-Google-Smtp-Source: AJdET5evJUemUNxx56nMCBvbcgx/PhcjNPGgyC5jhffjmnAkZl8XW5J+OBv6BCukIqJcI1eVMsgP X-Received: by 2002:a17:902:598e:: with SMTP id p14mr20110638pli.260.1542590798817; Sun, 18 Nov 2018 17:26:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542590798; cv=none; d=google.com; s=arc-20160816; b=Ll443Ak/QxMmzSSA/x049EJe2Kk0cC7KamR94Fro55jxE3jjNG4HqG3mPqGJUu7Psf yyvGMQLQOrYvGyZ4Q68xDO1FcbBg5zGIh/s+KfGVK5T5uWWlYCT6bTX1B4Rb/bAchHEZ 3V2uIC4ZwSq8PDpMQZdHCFlmImQohjCQg2X+cM8QGz5PfC5K+iJAQV575jYxqOwb7aKk h7ZzxcZCuAt2ulm6YBUfpmJKKH23weoQPZ+aOJ6w/uzZbWx0FhiL+L8lNf+tT5n8I3fS lQHhzEUHEvB8J8LY/Fw3mJOjMSPyePyqbnI9XyClPouFshIKDsgEJuIAdkfKkNkD41xV R85w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:in-reply-to :mime-version:user-agent:date:message-id:from:cc:references:to :subject; bh=zzvpUO9v4WCi2jAUspAZnBJVvSUcTwg2d9mDt3BKnZs=; b=fJnjc2COzt4/F5qkyPwcMyyjLzVvvKanTuPgOQ29FpoPTf7Tf69sq+ERZLLf/QFzRN Mxjk81ucdT3vyYLKCwJSPDHE9V29leJoOuO9WtxwK2xizODtn9mCdcKxF0x5cT+sr18o /VNC2/IgSwxYKPtj3tL9CbnchhsB9pE/V8oOUFvV7G6kgAgQXMxngFmCB/RtsrH5Zbhd Zac2yA9mXchIYxzlduw8GYjkQ5NaeCON+pg+CB33nUKw+LKHLk8BTE5XRuvJOa8inkhL qjsihXCAu22E/ugK29eO17AFMzsACQlMP7eXhL1qLoN/SJY5TqtFV6H4jTPjkXddm9V2 xWIQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id n33si35383899pgl.336.2018.11.18.17.26.24; Sun, 18 Nov 2018 17:26:38 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727811AbeKSLqC (ORCPT + 99 others); Mon, 19 Nov 2018 06:46:02 -0500 Received: from szxga05-in.huawei.com ([45.249.212.191]:14682 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727237AbeKSLqC (ORCPT ); Mon, 19 Nov 2018 06:46:02 -0500 Received: from DGGEMS414-HUB.china.huawei.com (unknown [172.30.72.60]) by Forcepoint Email with ESMTP id 3CA036B35A136; Mon, 19 Nov 2018 09:24:02 +0800 (CST) Received: from [127.0.0.1] (10.177.31.96) by DGGEMS414-HUB.china.huawei.com (10.3.19.214) with Microsoft SMTP Server id 14.3.408.0; Mon, 19 Nov 2018 09:23:58 +0800 Subject: Re: [PATCH -next] video: fbdev: pxafb: Fix "WARNING: invalid free of devm_ allocated data" To: Daniel Mack , , References: <20181116110255.19736-1-yuehaibing@huawei.com> CC: , , From: YueHaibing Message-ID: Date: Mon, 19 Nov 2018 09:23:57 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.177.31.96] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2018/11/17 4:05, Daniel Mack wrote: > On 16/11/2018 12:02 PM, YueHaibing wrote: >> 'info->modes' got allocated with devm_kcalloc in of_get_pxafb_display. >> >> This gives this error message: >> ./drivers/video/fbdev/pxafb.c:2238:2-7: WARNING: invalid free of devm_ allocated data >> >> Signed-off-by: YueHaibing > > Oh, yes. Thanks for spotting this! > > Could you please add these lines to the patch and resubmit? > Sure, will send v2. > > Fixes: c8f96304ec8b4 ("video: fbdev: pxafb: switch to devm_* API") > Cc: stable@kernel.org [v4.19+] > Reviewed-by: Daniel Mack > > > > >> --- >> drivers/video/fbdev/pxafb.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/drivers/video/fbdev/pxafb.c b/drivers/video/fbdev/pxafb.c >> index bbed039..d59c8a5 100644 >> --- a/drivers/video/fbdev/pxafb.c >> +++ b/drivers/video/fbdev/pxafb.c >> @@ -2234,10 +2234,8 @@ static struct pxafb_mach_info *of_pxafb_of_mach_info(struct device *dev) >> if (!info) >> return ERR_PTR(-ENOMEM); >> ret = of_get_pxafb_mode_info(dev, info); >> - if (ret) { >> - kfree(info->modes); >> + if (ret) >> return ERR_PTR(ret); >> - } >> /* >> * On purpose, neither lccrX registers nor video memory size can be >> > > > . >