[Freeipa-devel] [PATCH] bynd-dyndb-ldap: Fix keytab checking

[Simo Sorce]

On Fri, 17 Dec 2010 17:47:39 +0100
Zoran Pericic <zpericic at inet.hr> wrote:

> On 12/16/2010 08:06 PM, Simo Sorce wrote:
> 
> > Obvious ACK,
> > I will put the change in myself unless you can send me a git
> > formatted patch I can git am into my tree.
> 
> Thunerbird converted tabs to spaces. I hope this is ok.
> 
> Best regards,
> Zoran Pericic
> 
> diff --git a/src/krb5_helper.c b/src/krb5_helper.c
> index
> a52b412f10551dfb4079ef5add37d0ebe000d310..571f511c71a4a0d547e0e74f5b5109a0bd5498b1
> 100644 --- a/src/krb5_helper.c +++ b/src/krb5_helper.c
> @@ -111,7 +111,7 @@ get_krb5_tgt(isc_mem_t *mctx, const char
> *principal, const char *keyfile) DEFAULT_KEYTAB);
>   		keyfile = DEFAULT_KEYTAB;
>   	} else {
> -		if (strcmp(keyfile, "FILE:") != 0) {
> +		if (strncmp(keyfile, "FILE:", 5) != 0) {
>   			log_error("Unknown keytab file name format,
> " "missing leading 'FILE:' prefix");
>   			return ISC_R_FAILURE;

This patch was pushed by Adam.

Simo.


-- 
Simo Sorce * Red Hat, Inc * New York