]> granicus.if.org Git - sudo/commitdiff
If sudo_ldap_get_values_len() fails goto cleanup instead of oom.
authorTodd C. Miller <Todd.Miller@sudo.ws>
Thu, 18 Oct 2018 20:19:09 +0000 (14:19 -0600)
committerTodd C. Miller <Todd.Miller@sudo.ws>
Thu, 18 Oct 2018 20:19:09 +0000 (14:19 -0600)
This is not strictly necessary as there's not anything to cleanup
in this case but it is more consistent with the code that follows.

plugins/sudoers/ldap.c

index 60931a592bc8c35e8865ac5defbdb29c28045d69..e1f858268a28c582f3e7a8d45a700a02fdf54074 100644 (file)
@@ -1151,13 +1151,13 @@ ldap_to_sudoers(LDAP *ld, struct ldap_result *lres,
        struct berval **runasusers = NULL, **runasgroups = NULL;
        struct berval **opts = NULL, **notbefore = NULL, **notafter = NULL;
        struct privilege *priv = NULL;
-       char *cn;
+       char *cn = NULL;
 
        /* Ignore sudoRole without sudoCommand. */
        cmnds = sudo_ldap_get_values_len(ld, entry, "sudoCommand", &rc);
        if (cmnds == NULL) {
            if (rc == LDAP_NO_MEMORY)
-               goto oom;
+               goto cleanup;
            continue;
        }