From 246e722b4fb9a3a33c650cf536d2b0f51a1923b7 Mon Sep 17 00:00:00 2001 From: Tomas Babej Date: Wed, 16 Apr 2014 17:22:46 +0200 Subject: ipa_range_check: Make a new copy of forest_root_id attribute for range_info struct Not making a new copy of this attribute creates multiple frees caused by multiple pointers to the same forest_root_id from all the range_info structs for all the domains belonging to given forest. Part of: https://fedorahosted.org/freeipa/ticket/4137 Reviewed-By: Alexander Bokovoy --- daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c b/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c index fca4819a0..9a51d4196 100644 --- a/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c +++ b/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c @@ -147,7 +147,7 @@ static char* get_forest_root_id(struct domain_info *head, char* domain_id) { if (domain_id != NULL) { while(head) { if (strcasecmp(head->domain_id, domain_id) == 0) { - return head->forest_root_id; + return slapi_ch_strdup(head->forest_root_id); } head = head->next; } -- cgit