Skip to content

Commit 46a9ce5

Browse files
committed
Put in the "copy" part of util_ldap_search_node_copy.
1 parent e7694a8 commit 46a9ce5

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

modules/ldap/util_ldap_cache.c

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -156,13 +156,17 @@ void *util_ldap_search_node_copy(util_ald_cache_t *cache, void *c)
156156

157157
/* copy vals */
158158
if (node->vals) {
159-
newnode->vals = util_ald_alloc(cache, node->vals_len);
160-
newnode->vals_len = node->vals_len;
159+
void *vals = util_ald_alloc(cache, node->vals_len);
161160

162-
if (!newnode->vals) {
161+
if (!vals) {
163162
util_ldap_search_node_free(cache, newnode);
164163
return NULL;
165164
}
165+
166+
memcpy(vals, node->vals, node->vals_len);
167+
168+
newnode->vals = vals;
169+
newnode->vals_len = node->vals_len;
166170
}
167171
else {
168172
newnode->vals = NULL;

0 commit comments

Comments
 (0)