diff options
author | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2016-04-26 17:40:25 -0300 |
---|---|---|
committer | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2016-04-26 17:40:25 -0300 |
commit | f9123b5003e62b6e54996076e860f23aee9a0593 (patch) | |
tree | c32d506730558f13d1c93c3296c698a6a61cd7a0 /ChangeLog | |
parent | 085bbece2c6afc5f13698b52fac0299445c4317f (diff) | |
download | glibc-f9123b5003e62b6e54996076e860f23aee9a0593.tar glibc-f9123b5003e62b6e54996076e860f23aee9a0593.tar.gz glibc-f9123b5003e62b6e54996076e860f23aee9a0593.tar.bz2 glibc-f9123b5003e62b6e54996076e860f23aee9a0593.zip |
libio: Update internal fmemopen position after write (BZ #20005)
Current GLIBC fmemopen fails with a simple testcase:
char buffer[500] = "x";
FILE *stream;
stream = fmemopen(buffer, 500, "r+");
fwrite("fish",sizeof(char),5,stream);
printf("pos-1:%ld\n",ftell(stream));
fflush(stream);
printf("pos-2:%ld\n",ftell(stream));
It returns:
pos-1:5
pos-2:0
Where it should return:
pos-1:5
pos-2:5
This is due the internal write function does not correctly update the internal
object position state and then the seek operation returns a wrong value. This
patch fixes it.
It fixes both BZ #20005 and BZ #19230 (marked as duplicated). A new test is
added to check for such case.
Tested on x86_64 and i686.
* libio/fmemopen.c (fmemopen_write): Update internal position after
write.
* stdio-common/Makefile (tests): Add tst-fmemopen4.c.
* stdio-common/tst-fmemopen4.c: New file..
Diffstat (limited to 'ChangeLog')
-rw-r--r-- | ChangeLog | 8 |
1 files changed, 8 insertions, 0 deletions
@@ -1,3 +1,11 @@ +2016-04-26 Adhemerval Zanella <adhemerval.zanella@linaro.org> + + [BZ #20005] + * libio/fmemopen.c (fmemopen_write): Update internal position after + write. + * stdio-common/Makefile (tests): Add tst-fmemopen4.c. + * stdio-common/tst-fmemopen4.c: New file.. + 2016-04-26 Joseph Myers <joseph@codesourcery.com> [BZ #19996] |