[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZvWO7L-nGBBg0U0s@voyager>
Date: Thu, 26 Sep 2024 18:42:20 +0200
From: Markus Wichmann <nullplan@....net>
To: musl@...ts.openwall.com
Cc: "Jef STEELANT (EXT)" <jef.steelant_ext@...tathome.com>
Subject: Re: syslog does not detect a closed file descriptor
Am Thu, Sep 26, 2024 at 09:08:59AM +0000 schrieb Jef STEELANT (EXT):
> The syslog implementation does not detect if the file descriptor of
> /dev/log is closed, and keeps on logging to an invalid fd.
> This happens for instance when using lxc with musl libc ( see issue
> https://github.com/lxc/lxc/issues/4485). LXC will at a certain point
> close the fd of /dev/log, in the assumption that a next call to syslog
> will detect an EBADF when writing data, and that the log will be
> reopened. glibc does this, but musl does not. It would be better if
> syslog would either stop logging to the invalid fd, or reopen the log
> when s send returns EBADF.
This is why you do not close FDs you do not own. The application does
not own the socket FD for the syslog; it belongs to libc. And the
application can portably request the FD be closed by calling closelog().
And it should, because that will restore the initial state in that file,
and cause the log to be opened again.
Using EBADF for anything is a bad idea in the lib, since the lib must
work with multiple threads, and in a multi-threaded process EBADF means
"you could have sent this data to the wrong FD". I think the simplest
solution is for lxc to just call closelog() before closing the other
FDs.
Ciao,
Markus
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
