From: Todd C. Miller <Todd.Miller@sudo.ws>
Date: Thu, 3 May 2018 17:06:02 +0000 (-0600)
Subject: No need to explicitly free role on EOF, it will be freed after the
X-Git-Tag: SUDO_1_8_24^2~94
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7a940ce30b71cc9634ba7370c98629aa3023cbdc;p=sudo

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 da4739005..47a8be1da 100644
--- 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;