Pull in vdukhovni/ssl_dane 8270afba fix
authorPhil Pennock <pdp@exim.org>
Sat, 25 Feb 2017 03:39:02 +0000 (22:39 -0500)
committerJeremy Harris <jgh146exb@wizmail.org>
Sat, 25 Feb 2017 15:15:30 +0000 (15:15 +0000)
Negating an int and comparing to another int was a bug which we got
lucky on.

src/src/dane-openssl.c

index bb927383fc879bd6a8288bba1c91467b313e5609..97acccb5ab9fa1e7395b60991edd6004ecd2902a 100644 (file)
@@ -667,7 +667,7 @@ for (n = sk_X509_num(in); n > 0; --n, ++depth)
     {
     if (grow_chain(dane, UNTRUSTED, ca))
       {
     {
     if (grow_chain(dane, UNTRUSTED, ca))
       {
-      if (!X509_check_issued(ca, ca) == X509_V_OK)
+      if (X509_check_issued(ca, ca) != X509_V_OK)
        {
        /* Restart with issuer as subject */
        cert = ca;
        {
        /* Restart with issuer as subject */
        cert = ca;