Hello,

Working with nested arbiters I found a heap corruption when the nested
arbiter founds no devices b/c the master arbiter doesn't give it any
permissions. Attached is a patch to fix it. It also removes an unneeded
free().

On the other hand, I noticed that the pointer allocated at
alloc_file_system() in pcifs.c is never freed. This pointer is needed
during all the execution of the arbiter, so the moment to free it is
when the arbiter is going to be stopped. Is it worth to free it just
before the process is finished and all memory will be released anyway?



Reply via email to