From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-dy1-f178.google.com (mail-dy1-f178.google.com [74.125.82.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 13CDA3242AC for ; Mon, 16 Mar 2026 17:46:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773683180; cv=none; b=onrrcuwrYKl1UldF7lIQpHbphKmxZMLFrPbSriPoGNFNlF4qZgrbb2QsPftrXjgQdaFG8N9Vj4ipt2mG8Tnum1xuFqt4C+EKyh/CIBw4EpXJgxHFITDFaayyp34KjgpTOYCccPfJpQQnizz/v3LRPoFdSgKWr0v3Hap5MrvbqT0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773683180; c=relaxed/simple; bh=Fw1cbySh1egxleCzXRJ6Ic8rJblCTXWdzDTr8/1fiTQ=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=PIaeT0qAKLOaGS2r36Pn9dKCWYNGQ43+uZSpktGJ7crvZcJcDkvao2us4x2HZElfNmD7YBQi3ieyNQLTvOar35yxtotYA5ULEvaUvbN60XlDfrllZA598YUXGnXrn1+TRbpd8111irTMER/6ik7pc9wcfyGWTEUkTArD/qoOseI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=roeck-us.net; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Gz+19wq6; arc=none smtp.client-ip=74.125.82.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=roeck-us.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Gz+19wq6" Received: by mail-dy1-f178.google.com with SMTP id 5a478bee46e88-2c0c955a481so1340006eec.1 for ; Mon, 16 Mar 2026 10:46:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773683178; x=1774287978; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:from:to:cc:subject:date:message-id :reply-to; bh=FfsMC73FbjdBhiPsgRz6AJiY36W81X7vESrJAOPQgi8=; b=Gz+19wq6WXhZfApYXhK8+RuVnGTBrkiJp5ITsXiOwv9ZQdTjdOVnCp/uW48jJpisKS EW5xsQIDn+33NGEF5U80Dk8nPo71Xap4ItNsGQBt1/FtsuXhD4jIV+SC9rXWXlXJRbaH s09HLpcbQcE2IUC139ohWrU5wCsg6gJLNdvIIOYBG/hR0O5kuaL6+J0KygzdX/H2Nsdw nwcjmcZUrZVL68q07ktr874F7bfMzQJi93jjfi1lo8ZGa0JMTlv8KhIO7pVy7CBha2VJ jcbo2N28y2L5MjLEZ2SLkCKsoAKuRmsG8YearJlyIl/0fIUptitMDDjvXiQnj/J37O6z RA+w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773683178; x=1774287978; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=FfsMC73FbjdBhiPsgRz6AJiY36W81X7vESrJAOPQgi8=; b=YJD1FWC7LOV7C67FqavnbBcnhPWxdbFb0e6rKYgsmvJzIe7xo2iihpj80O6xHMIXYs XCKixL/LnnJPDk1szXOQLeDeEs7crPq8LjRPazsgLqWh455nDjPL2TvKrzQrmMAEMi46 tcbWfxrNpz+9E/GxXSjkHYWXfyBFYworhfs3z6ZyB2lips4RPXHbcwjXglcSamwPfj+t ZhSzLWtjkNEnm23Kp0ja4yXMgudCgcGES6WEHk2sEsOLNdbNRwBjotJzbJdZv7JMjwh/ iMurToSxzszYaqdgge23JqPP0iB/X+MJb0N98Gw0NvIlzgqE9d3Ou6udRvLAM6X2JXIu HCPg== X-Forwarded-Encrypted: i=1; AJvYcCWAE2Ucy783Bvv84nBEe1tsAmBXTFyHU9I8fYesEZnsBCKhKmstgbYhvWKnSaOM9Bp/ljQ+DGe5sCyjCBg=@vger.kernel.org X-Gm-Message-State: AOJu0Yx4CV7BtD8g6Cl9EfRRgK6ICZXo8r6HVlZETQfaTsHiT/yFxhw0 MTV1kguSY9/Dw9rW/NjVrV+VPhy/xcvdk5xFaomvlqHGHucJrVc2UqVx X-Gm-Gg: ATEYQzzn3q8qnKGti8O5+W+gkMxjeaiACFqONXi5FcSCT1uTaY5Lbn6wnYkbaaz4eUA Pg8jmV/I1HJXOMGxoUgbFcpvxev1YWMhihmw/QeNM6qEaOADmpsgW4L9h8WgMriiVXXzHYniTGw /L2FbYP/N1Dhf+kGMGC7pQJDGv4HYozETz+enFVmt2PCj4q2Ww1iW7jAcBtZjkvylB//qH+9Rzm xnKynK9Diibw+iKQdBU7CuELohZvGXaQxQAxBX5XIdE9MReNGyRHfeGUb1NmhxsUkm+O34MDXxj AFMe81IpUGox1FatEqZIaF/W91Cg+mPvvCFfbPyzr3/5qL86CA2ZXNUWOfpr45CpY1rtS10cHXD K84PPHo5wlnunrHTksAdUT4FTSy/5c35hKckXANFyKP1RB9dxkcQapzuQojIlSd8W8aiO0/m/v4 ckwh93j1vdY1SaPsWzDjcBFV5wmXkyi2UzfYWI X-Received: by 2002:a05:7301:3d0d:b0:2bd:e153:68bc with SMTP id 5a478bee46e88-2bea5559bc3mr6542555eec.25.1773683177973; Mon, 16 Mar 2026 10:46:17 -0700 (PDT) Received: from server.roeck-us.net ([2600:1700:e321:62f0:da43:aeff:fecc:bfd5]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-2beab3a1117sm18567628eec.3.2026.03.16.10.46.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Mar 2026 10:46:17 -0700 (PDT) Sender: Guenter Roeck Date: Mon, 16 Mar 2026 10:46:16 -0700 From: Guenter Roeck To: "Rafael J. Wysocki" Cc: Linux ACPI , LKML , "Dumbre, Saket" Subject: Re: [PATCH v1 05/26] ACPICA: Fix NULL pointer dereference in acpi_ev_address_space_dispatch() Message-ID: References: <12822121.O9o76ZdvQC@rafael.j.wysocki> <3030574.e9J7NaK4W3@rafael.j.wysocki> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3030574.e9J7NaK4W3@rafael.j.wysocki> Hi, On Wed, Jan 14, 2026 at 01:20:17PM +0100, Rafael J. Wysocki wrote: > From: Alexey Simakov > > Cover a missed execution path with a new check. > > Fixes: 0acf24ad7e10 ("ACPICA: Add support for PCC Opregion special context data") > Link: https://github.com/acpica/acpica/commit/f421dd9dd897 > Signed-off-by: Alexey Simakov > Signed-off-by: Rafael J. Wysocki > --- > drivers/acpi/acpica/evregion.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/acpi/acpica/evregion.c b/drivers/acpi/acpica/evregion.c > index fa3475da7ea9..b6198f73c81d 100644 > --- a/drivers/acpi/acpica/evregion.c > +++ b/drivers/acpi/acpica/evregion.c > @@ -163,7 +163,9 @@ acpi_ev_address_space_dispatch(union acpi_operand_object *region_obj, > return_ACPI_STATUS(AE_NOT_EXIST); > } > > - if (region_obj->region.space_id == ACPI_ADR_SPACE_PLATFORM_COMM) { > + if (field_obj > + && region_obj->region.space_id == > + ACPI_ADR_SPACE_PLATFORM_COMM) { > struct acpi_pcc_info *ctx = > handler_desc->address_space.context; > Google's experimental AI review agent provided the following feedback: If this setup block is executed with a NULL `field_obj`, it will skip initializing `ctx->length` and `ctx->subspace_id` even though they do not depend on `field_obj`. Additionally, because this initialization is part of the `!(region_obj->region.flags & AOPOBJ_SETUP_COMPLETE)` block, the setup flag will be set shortly after this. Does this mean that if the first call has a NULL `field_obj`, the region will be marked as setup complete, and a subsequent call with a valid `field_obj` will never initialize `ctx->internal_buffer`? Should the `field_obj` check only guard the assignment of `ctx->internal_buffer`? Please let me know if this is a real concern or not to help improve the agent. Thanks, Guenter