On 06/04/2011 10:11, Laszlo Papp wrote:
> Hi,
> 
> From the code:
> http://cvs.openssl.org/fileview?f=openssl/crypto/conf/conf_api.c&v=1.18.2.1
> 
> I think 'vv' could also be eliminated from the last function
> (_CONF_new_section) there and the assert could be called directly on
> the method.
> 
> Please englighten me if I am mistaken about it and thank you in advance ! :-)

In general it's not a good idea to assert on active code, since asserts
can be disabled.

-- 
http://www.apache-ssl.org/ben.html           http://www.links.org/

"There is no limit to what a man can do or how far he can go if he
doesn't mind who gets the credit." - Robert Woodruff
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to