diff options
author | Rich Felker <dalias@aerifal.cx> | 2011-09-04 16:04:28 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2011-09-04 16:04:28 -0400 |
commit | f81279ff583ef81bc88a46dd1d0140fb6e0ed222 (patch) | |
tree | 3a8c2651dcdd3b4ace8642d22f6cfdeb137c7d7c /src/stdio/fmemopen.c | |
parent | 7ee3dcb3c603b20fcd4547ffb00e11701c6d1cf4 (diff) | |
download | musl-f81279ff583ef81bc88a46dd1d0140fb6e0ed222.tar.gz |
fmemopen fixes
disallow seek past end of buffer (per posix)
fix position accounting to include data buffered for read
don't set eof flag when no data was requested
Diffstat (limited to 'src/stdio/fmemopen.c')
-rw-r--r-- | src/stdio/fmemopen.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/stdio/fmemopen.c b/src/stdio/fmemopen.c index f518528f..ddb24331 100644 --- a/src/stdio/fmemopen.c +++ b/src/stdio/fmemopen.c @@ -16,7 +16,7 @@ fail: return -1; } base = (size_t [3]){0, c->pos, c->len}[whence]; - if (off < -base || off > SSIZE_MAX-base) goto fail; + if (off < -base || off > (ssize_t)c->size-base) goto fail; return c->pos = base+off; } @@ -24,6 +24,7 @@ static size_t mread(FILE *f, unsigned char *buf, size_t len) { struct cookie *c = f->cookie; size_t rem = c->size - c->pos; + if (!len) return 0; if (len > rem) len = rem; memcpy(buf, c->buf+c->pos, len); c->pos += len; @@ -32,6 +33,7 @@ static size_t mread(FILE *f, unsigned char *buf, size_t len) f->rpos = f->buf; f->rend = f->buf + rem; memcpy(f->rpos, c->buf+c->pos, rem); + c->pos += rem; if (!len) f->flags |= F_EOF; return len; } |