On Mon, Aug 20, 2007 at 09:00:40AM +0200, Henrik Holst wrote:
>    Thanks Joe,
> 
>      the reason that I put it into ne_ssl_context_trustcert() was that it
>    was the real place where the pointer would be dereferenced and since there
>    also could be (in the future) other callers than ne_ssl_trust_cert() my
>    reasoning where that it would be better to protect all callers than to put
>    the protection higher up. Following this reason we should add the check to
>    the GnuTLS code aswell, but this is of course a matter of taste.

Ah, right.  In general there is no need to check where NULL is 
inappropriately passed to a function in place of a valid object; see the 
"Argument validation" section in the reference docs.

joe
_______________________________________________
neon mailing list
[email protected]
http://mailman.webdav.org/mailman/listinfo/neon

Reply via email to