diff options
author | Rich Felker <dalias@aerifal.cx> | 2018-08-28 19:16:40 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2018-08-28 19:16:40 -0400 |
commit | f66022dda8d18e6732626c7806f6c4d32023d574 (patch) | |
tree | 8eefb9fe10f74c6e0e1406aceceaa636ca9c68cb | |
parent | b39b47bac8ee5505cfc595000a140c35460e1cac (diff) | |
download | musl-f66022dda8d18e6732626c7806f6c4d32023d574.tar.gz |
make fmemopen's w+ mode truncate the buffer
the w+ mode is specified to "truncate the buffer contents". like most
of fmemopen, exactly what this means is underspecified. mode w and w+
of course implicitly 'truncate' the buffer if a write from the initial
position is flushed, so in order for this part of the text about w+
not to be spurious, it should be interpreted as requiring something
else, and the obvious reasonable interpretation is that the truncation
is immediately visible if you attempt to read from the stream or the
buffer before writing/flushing.
this interpretation agrees with reported conformance test failures.
-rw-r--r-- | src/stdio/fmemopen.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/stdio/fmemopen.c b/src/stdio/fmemopen.c index fb2656e3..5e0eeb50 100644 --- a/src/stdio/fmemopen.c +++ b/src/stdio/fmemopen.c @@ -112,6 +112,7 @@ FILE *fmemopen(void *restrict buf, size_t size, const char *restrict mode) if (!plus) f->f.flags = (*mode == 'r') ? F_NOWR : F_NORD; if (*mode == 'r') f->c.len = size; else if (*mode == 'a') f->c.len = f->c.pos = strnlen(buf, size); + else if (plus) *f->c.buf = 0; f->f.read = mread; f->f.write = mwrite; |