diff options
| author | Raphael Isemann <teemperor@gmail.com> | 2018-08-01 21:07:18 +0000 |
|---|---|---|
| committer | Raphael Isemann <teemperor@gmail.com> | 2018-08-01 21:07:18 +0000 |
| commit | f4590de99244d3a3d21a5a1b0994005d1d655962 (patch) | |
| tree | ad9beb015a11c8e471b50a5e69a488d6b545ea03 /clang/lib/AST/Decl.cpp | |
| parent | 9c66506604c7868c6e476df8410b8fcfe98ad975 (diff) | |
| download | bcm5719-llvm-f4590de99244d3a3d21a5a1b0994005d1d655962.tar.gz bcm5719-llvm-f4590de99244d3a3d21a5a1b0994005d1d655962.zip | |
Fix out-of-bounds read in Stream::PutCStringAsRawHex8
Summary:
When I added the Stream unit test (r338488), the build bots failed due to an out-of-
bound reads when passing an empty string to the PutCStringAsRawHex8 method.
In r338491 I removed the test case to fix the bots.
This patch fixes this in PutCStringAsRawHex8 by always checking for the terminating
null character in the given string (instead of skipping it the first time). It also re-adds the
test case I removed.
Reviewers: vsk
Reviewed By: vsk
Subscribers: vsk, lldb-commits
Differential Revision: https://reviews.llvm.org/D50149
llvm-svn: 338637
Diffstat (limited to 'clang/lib/AST/Decl.cpp')
0 files changed, 0 insertions, 0 deletions
