diff options
author | Rich Felker <dalias@aerifal.cx> | 2015-02-23 18:53:01 -0500 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2015-02-23 18:53:01 -0500 |
commit | 2de85a985654d2c944931267645d9a0686242dfe (patch) | |
tree | 8b9e08fab48af344d9eedbc0ac57bbc9eaacf7da /src/thread/sem_destroy.c | |
parent | e487c203db36750388228a2db75f96040cbfae58 (diff) | |
download | musl-2de85a985654d2c944931267645d9a0686242dfe.tar.gz musl-2de85a985654d2c944931267645d9a0686242dfe.tar.xz musl-2de85a985654d2c944931267645d9a0686242dfe.zip |
fix possible isatty false positives and unwanted device state changes
the equivalent checks for newly opened stdio output streams, used to determine buffering mode, are also fixed. on most archs, the TCGETS ioctl command shares a value with SNDCTL_TMR_TIMEBASE, part of the OSS sound API which was apparently used with certain MIDI and timer devices. for file descriptors referring to such a device, TCGETS will not fail with ENOTTY as expected; it may produce a different error, or may succeed, and if it succeeds it changes the mode of the device. while it's unlikely that such devices are in use, this is in principle very harmful behavior for an operation which is supposed to do nothing but query whether the fd refers to a tty. TIOCGWINSZ, used to query logical window size for a terminal, was chosen as an alternate ioctl to perform the isatty check. it does not share a value with any other ioctl commands, and it succeeds on any tty device. this change also cleans up strace output to be less ugly and misleading.
Diffstat (limited to 'src/thread/sem_destroy.c')
0 files changed, 0 insertions, 0 deletions