On Wed, May 28, 2014 at 10:10:16PM +0200, Ruediger Pluem wrote:
> Thanks, but I missed some stuff during review:
> 
> 1. We don't need to have two DH pointers in make_dh_params

Doh! 

> 2. There possible frees on NULL pointers in free_dh_params:

This is unnecessary because DH_free() does that already, but that 
deserves a comment too.  I'll fix this with your patch for (1) shortly, 
thanks!

Reply via email to