Bug 359829

Summary: Power PC test suite none/tests/ppc64/test_isa_2_07.c uses uninitialzed data
Product: [Developer tools] valgrind Reporter: Carl Love <cel>
Component: generalAssignee: Julian Seward <jseward>
Status: CLOSED FIXED    
Severity: normal CC: will_schmidt
Priority: NOR    
Version First Reported In: 3.12 SVN   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Carl Love 2016-02-26 16:23:25 UTC 
The test none/tests/ppc64/test_isa_2_07.c steps beyond the data array in a number of places.  The issue is the array is declared to be of size N.  The for loop is:   for (i=0; i<N; i=i+2).  In the body of the loop the array elements A[i] and A[i+1] are accessed.  On the last iteration i=N-1 and the second array access becomes A[N-1+1] which accesses one element past the declared array size.

Valgrind reports the use of the uninitialized data.

Reproducible: Always


Actual Results:  
Valgrind should not report any uninitialized data accesses.
Comment 1 Carl Love 2016-02-26 17:40:20 UTC 
The code was fixed in Valgrind commit 15816.  The expected output files were also updated for the new code.