On Tue, Nov 18, 2014 at 10:08:18PM +0100, Jan-Benedict Glaw wrote: > - if (len == 0 || data == end || len > (end - data)) > + if (len == 0 || data == end || len > (unsigned int) (end - data)) Strictly speaking that is the wrong cast. if (len == 0 || len > (uintptr_t) (end - data)) -- Alan Modra Australia Development Lab, IBM