Date: Sun, 22 May 2016 04:46:03 +0300 From: Andrey Chernov <ache@freebsd.org> To: "Pedro F. Giffuni" <pfg@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r300384 - head/sys/compat/ndis Message-ID: <82b74a0b-7551-31ed-6b89-42925e072900@freebsd.org> In-Reply-To: <201605220029.u4M0TPI6050123@repo.freebsd.org> References: <201605220029.u4M0TPI6050123@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 22.05.2016 3:29, Pedro F. Giffuni wrote:
> static void
> srand(unsigned int seed)
> {
>
> + if (seed == 0)
> + seed = 1;
> srandom(seed);
> }
IMHO it is not needed. srandom() already handles seed == 0 case, there
is no point to remap it to another constant.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?82b74a0b-7551-31ed-6b89-42925e072900>