From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 859 invoked by alias); 17 Mar 2008 22:16:33 -0000 Received: (qmail 852 invoked by uid 22791); 17 Mar 2008 22:16:32 -0000 X-Spam-Status: No, hits=-1.3 required=5.0 tests=AWL,BAYES_05,J_CHICKENPOX_33,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (66.187.233.31) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 17 Mar 2008 22:16:12 +0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id m2HMGBvu008665 for ; Mon, 17 Mar 2008 18:16:11 -0400 Received: from pobox-2.corp.redhat.com (pobox-2.corp.redhat.com [10.11.255.15]) by int-mx1.corp.redhat.com (8.13.1/8.13.1) with ESMTP id m2HMGALT006099 for ; Mon, 17 Mar 2008 18:16:10 -0400 Received: from localhost.localdomain (vpn-6-15.fab.redhat.com [10.33.6.15]) by pobox-2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id m2HMG9Fk015047 for ; Mon, 17 Mar 2008 18:16:10 -0400 Message-ID: <47DEEDA9.3080300@redhat.com> Date: Mon, 17 Mar 2008 22:16:00 -0000 From: Phil Muldoon User-Agent: Thunderbird 2.0.0.12 (X11/20080226) MIME-Version: 1.0 To: frysk@sourceware.org Subject: Re: [SCM] master: 2008-03-17 Phil Muldoon References: <20080317173318.6191.qmail@sourceware.org> In-Reply-To: <20080317173318.6191.qmail@sourceware.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.58 on 172.16.52.254 X-IsSubscribed: yes Mailing-List: contact frysk-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: frysk-owner@sourceware.org X-SW-Source: 2008-q1/txt/msg00154.txt.bz2 This commit fixes a test-case failure after the recent elfutils update. It was a small fix that changed the behavior to use getBytes() instead of accessing the ElfData directly. This should not, and never should have been the case. The next step is to limit the scope of that data structure and make sure to gate all access via get() routines. Regards Phil pmuldoon@sourceware.org wrote: > The branch, master has been updated > via d8919fbb113772ebc987e74061e838879556d9c4 (commit) > from ec2bf1299f4d201f2452558aafd0880c680c729c (commit) > > Those revisions listed above that are new to this repository have > not appeared on any other notification email. > > - Log ----------------------------------------------------------------- > commit d8919fbb113772ebc987e74061e838879556d9c4 > Author: Phil Muldoon > Date: Mon Mar 17 17:33:00 2008 +0000 > > 2008-03-17 Phil Muldoon > * TestCorefileByteBuffer.java (testCorefileByteBufferSlice): Use > getBytes(). Test for null returns. > > ----------------------------------------------------------------------- > > Summary of changes: > frysk-core/frysk/proc/dead/ChangeLog | 5 ++++- > .../frysk/proc/dead/TestCorefileByteBuffer.java | 8 +++++--- > 2 files changed, 9 insertions(+), 4 deletions(-) > > First 500 lines of diff: > diff --git a/frysk-core/frysk/proc/dead/ChangeLog b/frysk-core/frysk/proc/dead/ChangeLog > index 9229a67..4f8ff90 100644 > --- a/frysk-core/frysk/proc/dead/ChangeLog > +++ b/frysk-core/frysk/proc/dead/ChangeLog > @@ -1,5 +1,8 @@ > -2008-03-17 Andrew Cagney > +2008-03-17 Phil Muldoon > + * TestCorefileByteBuffer.java (testCorefileByteBufferSlice): Use > + getBytes(). Test for null returns. > > +2008-03-17 Andrew Cagney > * DeadHost.java: Update; using TaskAttachedObserverXXX; > * TestLinuxCore.java: Update. > * DeadTask.java: Ditto. > diff --git a/frysk-core/frysk/proc/dead/TestCorefileByteBuffer.java b/frysk-core/frysk/proc/dead/TestCorefileByteBuffer.java > index 1c2d3d2..aafe4bd 100644 > --- a/frysk-core/frysk/proc/dead/TestCorefileByteBuffer.java > +++ b/frysk-core/frysk/proc/dead/TestCorefileByteBuffer.java > @@ -57,7 +57,7 @@ public class TestCorefileByteBuffer > public void testCorefileByteBufferSlice() throws ElfException > { > > - ElfData rawData; > + ElfData rawData = null; > final long sliceBottom = 0x411bb000L; > final long sliceTop = 0x411bbfffL; > final long elfOffset = 0x28000; > @@ -74,16 +74,18 @@ public class TestCorefileByteBuffer > // Independently get the elf core data as a raw image > Elf segment = new Elf(Config.getPkgDataFile("test-core-x86"), > ElfCommand.ELF_C_READ); > + assertNotNull("Get Elf file for segment inspection", segment); > rawData = segment.getRawData(elfOffset,elfLen); > + assertNotNull("RawData is not null", rawData); > > + byte[] byteSet = rawData.getBytes(); > // coreSlice.position(sliceBottom); > for(int i=0; i assertEquals("Offset at 0x"+Long.toHexString(elfOffset+i) > +" does not match rawData at location " > + Long.toHexString(i), > - rawData.internal_buffer[i], > + byteSet[i], > coreSlice.get()); > - > segment.close(); > } > > > > hooks/post-receive > -- > frysk system monitor/debugger >