From mboxrd@z Thu Jan 1 00:00:00 1970 From: stefano.stabellini@eu.citrix.com Subject: [PATCH 08/12] Fix possible NULL pointer dereference in print_IO_APIC Date: Thu, 3 Jun 2010 14:10:41 +0100 Message-ID: <1275570645-27189-8-git-send-email-stefano.stabellini@eu.citrix.com> References: <1275570645-27189-1-git-send-email-stefano.stabellini@eu.citrix.com> Return-path: In-Reply-To: <1275570645-27189-1-git-send-email-stefano.stabellini@eu.citrix.com> Sender: linux-kernel-owner@vger.kernel.org To: linux-kernel@vger.kernel.org Cc: Stefano.Stabellini@eu.citrix.com, xen-devel@lists.xensource.com, jeremy@goop.org, ddutile@redhat.com, sheng@linux.intel.com, Stefano Stabellini List-Id: xen-devel@lists.xenproject.org From: Stefano Stabellini Make sure chip_data is not NULL before accessing it (the VIRQ_TIMER handler and virq handlers in general don't have any chip_data). Signed-off-by: Stefano Stabellini --- arch/x86/kernel/apic/io_apic.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c index eb2789c..c64499c 100644 --- a/arch/x86/kernel/apic/io_apic.c +++ b/arch/x86/kernel/apic/io_apic.c @@ -1732,6 +1732,8 @@ __apicdebuginit(void) print_IO_APIC(void) struct irq_pin_list *entry; cfg = desc->chip_data; + if (!cfg) + continue; entry = cfg->irq_2_pin; if (!entry) continue; -- 1.7.0.4