From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4698 invoked by alias); 25 Nov 2002 23:08:28 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 4691 invoked from network); 25 Nov 2002 23:08:27 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 25 Nov 2002 23:08:27 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.11.6/8.11.6) with ESMTP id gAPMi6P29483 for ; Mon, 25 Nov 2002 17:44:06 -0500 Received: from pobox.corp.redhat.com (pobox.corp.redhat.com [172.16.52.156]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id gAPN8QD11817; Mon, 25 Nov 2002 18:08:26 -0500 Received: from localhost.localdomain (vpn50-59.rdu.redhat.com [172.16.50.59]) by pobox.corp.redhat.com (8.11.6/8.11.6) with ESMTP id gAPN8Qw09004; Mon, 25 Nov 2002 18:08:26 -0500 Received: (from kev@localhost) by localhost.localdomain (8.11.6/8.11.6) id gAPN8KL15559; Mon, 25 Nov 2002 16:08:20 -0700 Date: Mon, 25 Nov 2002 15:08:00 -0000 From: Kevin Buettner Message-Id: <1021125230820.ZM15558@localhost.localdomain> In-Reply-To: Josh Martin "Re: [PATCH]: gdb/769 - segv fault on "info shared" on GDB 5.2.1 HPUX64 11.00" (Oct 9, 5:42pm) References: <200210092342.g99Ngrp05819@atoae450.abq.sc.philips.com> To: Josh Martin , gdb-patches@sources.redhat.com Subject: Re: [PATCH]: gdb/769 - segv fault on "info shared" on GDB 5.2.1 HPUX64 11.00 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2002-11/txt/msg00619.txt.bz2 On Oct 9, 5:42pm, Josh Martin wrote: > 2002-09-28 Josh Martin > > * solib.c (info_sharedlibrary_command): Added catch for potential > dereference of NULL pointer (current_target_so_ops). > Fix PR gdb/769. Josh, Your patch looks reasonable. I think I'd prefer to see a call to error() instead of printf_unfiltered() though: > > + printf_unfiltered("Shared library info is not yet implemented on this " > > + "platform\n"); Sorry for taking so long on this. Kevin