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

Simo Sorce ssorce at redhat.com
Thu Dec 16 19:06:59 UTC 2010


On Thu, 16 Dec 2010 19:39:38 +0100
Zoran Pericic <zpericic at inet.hr> wrote:

> 
> diff -ur bind-dyndb-ldap-0.1.0b.org/src/krb5_helper.c 
> bind-dyndb-ldap-0.1.0b.keytab/src/krb5_helper.c
> --- bind-dyndb-ldap-0.1.0b.org/src/krb5_helper.c    2009-09-06 
> 22:50:14.000000000 +0200
> +++ bind-dyndb-ldap-0.1.0b.keytab/src/krb5_helper.c    2010-12-16 
> 19:30:18.281267020 +0100
> @@ -111,7 +111,7 @@
>                 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;

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.

Simo.

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




More information about the Freeipa-devel mailing list