On 8/28/17 4:32 AM, Stefan Metzmacher via samba-technical wrote: > + if (dhparam.q) { > + dh->q = integer_to_BN(context, "DH p-1 factor", dhparam.q); > + if (dh->g == NULL) > + goto out; > + } Should that be dh->qin the null check? Doug