From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757441Ab3BVOjx (ORCPT ); Fri, 22 Feb 2013 09:39:53 -0500 Received: from www.linutronix.de ([62.245.132.108]:58007 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756216Ab3BVOjw (ORCPT ); Fri, 22 Feb 2013 09:39:52 -0500 Date: Fri, 22 Feb 2013 15:39:44 +0100 (CET) From: Thomas Gleixner To: "Rafael J. Wysocki" cc: Yinghai Lu , LKML , Toshi Kani , Huang Ying , ACPI Devel Maling List Subject: Re: [BUG bisected]: apei_hest_parse explosion In-Reply-To: <5350252.GArARx6b7p@vostro.rjw.lan> Message-ID: References: <4867361.uTndA5QxsU@vostro.rjw.lan> <5350252.GArARx6b7p@vostro.rjw.lan> User-Agent: Alpine 2.02 (LFD 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 22 Feb 2013, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > Subject: ACPI / APEI: Fix crash in apei_hest_parse() for acpi=off > > After commit 92ef2a2 (ACPI: Change the ordering of PCI root bridge > driver registrarion), acpi_hest_init() is never called for acpi=off > (acpi_disabled), so hest_disable is not set, but hest_tab is NULL, > which causes apei_hest_parse() to crash when it is called from > aer_acpi_firmware_first(). > > Fix that by making apei_hest_parse() check if hest_tab is not NULL > in addition to checking hest_disable. Also remove the now useless > acpi_disabled check from apei_hest_parse(). > > Reported-by: Thomas Gleixner > Signed-off-by: Rafael J. Wysocki Tested-by: Thomas Gleixner