summaryrefslogtreecommitdiff
path: root/src/stdio/__stdio_close.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2016-03-28 23:41:17 -0400
committerRich Felker <dalias@aerifal.cx>2016-03-28 23:41:17 -0400
commit6d1a3dfeaf2caac4033a3c65822fb4e7e14866c7 (patch)
tree4fe0c1142109ff962183ab7c7ab645c99d5db765 /src/stdio/__stdio_close.c
parent5978eb703ce0e64dd778a88c1ffffb76fe5e2202 (diff)
downloadmusl-6d1a3dfeaf2caac4033a3c65822fb4e7e14866c7.tar.gz
musl-6d1a3dfeaf2caac4033a3c65822fb4e7e14866c7.tar.bz2
musl-6d1a3dfeaf2caac4033a3c65822fb4e7e14866c7.tar.xz
musl-6d1a3dfeaf2caac4033a3c65822fb4e7e14866c7.zip
fix undefined pointer comparison in stdio-internal __toread
the comparison f->wpos > f->buf has undefined behavior when f->wpos is a null pointer, despite the intuition (and actual compiler behavior, for all known compilers) being that NULL > ptr is false for all valid pointers ptr. the purpose of the comparison is to determine if the write buffer is non-empty, and the idiom used elsewhere for that is comparison against f->wbase, which is either a null pointer when not writing, or equal to f->buf when writing. in the former case, both f->wpos and f->wbase are null; in the latter they are both non-null and point into the same array.
Diffstat (limited to 'src/stdio/__stdio_close.c')
0 files changed, 0 insertions, 0 deletions