Hello,

Joan Lledó, le dim. 24 nov. 2019 20:47:44 +0100, a ecrit:
> 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().

Applied, thanks!

> 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?

I don't think it's worth bothering.

Samuel

Reply via email to