Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp899679imu; Fri, 16 Nov 2018 12:06:27 -0800 (PST) X-Google-Smtp-Source: AJdET5fmgutdKrpj0W3ltpiVJSGp8iJn68+swn814jiTb2s0iR+mV9CjTcF6PQn+rlqfbVXxMoX/ X-Received: by 2002:a63:66c6:: with SMTP id a189mr11255563pgc.167.1542398787668; Fri, 16 Nov 2018 12:06:27 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542398787; cv=none; d=google.com; s=arc-20160816; b=pefb1D6YTabDMF2SVsuFdoOgm4qLzF+YUxk5tsi7JjV9H/aLjdyO9PIThY2tM1gbCD PkEsCDFsIdcb6DKyL5pcwgtAbh49Y60bKpFwR13FRCW4UMbVC/AbBS1P4Y06H2gLx7rI QaneIBDsfiQsfd7mWX2mAE1zxqqSSJ8OfhQdZ4ztVwKfCAS4JHXKG05eqY7qnwAXtXmG 2ii+HlzgBLEF+vmLlAS/T7YGecXAPdDDmA+HkCAS4UU68VQGeWwLlYHSzKtckeHIded7 x9kVyxLTajvSCkSFW2sZf17FK9pWXM3wQCvBYU5qBmLuXzo8kaCNFWUzRW9fkKQK2IyW BMBQ== 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 :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=Bzaqy/7g4HAQAFKGWXlJFhR0ZKsD4CS5EdJIQEnNAF4=; b=y6zr7ZJDUQg2i0INXeYvoqUv75wycDH+slxY1CmvdrRNxoq5Ta9vxvEZPeReWl0gK6 FtT0Xvgy6jyCScId0lzDaYKqwxc0LH2ueoP8zkW3MByFTqgMAiz5B9Ow4NFxXGuz3TiS txbWmqzA+JA40zFOHGwvCOtNtB3uiNA59ofxilMeYaA1glJ7tNg6XBjOjd7W3XosbCEt A5G3jSLIEPvR5b5ETgS9isdN5YnnazEpGN56hOxa8YZo/2iZsZMvTybBUQGZTId0mFBp NwrSB5/h1Fr2pN1p0chBYHRE9nK8+EmyVMeBp6Ke0ylX7cZSGPkhpHGwaju8SXVqrNNq AwUA== 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 g22si17311873pfj.222.2018.11.16.12.06.12; Fri, 16 Nov 2018 12:06:27 -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 S1727121AbeKQGTW (ORCPT + 99 others); Sat, 17 Nov 2018 01:19:22 -0500 Received: from mail.bugwerft.de ([46.23.86.59]:59468 "EHLO mail.bugwerft.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725763AbeKQGTV (ORCPT ); Sat, 17 Nov 2018 01:19:21 -0500 Received: from [192.168.178.170] (pD95EF84F.dip0.t-ipconnect.de [217.94.248.79]) by mail.bugwerft.de (Postfix) with ESMTPSA id DEB512C0BE0; Fri, 16 Nov 2018 20:05:31 +0000 (UTC) Subject: Re: [PATCH -next] video: fbdev: pxafb: Fix "WARNING: invalid free of devm_ allocated data" To: YueHaibing , b.zolnierkie@samsung.com, robert.jarzmik@free.fr Cc: linux-kernel@vger.kernel.org, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org References: <20181116110255.19736-1-yuehaibing@huawei.com> From: Daniel Mack Message-ID: Date: Fri, 16 Nov 2018 21:05:32 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.0 MIME-Version: 1.0 In-Reply-To: <20181116110255.19736-1-yuehaibing@huawei.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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? 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 >