From fddcd19abfee871994f63ad22e3a1957b81a7db3 Mon Sep 17 00:00:00 2001 From: Jonathan Wakely Date: Wed, 19 Jul 2017 00:39:34 +0100 Subject: [PATCH] PR libstdc++/81395 fix crash when write follows large read PR libstdc++/81395 * include/bits/fstream.tcc (basic_filebuf::xsgetn): Don't set buffer pointers for write mode after reading. * testsuite/27_io/basic_filebuf/sgetn/char/81395.cc: New. From-SVN: r250328 --- libstdc++-v3/ChangeLog | 7 +++ libstdc++-v3/include/bits/fstream.tcc | 2 +- .../27_io/basic_filebuf/sgetn/char/81395.cc | 46 +++++++++++++++++++ 3 files changed, 54 insertions(+), 1 deletion(-) create mode 100644 libstdc++-v3/testsuite/27_io/basic_filebuf/sgetn/char/81395.cc diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 7af51ac01bc..2489fcaa469 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,10 @@ +2017-07-18 Jonathan Wakely + + PR libstdc++/81395 + * include/bits/fstream.tcc (basic_filebuf::xsgetn): Don't set buffer + pointers for write mode after reading. + * testsuite/27_io/basic_filebuf/sgetn/char/81395.cc: New. + 2017-07-18 François Dumont * include/bits/stl_list.h diff --git a/libstdc++-v3/include/bits/fstream.tcc b/libstdc++-v3/include/bits/fstream.tcc index b1beff86ab5..ef51a8409fc 100644 --- a/libstdc++-v3/include/bits/fstream.tcc +++ b/libstdc++-v3/include/bits/fstream.tcc @@ -699,7 +699,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION if (__n == 0) { - _M_set_buffer(0); + // Set _M_reading. Buffer is already in initial 'read' mode. _M_reading = true; } else if (__len == 0) diff --git a/libstdc++-v3/testsuite/27_io/basic_filebuf/sgetn/char/81395.cc b/libstdc++-v3/testsuite/27_io/basic_filebuf/sgetn/char/81395.cc new file mode 100644 index 00000000000..49856286277 --- /dev/null +++ b/libstdc++-v3/testsuite/27_io/basic_filebuf/sgetn/char/81395.cc @@ -0,0 +1,46 @@ +// Copyright (C) 2017 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +// { dg-do run } + +// PR libstdc++/81395 + +#include +#include // for std::memset +#include // For BUFSIZ + +using std::memset; + +int main() +{ + { + std::filebuf fb; + fb.open("test.txt", std::ios::out); + char data[BUFSIZ]; + memset(data, 'A', sizeof(data)); + fb.sputn(data, sizeof(data)); + } + + std::filebuf fb; + fb.open("test.txt", std::ios::in|std::ios::out); + char buf[BUFSIZ]; + memset(buf, 0, sizeof(buf)); + fb.sgetn(buf, sizeof(buf)); + // Switch from reading to writing without seeking first: + fb.sputn("B", 1); + fb.pubsync(); +} -- 2.30.2