> > 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.
The code is already safe.
