DragonFly BSD
DragonFly submit List (threaded) for 2005-01
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: sysctl warns6 cleanup


From: Peter Schuller <peter.schuller@xxxxxxxxxxxx>
Date: Sat, 8 Jan 2005 20:37:14 +0100

I'll re-examine sysctl after Chris' commit.

> There should be at least a check for the value of len, to ensure that
> it actually fits into an int.

Roger that.

[snip]

> Dont do this, please. Use a new variable or reuse sep, this are constant
> strings and should stay so.

Sorry; I got into my head that val was being passed to some destructive
function. Obviously I was dreaming.

[snip]

> Change oidfmt and sysctl_all to take the lenght as size_t instead.

Will do. My misstake was assuming sysctl_all accepted an int for a reason,
instead of checking if it made sense to just go ahead and change it.

-- 
/ Peter Schuller, InfiDyne Technologies HB

PGP userID: 0xE9758B7D or 'Peter Schuller <peter.schuller@xxxxxxxxxxxx>'
Key retrieval: Send an E-Mail to getpgpkey@xxxxxxxxx
E-Mail: peter.schuller@xxxxxxxxxxxx Web: http://www.scode.org




[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]