Hi Andreas, On Tue, May 7, 2013 at 10:22 AM, Fuchs, Andreas <[email protected]> wrote: > Hi list... > > Happy to join you... > > Attached you find a patch that resolves an integer-overflow. > It hits when you request UINT32_MAX events starting at a non-Zero-Index.
Thanks for the bug report! I'm hoping we can reduce the logic down a bit in the fix. Does this look like it will work to you? if ((FirstEvent + *pEventCount) >= FirstEvent && (FirstEvent + *pEventCount) >= *pEventCount) lastEventNumber = MIN(lastEventNumber, FirstEvent + *pEventCount); Else we leave lastEventNumber as is. It also looks like I'm missing a check if *pEventCount == 0 in this function, which I'll add. Kent > Cheers, > Andreas > ------------------------------------------------------------------------------ > Learn Graph Databases - Download FREE O'Reilly Book > "Graph Databases" is the definitive new guide to graph databases and > their applications. This 200-page book is written by three acclaimed > leaders in the field. The early access version is available now. > Download your free book today! http://p.sf.net/sfu/neotech_d2d_may > _______________________________________________ > TrouSerS-tech mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/trousers-tech > ------------------------------------------------------------------------------ Learn Graph Databases - Download FREE O'Reilly Book "Graph Databases" is the definitive new guide to graph databases and their applications. This 200-page book is written by three acclaimed leaders in the field. The early access version is available now. Download your free book today! http://p.sf.net/sfu/neotech_d2d_may _______________________________________________ TrouSerS-tech mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/trousers-tech
