Joe Orton wrote: > 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! >
Are you waiting for any action from my side? Just want to avoid that we wait for each other and deadlock :-) Regards Rüdiger
