[LTP] [PATCH v3 3/3] syscalls/ioctl_ns0[156]: align stack and wait for child

Jan Stancek jstancek@redhat.com
Thu Jun 13 16:57:35 CEST 2019


----- Original Message -----
> Hi!
> > > We need free(child_stack) in the cleanup function.
> > 
> > Can you elaborate?
> 
> If I remember correctly at some point we decided to clean up after tests
> properly so that we don't upset various debugging tools, i.e. coverity,
> valgrind, etc. and I think that you were part of that discussion.

I recall I started with that position (free all), and I thought you
turned me around after this many years :-).

Do we have anything about this in style guide? I only found brief mention 
in "don't call cleanup from setup" section, which isn't even possible with newlib.

...
You don't need to clean up the following:                                                                                                                                                                          
                                                                                                                                                                                                                   
 * +malloc(3)+'ed memory.                                                                                                                                                                                          
 * Read-only file descriptors in persistent paths (i.e. not                                                                                                                                                        
   temporary directories).  

> 
> --
> Cyril Hrubis
> chrubis@suse.cz
> 


More information about the ltp mailing list