Received: by 10.223.185.116 with SMTP id b49csp2043228wrg; Thu, 22 Feb 2018 07:19:36 -0800 (PST) X-Google-Smtp-Source: AH8x2275ZoLaVqLNCVgLqjwaimRz+lK4DEqVSIcjY9u9ezxchWG8DMq6FtCeDu2cqXcaLWhCFm+e X-Received: by 10.99.177.72 with SMTP id g8mr1281791pgp.99.1519312776423; Thu, 22 Feb 2018 07:19:36 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519312776; cv=none; d=google.com; s=arc-20160816; b=S/4QLa+vAAtriaUx7GNO+7fvbbdYS/Q3hqPlYDY1e7T09ZcA6+qaGJGvJlDCxdagu5 MznQKBN3BEWUHL9GKH+YSrKHy5GrHV4+WRY1S9HzvYIp0J+M2NwFHKZI33ztdra3xczj XiTiO6kmHLIrovvOjxGrMfAmr7i8UGapJKz92MzDrFXEGtd28oILR9fsM3SVUvf5z4aP 6oKOn3/8HMPWVbOoBQpXu1CZA2GH8iNvw92PvMzK8M4djzGEd01kAY9pHalOFw1isY68 0PbxXHGTlyFt1a7EjNgfYp/efYpHa6PHjcDtQ7j0JkvTracJg0zSBxPQR5c7nLuk7Y27 eXXg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :references:in-reply-to:mime-version:dkim-signature :arc-authentication-results; bh=Fc2txzdiRop19Mx72k48iHy6m6Y5nS2TRZYI3J4hXuM=; b=cD4XHYhV2ZVLQzZkIuc50OUJ96JbAwKpAz1tYyUIj/Ti7Uk2J+q0IcRrEhWK2pHecb 2uK7eyc8urj5sMlv9ueVJeBWDlJDRwrf8dW2jcuuxhXT3gLqB1EQ++T6WaWgkFS2Ziof c9Nlb+na6BD5oOb8L0HzawAri7Zwf8/eOSrNNR9fivhRu52YAJSHi+1QCeUzLFPQz0UF H+MNKZTTYmk6HSDK8UCx0p2k9QPTfZLnolO+tBfhizCkbLv9/SAQOf/W6eyb7W7ZoEt+ a6sIcKjnsNoAE9FMrX1zGjmdpd5bHS1n9mRwgYvsa9MQc0rWfF/ucNgkPE9h1ScvOwvw Zcjg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=QjL6KrFZ; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e1si147052pgs.425.2018.02.22.07.19.12; Thu, 22 Feb 2018 07:19:36 -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=@gmail.com header.s=20161025 header.b=QjL6KrFZ; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932980AbeBVPSU (ORCPT + 99 others); Thu, 22 Feb 2018 10:18:20 -0500 Received: from mail-io0-f193.google.com ([209.85.223.193]:43559 "EHLO mail-io0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932828AbeBVPSR (ORCPT ); Thu, 22 Feb 2018 10:18:17 -0500 Received: by mail-io0-f193.google.com with SMTP id l12so6343278ioc.10; Thu, 22 Feb 2018 07:18:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=Fc2txzdiRop19Mx72k48iHy6m6Y5nS2TRZYI3J4hXuM=; b=QjL6KrFZp41RDkv5aQhlwfL2WIaGw1G+xx7NM0lZjhA0X7Cc5EjVnRZeg93ruNbb8x 7bFnT1Wg/2krJLA4vv5y4aFynw2056oOwlBkb68Dsnf3ShfshN8SpnugUI7hKUQZNqcU l5smNacvrNSWntJGP3K+w07tqTcpT07t9DLZd6zJjlao5OpZTgyuHw1BYcyJxEm22u/a E0ICbkKIyH94iLJTSudwwKLFDC/uz0hXLWt1LGrSu3/1E554lADNxJnZ7aFHZhnwmrvM nqTcCyiomDKAQkNEyvxI/quSDmDyWFgVbHE4mHf3vdnx7T8fQ0j3IcG6yi9GogkfVf0j 8bDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=Fc2txzdiRop19Mx72k48iHy6m6Y5nS2TRZYI3J4hXuM=; b=mEQzWI9XwL2w7rlygOfa8ikobp4UxBD/hVbj25w4LOJDgwGPzmIM1HjOE4gv7kA42F J4AWf09uSX7yQUkHVWZfHvcHCg3fcJmpM+q+8R3/3Sh4UHfzdELodtlKjAfWbqjOuc9g Lwy6UOB0c1H7K23K+bV2cjkPpNfWfxjOdRWnIONPf+eElaqwue986nX/WI+ozyCztlyf X3e+OKxpe6ZXiqidP1gjERJLXDnmA6V2YqBnmhyeFAFxgh9n3zS+D6C8Ld3jrjkLxCJR 8IqYipaGm244wVAJHleWpJJQyNQkIOrOV42lxQZhK6qM9JU2KzCqzsCUdlYEKQsN0iSf EKnw== X-Gm-Message-State: APf1xPBwd9xFppgSuL9oVtN+FbT/y3iEW/kRDsXGwbwQ4Di5eWh8NLX1 1nbc1c0j2JwMnAY1Q1Gi27OQq2zyhUtJZOZiUv0= X-Received: by 10.107.129.8 with SMTP id c8mr9139259iod.218.1519312696317; Thu, 22 Feb 2018 07:18:16 -0800 (PST) MIME-Version: 1.0 Received: by 10.107.3.85 with HTTP; Thu, 22 Feb 2018 07:18:15 -0800 (PST) In-Reply-To: <20180220181751.20826-1-krzk@kernel.org> References: <20180220181751.20826-1-krzk@kernel.org> From: Tomasz Figa Date: Fri, 23 Feb 2018 00:18:15 +0900 Message-ID: Subject: Re: [PATCH] pinctrl: samsung: Validate alias coming from DT To: Krzysztof Kozlowski , Geert Uytterhoeven Cc: Sylwester Nawrocki , Linus Walleij , Kukjin Kim , linux-arm-kernel , "moderated list:SAMSUNG SOC CLOCK DRIVERS" , linux-gpio@vger.kernel.org, linux-kernel , stable@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2018-02-21 3:17 GMT+09:00 Krzysztof Kozlowski : > Driver uses alias from Device Tree as an index of pin controller data > array. In case of a wrong DTB or an out-of-tree DTB, the alias could be > outside of this data array leading to out-of-bounds access. > > Depending on binary and memory layout, this could be handled properly > (showing error like "samsung-pinctrl 3860000.pinctrl: driver data not > available") or could lead to exceptions. > > Reported-by: Geert Uytterhoeven > Cc: > Fixes: 30574f0db1b1 ("pinctrl: add samsung pinctrl and gpiolib driver") > Signed-off-by: Krzysztof Kozlowski > > --- > > Issue is present since first version of this driver so this should be > backported as far as possible. > > Tested on Odroid HC1 (Exynos5422). Tests on S3C24xx, S3C64xx, Exynos > ARM64 and any other chips are highly appreciated. > > Linus, > I can take it through samsung/pinctrl tree and later send you again as > pull request... or you can apply it directly as I do not have any other > pinctrl patches queued so far. > --- > drivers/pinctrl/samsung/pinctrl-exynos-arm.c | 56 +++++++++++++++++++---- > drivers/pinctrl/samsung/pinctrl-exynos-arm64.c | 14 +++++- > drivers/pinctrl/samsung/pinctrl-s3c24xx.c | 28 ++++++++++-- > drivers/pinctrl/samsung/pinctrl-s3c64xx.c | 7 ++- > drivers/pinctrl/samsung/pinctrl-samsung.c | 61 ++++++++++++++++---------- > drivers/pinctrl/samsung/pinctrl-samsung.h | 40 ++++++++++------- > 6 files changed, 154 insertions(+), 52 deletions(-) Acked-by: Tomasz Figa Thanks Geert for the report and Krzysztof for the patch. Best regards, Tomasz