
Dan Smith wrote:
JG> +static void cleanup_domain_list(virDomainPtr *list, int size) JG> +{ JG> + int i; JG> + JG> + for (i = 0; i < size; i++) { JG> + virDomainFree(list[i]); JG> + } JG> +}
This should be in misc_util with the other domain_list functions.
Okay.
JG> @@ -221,15 +409,34 @@ static CMPI_THREAD_RETURN lifecycle_thre JG> conn, JG> virDomainGetName(prev_list[i]), JG> CS_DELETED); JG> - virDomainFree(prev_list[i]); JG> }
Was this intentional? Cleaning up the list as we go prevents us from having to go back through it again. I'm not sure if I see the items used later, so it seems like we could eliminate the second trip through with cleanup_domain_list().
I think this is a victim of the two patches working on the same code, and/or this change probably belongs in the second patch. When I do the more complete merge, all three loops work on the lists of dom_xml structs, so the only time the virDomaintPtr lists get used is when they are converted to dom_xml lists.
JG> + /* Should I free args as well here, since it's malloced in activate? */
Yes :)
Yea I was pretty sure on that. I mean it definitely needed to be freed, I just wasn't positive that this was the right spot. -- -Jay