cvs commit: src/sys/kern sys_generic.c

Andrey Chernov ache at nagual.pp.ru
Sun Jul 3 01:10:44 GMT 2005


On Tue, Jan 25, 2005 at 12:09:33PM +0100, Poul-Henning Kamp wrote:
> >I.e. look at ERRORS section list and inspect each errno returning broken 
> >by preliminary zero bytes check.
> 
> That's another check we didn't do before to the resid==0 handling before.
> 
> Now, I'm going to killfile this subject until you come up with code which
> real-life code which breaks because of this, OK ?

I just show you the way of finding such cases, I already say I don't have 
much time to dig out real one. It is you who must prove that your addition 
not change anything or not violate POSIX. Because there is strong 
suspicion that you violates POSIX quote I already mention.

I.e. look in the fo_read() chain and see which errors are skipped now. 
Even if you don't find such ones, it may simple mean that resid==0 check 
was placed preliminary in our old code, and it is old code which needs to 
be fixed.

-- 
http://ache.pp.ru/



More information about the cvs-src mailing list