No need to explicitly free role on EOF, it will be freed after the

loop is done.

diff --git a/plugins/sudoers/cvtsudoers_ldif.c b/plugins/sudoers/cvtsudoers_ldif.c
index da473900530f4cf33ae99b8acb50258445e8aaf7..47a8be1daf2015a193b24a0d94d0508a18ddffd7 100644 (file)
--- a/plugins/sudoers/cvtsudoers_ldif.c
+++ b/plugins/sudoers/cvtsudoers_ldif.c
@@ -1030,8 +1030,7 @@ parse_ldif(const char *input_file, struct cvtsudoers_config *conf)
                in_role = false;
            }
            if (len == -1) {
-               sudo_role_free(role);
-               role = NULL;
+               /* EOF */
                break;
            }
            mismatch = false;