On 07/03/2015 12:43 PM, Daniel Deptuła wrote:
> -    for (princptr = princs; *princptr != NULL; princptr++)
> -        TRACE_PKINIT_CLIENT_SAN_KDCCERT_PRINC(context, *princptr);
> +    if (princs != NULL) {
> +        for (princptr = princs; *princptr != NULL; princptr++)
> +            TRACE_PKINIT_CLIENT_SAN_KDCCERT_PRINC(context, *princptr);
> +    }

This is correct in spirit; other parts of the function allow for princs
to be NULL, and there are rare cases where the OpenSSL implementation
will return 0 with princs set to NULL.  I filed in a pull request for a
similar fix.  Thanks for the report.
________________________________________________
Kerberos mailing list           Kerberos@mit.edu
https://mailman.mit.edu/mailman/listinfo/kerberos

Reply via email to