On Thu, 8 Mar 2012, Jacek Caban wrote:
I noticed we
return in this case, without initializing this variable.
Visual inspection indicates we do not seem to access the variable in
this error case, but a) better safe than sorry, and b) GCC 4.8 currently
warns about it.
I generally don't think this patch makes things any better (or
worse).
If we return an error, the caller should not expect this value to be
sane. What's the GCC 4.8 warning?
secmgr.c:230:1: warning: 'policy' may be used uninitialized in this
function [-Wmaybe-uninitialized]
An alternative to silence this would be initializing policy to some
default value. Would that be preferrable?
Gerald