On Sun, Feb 04, 2018 at 07:19:57PM -0700, Todd C. Miller wrote: > On Mon, 05 Feb 2018 09:21:32 +1300, Theo Buehler wrote: > > > The load_user() function gets a file descriptor from process_crontab(). > > It fdopen()s it directly and fclose()s the resulting stream. Then > > process_crontab() closes the stream a second time before exiting. > > > > Since crontab_fd is not load_user()'s descriptor, let's dup it before > > opening the stream. > > Alternately, we could just pass in a FILE * to load_user().
That's much nicer. ok
