Rob Wing <[email protected]> wrote:
> +       if (!cn.status) {
>                 buf = NULL;
> +               if (err == 0)           /* keep compiler happy */
> +                       buf = NULL;
> +       }
>         return (buf);
>  }
> 
> And considering the comment at contrib/bearssl/inc/bearssl_x509.h
> is it intended for buf to be returned as-is (i.e., not set to NULL)
> when the decoding status indicates an error condition?

Good question, probably not, will fix.

Thanks for spotting.

Reply via email to