ptclient/ldap.c:ptsmodule_make_authstate_group(): avoid strdup()+free()

ksmurchison · Nov 28, 2024 · bb576b8 · bb576b8
1 parent 72572ff
commit bb576b8
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/ptclient/ldap.c b/ptclient/ldap.c
@@ -1341,10 +1341,8 @@ static int ptsmodule_make_authstate_group(
                     ptsm->group_base = vals[0];
                     rc = PTSM_OK;
                 } else if ((vals = ldap_get_values(ptsm->ld, entry, ptsm->domain_name_attribute)) != NULL) {
-                    char *new_domain = xstrdup(vals[0]);
-                    syslog(LOG_DEBUG, "(groups) Domain %s is now domain %s", domain, new_domain);
-                    rc = ptsmodule_standard_root_dn(new_domain, &ptsm->group_base);
-                    free(new_domain);
+                    syslog(LOG_DEBUG, "(groups) Domain %s is now domain %s", domain, vals[0]);
+                    rc = ptsmodule_standard_root_dn(vals[0], &ptsm->group_base);
                 } else {
                     rc = ptsmodule_standard_root_dn(domain, &ptsm->group_base);
                 }