Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240820111100.GX10433@brightrain.aerifal.cx>
Date: Tue, 20 Aug 2024 07:11:01 -0400
From: Rich Felker <dalias@...c.org>
To: Gil Pedersen <kanongil@...il.com>
Cc: musl@...ts.openwall.com
Subject: Re: bug: isatty() can return wrong value

On Tue, Aug 20, 2024 at 11:50:02AM +0200, Gil Pedersen wrote:
> Hi,
> 
> I found a musl related issue in systemd, where it relies on glibc specific behaviour for isatty(), which happens to work out. When using musl, it will fail in another way, causing some essential retry logic to not run.
> 
> The error occurs when the isatty() implementation calls an ioctl() that sets the EIO error. glibc will directly forward any errors, while musl remaps it to an ENOTTY error.
> 
> Neither behaviour are POSIX compliant. glibc sets an undocumented errno, while musl sets ENOTTY for something that is definitely a TTY.
> 
> It seems that the correct fix is to remap the EIO to a success (1).
> 
> See https://github.com/systemd/systemd/pull/34039 and https://sourceware.org/bugzilla/show_bug.cgi?id=32103.
> 
> I'm not on the mailing list, so please CC any replies.

What guarantee do we have that nothing other than a tty in this state
will return EIO for the ioctl? The implementation on the kernel side
suggests that, if they tried to put any other device in such a state
by replacing its ioctl function the same way, it would also give EIO
for these ioctls.

The right way to do this on the kernel side would have been to only
return EIO for supported tty ioctls when in this state, and otherwise
return ENOTTY (inappropriate ioctl for device type).

I'm hopeful there's some good fix here where we don't have to have
either of these bad behaviors, but returning true for isatty() of a
device that is absolutely not a tty is a much worse behavior than
returning false for a device that is/was a tty when it was opened but
that's been replaced by the kernel with a defunct device.

Rich

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.