fix undefined pointer comparison in stdio-internal __toread
authorRich Felker <dalias@aerifal.cx>
Tue, 29 Mar 2016 03:41:17 +0000 (23:41 -0400)
committerRich Felker <dalias@aerifal.cx>
Tue, 29 Mar 2016 03:41:17 +0000 (23:41 -0400)
commit6d1a3dfeaf2caac4033a3c65822fb4e7e14866c7
tree4fe0c1142109ff962183ab7c7ab645c99d5db765
parent5978eb703ce0e64dd778a88c1ffffb76fe5e2202
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.
src/stdio/__toread.c