Re: [PATCH] (2/4) support large number of network devices -- name hash

[YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@xxxxxxxxxxxxxx>]

In article <20040119112850.3b3575a7.shemminger@xxxxxxxx> (at Mon, 19 Jan 2004 
11:28:50 -0800), Stephen Hemminger <shemminger@xxxxxxxx> says:

> Use strnlen, great idea, thanks.

> -     size_t len = min(strlen(name),(size_t)(IFNAMSIZ-1));
> -     unsigned hash = full_name_hash(name, len);
> +     unsigned hash = full_name_hash(name, 
> +                                    strnlen(name, IFNAMSIZ-1));

I prefer IFNAMSIZ over IFNAMSIZ-1 here
bacause maximum length of valid name is IFNAMSIZ-1.

strnlen(3):
       size_t strnlen(const char *s, size_t maxlen);
:
       The strnlen function returns strlen(s), if that is less than maxlen, or
       maxlen  if there is no '\0' character among the first maxlen characters
       pointed to by s.

If strnlen(name,IFNAMSIZ) returns IFNAMSIZ, name is not terminated.
while strnlen(name,IFNAMSIZ-1) does not tell whether it is terminated or not.

--yoshfuji