Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 3 Jan 2015 00:33:13 -0800
From:      John-Mark Gurney <jmg@funkthat.com>
To:        Hans Petter Selasky <hselasky@freebsd.org>
Cc:        John-Mark Gurney <jmg@freebsd.org>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r275732 - in head: etc/mtree share/man/man4 share/man/man7 share/man/man9 sys/conf sys/crypto/aesni sys/crypto/via sys/geom/eli sys/libkern sys/mips/rmi/dev/sec sys/modules/aesni sys/mo...
Message-ID:  <20150103083313.GB16816@funkthat.com>
In-Reply-To: <54A65C52.8090207@freebsd.org>
References:  <201412121956.sBCJucpj048475@svn.freebsd.org> <54A65C52.8090207@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Hans Petter Selasky wrote this message on Fri, Jan 02, 2015 at 09:52 +0100:
> On 12/12/14 20:56, John-Mark Gurney wrote:
> > Author: jmg
> > Date: Fri Dec 12 19:56:36 2014
> > New Revision: 275732
> > URL: https://svnweb.freebsd.org/changeset/base/275732
> >
> > Log:
> 
> Compile breakage with older compilers:
> 
> > crypto/aesni/aesni.c: In function 'aesni_process':
> > crypto/aesni/aesni.c:458: warning: 'ivlen' may be used uninitialized in this function
> > crypto/aesni/aesni.c:458: note: 'ivlen' was declared here

Thanks, I'll fix that tomorrow... I had a email server outage and now
just getting things back to working...

We really should look at turning of gcc's uninitialized var warning
now that it doesn't do a good job...  We are now just poluting our code
base w/ bad initalizations that could turn into bugs in the future when
someone adds a new code path that doesn't initalize the variable and
gets the bogus initalization...

-- 
  John-Mark Gurney				Voice: +1 415 225 5579

     "All that I will do, has been done, All that I have, has not."



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150103083313.GB16816>