diff options
author | Gabe Black <gabeblack@google.com> | 2018-09-11 00:55:28 -0700 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2018-09-11 21:39:05 +0000 |
commit | 1023a92971a32cdfa58e119b5fe62b91d580e729 (patch) | |
tree | ca6cf65894d8a394d415d7d091ff120d0f82eaf3 /src/base/cprintf.cc | |
parent | 36276ba2392f030250fa9dd1e1ea28a026bc51c6 (diff) | |
download | gem5-1023a92971a32cdfa58e119b5fe62b91d580e729.tar.xz |
base: Save and restore the width setting of streams used with cprintf.
The flags, precision, and fill character were all being saved and
restored, but cprintf might also change the width setting of the
stream, and that wasn't being saved or restored and could leak from
the cprintf statement.
This change adds the code to save and restore that value.
Change-Id: Ibedb26f7f538cd3be4fe0462d2ee4e5efd62bc59
Reviewed-on: https://gem5-review.googlesource.com/12571
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
Diffstat (limited to 'src/base/cprintf.cc')
-rw-r--r-- | src/base/cprintf.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/base/cprintf.cc b/src/base/cprintf.cc index caf1bb847..b764f60e9 100644 --- a/src/base/cprintf.cc +++ b/src/base/cprintf.cc @@ -47,6 +47,7 @@ Print::Print(std::ostream &stream, const std::string &format) saved_flags = stream.flags(); saved_fill = stream.fill(); saved_precision = stream.precision(); + saved_width = stream.width(); } Print::Print(std::ostream &stream, const char *format) @@ -55,6 +56,7 @@ Print::Print(std::ostream &stream, const char *format) saved_flags = stream.flags(); saved_fill = stream.fill(); saved_precision = stream.precision(); + saved_width = stream.width(); } Print::~Print() @@ -310,6 +312,7 @@ Print::end_args() stream.flags(saved_flags); stream.fill(saved_fill); stream.precision(saved_precision); + stream.width(saved_width); } } // namespace cp |