--- On Tue, 5/8/12, Bastiaan Timmer <[email protected]> wrote:

> From: Bastiaan Timmer <[email protected]>
> Subject: Re: [flac-dev] [PATCH] Add missing functions to VorbisComment class 
> + a few other things
> To: [email protected]
> Date: Tuesday, May 8, 2012, 3:13 PM
> --- On Tue, 5/8/12, Erik de Castro
> Lopo <[email protected]>
> wrote:
> 
> > Honestly, I really doubt this is a bug in valgrind :-).
> How
> > were you testing
> > this?
> 
> Well, I've read that there have been bugs in valgrind, were
> SSE optimized versions of strlen() do guaranteed safe
> overreads of memory, but valgrind wasn't aware the overreads
> were safe.

I am now more certain this is a valgrind bug in handling GNU's optimized 
strlen() (still existing in current valgrind 3.7.0). The code looks fine and 
when building flac with '-fno-builtin-strlen' valgrind reports no errors.
_______________________________________________
flac-dev mailing list
[email protected]
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to