Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp454880pxa; Fri, 21 Aug 2020 11:30:20 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyKGuUZaMJQ2q8Brl9+jI7nCA4sfQohhYpQoMLuCCpfj8CtNiBvSfcs/ik0THmF98nPlMdv X-Received: by 2002:a17:907:42cc:: with SMTP id nz20mr4157320ejb.429.1598034620792; Fri, 21 Aug 2020 11:30:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1598034620; cv=none; d=google.com; s=arc-20160816; b=kJqJCbuZqGcxaLUZoMsfOYGph0/0H6+E8EIACNQegrmBwjq58262qT4IA5dDZvZFTp j3ClgYS8ydCGCLu5wai6pjafnZ+rZiX6IH5NVWnTnnLV0R89n1S2PXKb7XvFZLGa16Mn Po9JL0R6fuk3FCCST/9giL3lHszByGyqrIQQz74JQYq4TUEmoBt6qyw8ka7JBs7yVsnL 3BXCf4reYMXFNHLLfiySe4Y/waTi1NEhb7JYLSFYw7CDoObxISfWhYTMvYGiL6R4EXUP BjnsSWao3cpfB5m+XafbT2i85HhwOlVHj3Wyyjb8Tl5SCT0RG9tc7m7GOJNutRtHxkfT AhKw== 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 :in-reply-to:references:mime-version; bh=/zXZIgCAakjsdNMqDTQ9OdhUqc8aoLFT/Ev7IhMzMsk=; b=XiIQ6BuyyuKNW17NX0L2MaGPt+g0A+0b9zLDjZRpkuN5qabiEUGpPEPHfNyeRJrp+b NaK2ER13ofMfebFebwCgzeNH/2//E6Ax9Gqawk/d1qtiNCIfG0PBK1MRmBUXE2RbEND5 1OSFel+lMSS9YvK6hAYFo50QEEFAmNBy39r9eE49Iwvus4D7BDkozqE8bkR6dsFv/3L3 +fb8Bgh4DbAv9hUUQFbm6sSRoqOqqvjf94wV5RePQ6hCpslW+crLg+aNZsmAX8ElbWWS 8Nkj5ejIaHZXrJ3+F+9OSVzhgD2u8/BRS3chuiC55SFGrl2eZWP1NSby9ZIN7Jh3EAN4 oq8w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 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. [23.128.96.18]) by mx.google.com with ESMTP id bs4si1680903edb.568.2020.08.21.11.29.56; Fri, 21 Aug 2020 11:30:20 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 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 S1726796AbgHUS2y (ORCPT + 99 others); Fri, 21 Aug 2020 14:28:54 -0400 Received: from mail-oi1-f195.google.com ([209.85.167.195]:45979 "EHLO mail-oi1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726638AbgHUS2x (ORCPT ); Fri, 21 Aug 2020 14:28:53 -0400 Received: by mail-oi1-f195.google.com with SMTP id o21so2298607oie.12; Fri, 21 Aug 2020 11:28:52 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=/zXZIgCAakjsdNMqDTQ9OdhUqc8aoLFT/Ev7IhMzMsk=; b=CqNS/nixEAmTficKp/RL0Ks0o5uUl3ztZ6hnmvp4yxJUGCef9wr/LLJ7OcJdfZe9C0 nJUKLbK7oyHhM2RhLbQIuFPgpQDwKD23fvZ7anngUeV4NpbbMTq6I5mhpYUWWkqCOI7u i5NKsp4CostSKgSmwqV0jaypGEaRtJmq3a4Er15ESYf5DFGP9EjRr4VQroPjF8XU9ro3 X42r/Yi4NhOE3i0T7XE3DD+o9jWKxcP1vHw6ndW5mEM0yo87O1Q8PEMMotqUosK0VxG8 F98WiHAKCUY2rruvcmJylcICBVWxUpG0j+jUcYImUyDSjo/jO6bZepi4iVgOEn9Mvqbu +jBA== X-Gm-Message-State: AOAM530YgAUvem2Y55E0VASeDYYjGfd0q32GWGsRqFqSEe+T9+cl63a+ aMpmvQZqoVV23mcDpCLyX1esX3ixSmf953o0vGk= X-Received: by 2002:aca:a88e:: with SMTP id r136mr2692389oie.110.1598034532187; Fri, 21 Aug 2020 11:28:52 -0700 (PDT) MIME-Version: 1.0 References: <20200819171656.2650926-1-furquan@google.com> In-Reply-To: <20200819171656.2650926-1-furquan@google.com> From: "Rafael J. Wysocki" Date: Fri, 21 Aug 2020 20:28:41 +0200 Message-ID: Subject: Re: [PATCH] drivers: acpi: apd: Check return value of acpi_dev_get_property To: Furquan Shaikh Cc: "Rafael J . Wysocki" , Len Brown , Aaron Durbin , "Agrawal, Akshu" , ACPI Devel Maling List , Linux Kernel Mailing List 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 On Wed, Aug 19, 2020 at 7:17 PM Furquan Shaikh wrote: > > `fch_misc_setup()` uses `acpi_dev_get_property()` to read the value of > "is-rv" passed in by BIOS in ACPI tables. However, not all BIOSes > might pass in this property and hence it is important to first check > the return value of `acpi_dev_get_property()` before referencing the > object filled by it. > > Signed-off-by: Furquan Shaikh > --- > drivers/acpi/acpi_apd.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c > index 4c348377a39d..806b8ce05624 100644 > --- a/drivers/acpi/acpi_apd.c > +++ b/drivers/acpi/acpi_apd.c > @@ -99,8 +99,8 @@ static int fch_misc_setup(struct apd_private_data *pdata) > if (ret < 0) > return -ENOENT; > > - acpi_dev_get_property(adev, "is-rv", ACPI_TYPE_INTEGER, &obj); > - clk_data->is_rv = obj->integer.value; > + if (!acpi_dev_get_property(adev, "is-rv", ACPI_TYPE_INTEGER, &obj)) > + clk_data->is_rv = obj->integer.value; > > list_for_each_entry(rentry, &resource_list, node) { > clk_data->base = devm_ioremap(&adev->dev, rentry->res->start, > -- Applied (with minor subject edits) as 5.9-rc material, thanks!