Messages in this thread | | | Date | Wed, 6 Sep 2006 08:54:22 -0500 (CDT) | From | Chase Venters <> | Subject | Re: [take16 1/4] kevent: Core files. |
| |
On Wed, 6 Sep 2006, Chase Venters wrote:
>> + if (start + num >= KEVENT_MAX_EVENTS || >> + start >= KEVENT_MAX_EVENTS || >> + num >= KEVENT_MAX_EVENTS) > > Since start and num are unsigned, the last two checks are redundant. If start > or num is individually >= KEVENT_MAX_EVENTS, start + num must be. >
Actually, my early-morning brain code optimizer is apparently broken, because it forgot all about integer wraparound. Disregard please.
> > Thanks, > Chase > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |