Subject: Re: CVS commit: src/sys/kern
To: matthew green <mrg@eterna.com.au>
From: None <itojun@iijlab.net>
List: source-changes
Date: 05/17/2003 02:23:13
>in (nearly?) all the cases i looked at the "assumptions" were present
>only in the code of the block changed. in particular the MALLOC() case
>you present is *100%* a red-herring. these are *exactly* the safe uses
>of strc* that should be left alone. avoiding implicit assuptions such
>as your "foo.h" example is a good idea (i'm sure they exist even though
>my sample of your changes didn't seem to include any like that) but only
>those cases seem to be worth changing.
if it's not strcpy() after MALLOC(len + 1), it is "foo.h" case.
i know this because, before i put sizeof() at the end of arg list,
i need to check what kind of type "dst" arg is. when it is an array,
in many cases their size is hardcoded (like bar[16]).
>> i am doing it with reason. care to provide asm version of strlcpy? :-)
>
>but it's not necessarily a *good* reason. please stop converting any
>more code until it's shown to be a good reason.
determined by whom? and why should i obey you?
>my original post about
>this issue asked for actual *problems* being solved to be listed, but
>besides the "foo.h" case (of which no actual bugs have been fixed?) i'm
>really not seeing anything yet.
bzzt, there was a off-by-one error as seen in commit message.
itojun