Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp409331imu; Tue, 27 Nov 2018 00:11:38 -0800 (PST) X-Google-Smtp-Source: AFSGD/WWu5cLrELLfLY804BCtEekPITQIKXLqbuFCGbbhPWmBxK/mS54eblhb+sfpoDHh2nFqa4F X-Received: by 2002:a63:9041:: with SMTP id a62mr27748997pge.163.1543306298308; Tue, 27 Nov 2018 00:11:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543306298; cv=none; d=google.com; s=arc-20160816; b=Yu3TfnDgumtTxgcUjxdr+VqpZPRzNwrVk0X9Mpmn6zkO7jI2pS+NfAB8psDq+8IDJU htBhIhLVib3lIgCxeYtxAt5n9XKHlXD2iFOX0SV3wR6z0QUA+Mcvn9omOzK2C1NPKnJs GnQcVDCGI6G9KayuXNVb0J83qp4xlFvNTszfQiO+ra4Zb3TyeiIahK0BuVJg1EtDcUs8 9N5vwJv7blfhWTaUhbg1yHwKc8j2ht1cO57HefJJznrUGAydlgrvc7WXT9Ty3IfvKF+d 5ACTXyhbO8cfZmPD7H0TFQd+ay8AcoaQpDkEGdq91QJZCqTVRMlgSkTilG0S+135Dhnu ypIQ== 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:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=nnFqodSs11OexaPM/i330k3eX94BAD4qrPcsfeSOmBY=; b=GTVzKaZYbIfLDrM9gtAA0HnA40A7wDGu5uZLy+/vDD8yiP/VQ83FxxWfPlfDs5P8HO fNLpf8tyD18Vhq12hxZvbdBsrYxHtffGdjEW+6bpDqkIVutUj6ET5inzattZLOXnHCmj bp/umiVQtdOI131yaAfwTjZsugn0CzZN9J1sD0uDkHR9fHhPlQWjI85lHTnf2so5wXML CCu5tZdDiI83cDAHY1K5HquZ2e/K6X6vnKnYpZuUXuFMMbLABNz8YOebcGNNzbhJHJzz JzIoOLOGVuIN0sJqd4jnFP7XPACOP/jwM9sP6VE+wU0bNHqgrDXyps3XkX1qqkvVM+8Z BpYQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=fNctyP5A; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x5-v6si3203788pfx.74.2018.11.27.00.11.23; Tue, 27 Nov 2018 00:11: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; dkim=pass header.i=@kernel.org header.s=default header.b=fNctyP5A; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729064AbeK0SSC (ORCPT + 99 others); Tue, 27 Nov 2018 13:18:02 -0500 Received: from mail.kernel.org ([198.145.29.99]:53060 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727745AbeK0SSC (ORCPT ); Tue, 27 Nov 2018 13:18:02 -0500 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 3C48921104; Tue, 27 Nov 2018 07:21:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1543303265; bh=1F28aIS+1vD0g4RixfeDhO3ZV6pSBzptwIjaQ3mb/zg=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=fNctyP5AYxtKvELH7vzxqPv1zGyygcVCCatFiOMflsnhv/0BWH5vDsmwamh006nU+ xR1DGdfbTeWDt8+Cb7IWC/GzVMM/7utIDHvlm93VNmpdw1Z8DLSIc2tryjWAP+Ra5k uFI+rDYQF6p/cZwICWp2ZVJdNe5FrWpqLpLnyvUU= Received: by mail-wm1-f52.google.com with SMTP id 125so20718175wmh.0; Mon, 26 Nov 2018 23:21:05 -0800 (PST) X-Gm-Message-State: AA+aEWaLmxbJKKeMzOa3eF9lFGMGGzrNJsSZ0MdQmypLfejl+rm2OD0A x1E+BfDOgmFL04hB/DKl/URkLkt7Wsi6oJlnv0s= X-Received: by 2002:a1c:2c56:: with SMTP id s83mr740345wms.48.1543303263662; Mon, 26 Nov 2018 23:21:03 -0800 (PST) MIME-Version: 1.0 References: <1542823573-20228-1-git-send-email-hofrat@osadl.org> In-Reply-To: <1542823573-20228-1-git-send-email-hofrat@osadl.org> From: Sean Wang Date: Mon, 26 Nov 2018 23:20:53 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 1/2] gpio: mt7621: report failure of devm_kasprintf() To: hofrat@osadl.org Cc: Linus Walleij , sergio.paracuellos@gmail.com, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, bgolaszewski@baylibre.com, linux-mediatek@lists.infradead.org, Matthias Brugger , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Nicholas Mc Guire =E6=96=BC 2018=E5=B9=B411=E6=9C=8821= =E6=97=A5 =E9=80=B1=E4=B8=89 =E4=B8=8A=E5=8D=8810:13=E5=AF=AB=E9=81=93=EF= =BC=9A > > kasprintf() may return NULL on failure of internal allocation thus the > assigned label is not safe if not explicitly checked. On error > mediatek_gpio_bank_probe() returns negative values so -ENOMEM in the > (unlikely) failure case should be fine here. > > Signed-off-by: Nicholas Mc Guire Acked-by: Sean Wang > Fixes: 4ba9c3afda41 ("gpio: mt7621: Add a driver for MT7621") > --- > > V2: The dev_err() for the unlikely case of the devm_kasprintf() > failing was removed on request from Bartosz Golaszewski > > > Problem located with experimental coccinelle script > > Patch was compile tested with: omega2p_defconfig, SOC_MT7621=3Dy, > GPIOLIB=3Dy, GPIO_MT7621=3Dy > > Patch is against 4.20-rc3 (localversion-next is next-20181121) > > drivers/gpio/gpio-mt7621.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpio/gpio-mt7621.c b/drivers/gpio/gpio-mt7621.c > index d72af6f..1ec95bc 100644 > --- a/drivers/gpio/gpio-mt7621.c > +++ b/drivers/gpio/gpio-mt7621.c > @@ -244,6 +244,8 @@ mediatek_gpio_bank_probe(struct device *dev, > rg->chip.of_xlate =3D mediatek_gpio_xlate; > rg->chip.label =3D devm_kasprintf(dev, GFP_KERNEL, "%s-bank%d", > dev_name(dev), bank); > + if (!rg->chip.label) > + return -ENOMEM; > > ret =3D devm_gpiochip_add_data(dev, &rg->chip, mtk); > if (ret < 0) { > -- > 2.1.4 > > > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek