Thanks James, fix merged and submitted to svn/trunk.

On 15 November 2014 15:17, James Turner <[email protected]> wrote:

>
> On 14 Nov 2014, at 10:49, James Turner <[email protected]> wrote:
>
> I realise my original report was a little lacking in info, I am working to
> get everything compiled with additional debug / logging so I can verify if
> the issue is out-of-bounds i value or something else.
>
>
> Converting the loops to forward versions fixed the issue. The problem is
> size_t is unsigned; at the limit condition it doesn't go negative but wraps
> around to 0xffffffffffffffff .... and boom.
>
> Patch is here (since unsure if attachments are permitted on this list):
>
> http://dpaste.com/32935WT
>
> I have /not/ audited the entire patch for any other similar issues. I am
> little surprised Clang didn't flag comparison of an unsigned value with >=0
>  - since this is a 'test will always return true' case which it normally
> reports on.
>
> Kind regards,
> James
>
>
> _______________________________________________
> osg-users mailing list
> [email protected]
> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
>
>
_______________________________________________
osg-users mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org

Reply via email to