Re: [libvirt] [PATCH 3/5] list: Implement listAllInterfaces

>> + >> + if (VIR_ALLOC_N(names, count)< 0) { >> + virReportOOMError(); >> + return -1; >> + } > > > If you want to check for a race here, you could allocate count+1 items > in the array, then do an ncf_list_interfaces telling it the maxcount is > "count+1" and check to see that you really only got count items back. No want I think, :-) Since I will adopt your below suggestion to igore the no found interface by "ncf_lookup_by_name", that means we should either ignore the race, or raise up error earlier here. I think ignoring is better, any objections?