On Mon, 2007-07-02 at 18:08 +0200, Peter Eriksson wrote:
> 
> +                               AC_CHECK_LIB(gss, __gss_userok, [
> +                                       AC_DEFINE(HAVE___GSS_USEROK,,
> +                                               Define if you have 
> __gss_userok())
> +                               ]) 

Instead of changing $LIBS, I think you can use:

AC_CHECK_LIB(gss, __gss_userok, [ ...
],, `krb5-config --libs gssapi`)

> +#if 0 /* Added 070702 Peter Eriksson <[EMAIL PROTECTED]> - ref cyrus-sasl 
> Solaris 8/9 has problems with NO_OID */
> +                                      GSS_C_NT_USER_NAME,
> +#else
>                                        GSS_C_NO_OID,
> +#endif

Is this a common problem? Should it be a setting? Although with your
change this code is never even executed, right?

> +       name = p_strndup(request->auth_request.pool,
> +                        (unsigned char *)outbuf.value + 4,
> +                        outbuf.length - 4);
> +
> +       if (!name) {
> +               auth_request_log_error(&request->auth_request,
> "gssapi",
> +                                      "Invalid response size");
> +               auth_request_fail(&request->auth_request);
> +               return;
> +       }           

name is never NULL.

> +               free(name);

And you shouldn't free() it either.

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to