Constantine Aleksandrovich Murenin wrote:
> As reported elsewhere (http://seclists.org/oss-sec/2015/q4/87 via
> http://www.opennet.ru/43146), both of these errors were introduced as
> part of the refactoring.
> 
> Quick glance through
> http://bxr.su/o/lib/libssl/src/crypto/objects/obj_dat.c#OBJ_obj2txt
> indicates that the memory leak issue was introduced when a block scope
> variable within an if condition within a while loop was moved to
> function scope instead:
> 
> http://cvsweb.allbsd.org/cvsweb.cgi/src/lib/libssl/src/crypto/objects/obj_dat.c?cvsroot=openbsd#rev1.25
> http://cvsweb.allbsd.org/cvsweb.cgi/src/lib/libssl/src/crypto/objects/obj_dat.c.diff?cvsroot=openbsd&r2=1.25&r1=1.24&f=H
> 
> May I ask whether after this error, is it still frowned upon declaring
> block scope variables?

Did you look at the patch?

Reply via email to