do_job() return value assigned to an unsigned int

[tnorth]

If the allocation of temporaries fails, do_job() returns -1. But on line 713 and line 734 :

ntbytes = do_job();
if (ntbytes < 0) {
   return -1;
}

Where ntbytes is uint32_t, line 623. The result will be cast to an unsigned it, and the failure will never be catched. (right ?)

[esc]

Is there any way to provoke this? For example by getting do_job() to return -1 on 25% of it's invocations?

[FrancescAlted]

That's right. That's a bug and must be fixed.