rlm_krb5: krb5_unparse_name return ignored

diff --git a/src/modules/rlm_krb5/rlm_krb5.c b/src/modules/rlm_krb5/rlm_krb5.c
index 9ef3cd59f854b0a5ee33d20c0aa65c03a7a0fb2b..f8be15fe3c71ef2af13bc3351982ad43638fda5e 100644 (file)
--- a/src/modules/rlm_krb5/rlm_krb5.c
+++ b/src/modules/rlm_krb5/rlm_krb5.c
@@ -263,13 +263,14 @@ static rlm_rcode_t krb5_parse_user(krb5_principal *client, KRB5_UNUSED rlm_krb5_
                return RLM_MODULE_FAIL;
        }
 
-       krb5_unparse_name(context, *client, &princ_name);
-       RDEBUG2("Using client principal \"%s\"", princ_name);
+       if (krb5_unparse_name(context, *client, &princ_name) == 0) {
+               RDEBUG2("Using client principal \"%s\"", princ_name);
 #ifdef HEIMDAL_KRB5
-       free(princ_name);
+               free(princ_name);
 #else
-       krb5_free_unparsed_name(context, princ_name);
+               krb5_free_unparsed_name(context, princ_name);
 #endif
+       }
        return RLM_MODULE_OK;
 }