Pass correct arguments to memcpy
authorAlan T. DeKok <aland@freeradius.org>
Thu, 25 Oct 2012 08:10:22 +0000 (10:10 +0200)
committerAlan T. DeKok <aland@freeradius.org>
Thu, 25 Oct 2012 08:10:22 +0000 (10:10 +0200)
Copying pointers means you pass a pointer to the pointer.

src/modules/rlm_krb5/rlm_krb5.c

index ab86b3f..4213262 100644 (file)
@@ -96,7 +96,7 @@ static int verify_krb5_tgt(krb5_context context, rlm_krb5_t *inst,
        char *keytab_name;
        
        /* krb5_kt_read_service_key lacks const qualifier */
-       memcpy(keytab_name, inst->keytab, sizeof(keytab_name));
+       memcpy(&keytab_name, &inst->keytab, sizeof(keytab_name));
 
        if (inst->service_princ != NULL) {
                server_name = strchr(inst->service_princ, '/');