Re: [REQUEST FOR VOTES] snmpd: Call fclose() after fdopen() but not close()

[Bart V. A. <bva...@ac...>]

On 2020-08-08 16:16, Magnus Fromreide wrote:
> That file is a mess.
> 
> Have anyone heard from Alexander Prömel regarding those temporary hard coded
> pathnames that should be fixed soon? I think 14 years cover any reasonable
> definition of the word 'soon'.
> 
> Why are both the file descriptor and the FILE* stored in the first place?
> It seems the read part of the code is using buffered I/O (fIn) while the
> write part of the code is using unbuffered I/O (fdOut) so I think fdIn and
> fOut should be removed completley.

I propose to delete the uClinux code from agent/mibgroup/util_funcs.c.
get_exec_pipes() is only used to open persistent pipes. So it is essential
to use fork(). Using vfork() in the implementation of get_exec_pipes() is
wrong because vfork() blocks the caller until the child process has
finished.

Bart.