a.out gives different results when compiled by g++ 3.1 and g++ 3.1 -O

Falk Hueffner falk.hueffner@student.uni-tuebingen.de
Sun Jun 16 12:40:00 GMT 2002


Erick Alphonse <alphonse@lri.fr> writes:

> Falk Hueffner wrote:
> 
> >> enum {N = 624,              // length of state vector
> >>       M = 397,              // period parameter
> >>       MAGIC = 0x9908B0DFU}; // magic constant
> >>
> >> ui32 state[N];              // internal state
> >>[...]
> >>*p++ = twist( p[M-N], p[0], p[1] );
> >>
> > You're accessing out of the array bounds here, so anything might
> > happen.
> 
> I can't see the problem, the for-loop is entered N-1 times. Valgrind
> reported nothing wrong.

MAGIC = 0x9908B0DFU forces the enum to become unsigned, therefore M-N
is done with unsigned arithmetic and you're accessing p[4294967069].

-- 
	Falk



More information about the Gcc-bugs mailing list