> Well, when the world changes, that cast is suddenly wrong.

I.e. instead of
>       if (ret == -1 || ret >= sizeof(buffer))
one could use
>       if (ret < 0 || (size_t)ret >= sizeof(buffer))

And be safe from a changing world.

Reply via email to