mirror of
https://github.com/chromium/crashpad.git
synced 2024-12-29 00:32:35 +08:00
cc166d71f4
This is a follow-up to c8a016b99d97, following the post-landing discussion at https://chromium-review.googlesource.com/c/crashpad/crashpad/+/1393921/5#message-2058541d8c4505d20a990ab7734cd758e437a5f7 base::size, and std::size that will eventually replace it when C++17 is assured, does not allow the size of non-static data members to be taken in constant expression context. The remaining uses of ArraySize are in: minidump/minidump_exception_writer.cc (×1) minidump/minidump_system_info_writer.cc (×2, also uses base::size) snapshot/cpu_context.cc (×4, also uses base::size) util/misc/arraysize_test.cc (×10, of course) The first of these occurs when initializing a constexpr variable. All others are in expressions used with static_assert. Includes: Update mini_chromium to 737433ebade4d446643c6c07daae02a67e8deccao f701716d9546 Add Windows ARM64 build target to mini_chromium 87a95a3d6ac2 Remove the arraysize macro 1f7255ead1f7 Placate MSVC in areas of base::size usage 737433ebade4 Add cast Bug: chromium:837308 Change-Id: I6a5162654461b1bdd9b7b6864d0d71a734bcde19 Reviewed-on: https://chromium-review.googlesource.com/c/1396108 Commit-Queue: Mark Mentovai <mark@chromium.org> Reviewed-by: Mark Mentovai <mark@chromium.org>
270 lines
9.4 KiB
C++
270 lines
9.4 KiB
C++
// Copyright 2014 The Crashpad Authors. All rights reserved.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
#include "minidump/minidump_string_writer.h"
|
|
|
|
#include <string>
|
|
|
|
#include "base/compiler_specific.h"
|
|
#include "base/format_macros.h"
|
|
#include "base/stl_util.h"
|
|
#include "base/strings/stringprintf.h"
|
|
#include "base/strings/utf_string_conversions.h"
|
|
#include "gtest/gtest.h"
|
|
#include "minidump/test/minidump_rva_list_test_util.h"
|
|
#include "minidump/test/minidump_string_writer_test_util.h"
|
|
#include "minidump/test/minidump_writable_test_util.h"
|
|
#include "util/file/string_file.h"
|
|
|
|
namespace crashpad {
|
|
namespace test {
|
|
namespace {
|
|
|
|
TEST(MinidumpStringWriter, MinidumpUTF16StringWriter) {
|
|
StringFile string_file;
|
|
|
|
{
|
|
SCOPED_TRACE("unset");
|
|
string_file.Reset();
|
|
crashpad::internal::MinidumpUTF16StringWriter string_writer;
|
|
EXPECT_TRUE(string_writer.WriteEverything(&string_file));
|
|
ASSERT_EQ(string_file.string().size(), 6u);
|
|
|
|
const MINIDUMP_STRING* minidump_string =
|
|
MinidumpStringAtRVA(string_file.string(), 0);
|
|
EXPECT_TRUE(minidump_string);
|
|
EXPECT_EQ(MinidumpStringAtRVAAsString(string_file.string(), 0),
|
|
base::string16());
|
|
}
|
|
|
|
static constexpr struct {
|
|
size_t input_length;
|
|
const char* input_string;
|
|
size_t output_length;
|
|
base::char16 output_string[10];
|
|
} kTestData[] = {
|
|
{0, "", 0, {}},
|
|
{1, "a", 1, {'a'}},
|
|
{2, "\0b", 2, {0, 'b'}},
|
|
{3, "cde", 3, {'c', 'd', 'e'}},
|
|
{9, "Hi world!", 9, {'H', 'i', ' ', 'w', 'o', 'r', 'l', 'd', '!'}},
|
|
{7, "ret\nurn", 7, {'r', 'e', 't', '\n', 'u', 'r', 'n'}},
|
|
{2, "\303\251", 1, {0x00e9}}, // é
|
|
|
|
// oóöőo
|
|
{8, "o\303\263\303\266\305\221o", 5, {'o', 0x00f3, 0x00f6, 0x151, 'o'}},
|
|
{4, "\360\220\204\202", 2, {0xd800, 0xdd02}}, // 𐄂 (non-BMP)
|
|
};
|
|
|
|
for (size_t index = 0; index < base::size(kTestData); ++index) {
|
|
SCOPED_TRACE(base::StringPrintf(
|
|
"index %" PRIuS ", input %s", index, kTestData[index].input_string));
|
|
|
|
// Make sure that the expected output string with its NUL terminator fits in
|
|
// the space provided.
|
|
ASSERT_EQ(
|
|
kTestData[index]
|
|
.output_string[base::size(kTestData[index].output_string) - 1],
|
|
0);
|
|
|
|
string_file.Reset();
|
|
crashpad::internal::MinidumpUTF16StringWriter string_writer;
|
|
string_writer.SetUTF8(std::string(kTestData[index].input_string,
|
|
kTestData[index].input_length));
|
|
EXPECT_TRUE(string_writer.WriteEverything(&string_file));
|
|
|
|
const size_t expected_utf16_units_with_nul =
|
|
kTestData[index].output_length + 1;
|
|
MINIDUMP_STRING* tmp;
|
|
ALLOW_UNUSED_LOCAL(tmp);
|
|
const size_t expected_utf16_bytes =
|
|
expected_utf16_units_with_nul * sizeof(tmp->Buffer[0]);
|
|
ASSERT_EQ(string_file.string().size(), sizeof(*tmp) + expected_utf16_bytes);
|
|
|
|
const MINIDUMP_STRING* minidump_string =
|
|
MinidumpStringAtRVA(string_file.string(), 0);
|
|
EXPECT_TRUE(minidump_string);
|
|
base::string16 expect_string = base::string16(
|
|
kTestData[index].output_string, kTestData[index].output_length);
|
|
EXPECT_EQ(MinidumpStringAtRVAAsString(string_file.string(), 0),
|
|
expect_string);
|
|
}
|
|
}
|
|
|
|
// Related tracking issues:
|
|
// https://fuchsia.atlassian.net/browse/DX-487
|
|
// https://bugs.chromium.org/p/chromium/issues/detail?id=872892
|
|
// https://bugs.chromium.org/p/chromium/issues/detail?id=889582
|
|
// TODO: Re-enable test once LUCI supports invalid UTF8 characters in test logs.
|
|
#if !defined(CRASHPAD_IS_IN_FUCHSIA)
|
|
TEST(MinidumpStringWriter, ConvertInvalidUTF8ToUTF16) {
|
|
StringFile string_file;
|
|
|
|
static constexpr const char* kTestData[] = {
|
|
"\200", // continuation byte
|
|
"\300", // start byte followed by EOF
|
|
"\310\177", // start byte without continuation
|
|
"\340\200", // EOF in middle of 3-byte sequence
|
|
"\340\200\115", // invalid 3-byte sequence
|
|
"\303\0\251", // NUL in middle of valid sequence
|
|
};
|
|
|
|
for (size_t index = 0; index < base::size(kTestData); ++index) {
|
|
SCOPED_TRACE(base::StringPrintf(
|
|
"index %" PRIuS ", input %s", index, kTestData[index]));
|
|
string_file.Reset();
|
|
crashpad::internal::MinidumpUTF16StringWriter string_writer;
|
|
string_writer.SetUTF8(kTestData[index]);
|
|
EXPECT_TRUE(string_writer.WriteEverything(&string_file));
|
|
|
|
// The requirements for conversion of invalid UTF-8 input are lax. Make sure
|
|
// that at least enough data was written for a string that has one unit and
|
|
// a NUL terminator, make sure that the length field matches the length of
|
|
// data written, and make sure that at least one U+FFFD replacement
|
|
// character was written.
|
|
const MINIDUMP_STRING* minidump_string =
|
|
MinidumpStringAtRVA(string_file.string(), 0);
|
|
EXPECT_TRUE(minidump_string);
|
|
MINIDUMP_STRING* tmp;
|
|
ALLOW_UNUSED_LOCAL(tmp);
|
|
EXPECT_EQ(
|
|
minidump_string->Length,
|
|
string_file.string().size() - sizeof(*tmp) - sizeof(tmp->Buffer[0]));
|
|
base::string16 output_string =
|
|
MinidumpStringAtRVAAsString(string_file.string(), 0);
|
|
EXPECT_FALSE(output_string.empty());
|
|
EXPECT_NE(output_string.find(0xfffd), base::string16::npos);
|
|
}
|
|
}
|
|
#endif // !defined(CRASHPAD_IS_IN_FUCHSIA)
|
|
|
|
TEST(MinidumpStringWriter, MinidumpUTF8StringWriter) {
|
|
StringFile string_file;
|
|
|
|
{
|
|
SCOPED_TRACE("unset");
|
|
string_file.Reset();
|
|
crashpad::internal::MinidumpUTF8StringWriter string_writer;
|
|
EXPECT_TRUE(string_writer.WriteEverything(&string_file));
|
|
ASSERT_EQ(string_file.string().size(), 5u);
|
|
|
|
const MinidumpUTF8String* minidump_string =
|
|
MinidumpUTF8StringAtRVA(string_file.string(), 0);
|
|
EXPECT_TRUE(minidump_string);
|
|
EXPECT_EQ(MinidumpUTF8StringAtRVAAsString(string_file.string(), 0),
|
|
std::string());
|
|
}
|
|
|
|
static constexpr struct {
|
|
size_t length;
|
|
const char* string;
|
|
} kTestData[] = {
|
|
{0, ""},
|
|
{1, "a"},
|
|
{2, "\0b"},
|
|
{3, "cde"},
|
|
{9, "Hi world!"},
|
|
{7, "ret\nurn"},
|
|
{2, "\303\251"}, // é
|
|
|
|
// oóöőo
|
|
{8, "o\303\263\303\266\305\221o"},
|
|
{4, "\360\220\204\202"}, // 𐄂 (non-BMP)
|
|
};
|
|
|
|
for (size_t index = 0; index < base::size(kTestData); ++index) {
|
|
SCOPED_TRACE(base::StringPrintf(
|
|
"index %" PRIuS ", input %s", index, kTestData[index].string));
|
|
|
|
string_file.Reset();
|
|
crashpad::internal::MinidumpUTF8StringWriter string_writer;
|
|
std::string test_string(kTestData[index].string, kTestData[index].length);
|
|
string_writer.SetUTF8(test_string);
|
|
EXPECT_EQ(string_writer.UTF8(), test_string);
|
|
EXPECT_TRUE(string_writer.WriteEverything(&string_file));
|
|
|
|
const size_t expected_utf8_bytes_with_nul = kTestData[index].length + 1;
|
|
ASSERT_EQ(string_file.string().size(),
|
|
sizeof(MinidumpUTF8String) + expected_utf8_bytes_with_nul);
|
|
|
|
const MinidumpUTF8String* minidump_string =
|
|
MinidumpUTF8StringAtRVA(string_file.string(), 0);
|
|
EXPECT_TRUE(minidump_string);
|
|
EXPECT_EQ(MinidumpUTF8StringAtRVAAsString(string_file.string(), 0),
|
|
test_string);
|
|
}
|
|
}
|
|
|
|
struct MinidumpUTF16StringListWriterTraits {
|
|
using MinidumpStringListWriterType = MinidumpUTF16StringListWriter;
|
|
static base::string16 ExpectationForUTF8(const std::string& utf8) {
|
|
return base::UTF8ToUTF16(utf8);
|
|
}
|
|
static base::string16 ObservationAtRVA(const std::string& file_contents,
|
|
RVA rva) {
|
|
return MinidumpStringAtRVAAsString(file_contents, rva);
|
|
}
|
|
};
|
|
|
|
struct MinidumpUTF8StringListWriterTraits {
|
|
using MinidumpStringListWriterType = MinidumpUTF8StringListWriter;
|
|
static std::string ExpectationForUTF8(const std::string& utf8) {
|
|
return utf8;
|
|
}
|
|
static std::string ObservationAtRVA(const std::string& file_contents,
|
|
RVA rva) {
|
|
return MinidumpUTF8StringAtRVAAsString(file_contents, rva);
|
|
}
|
|
};
|
|
|
|
template <typename Traits>
|
|
void MinidumpStringListTest() {
|
|
std::vector<std::string> strings;
|
|
strings.push_back(std::string("One"));
|
|
strings.push_back(std::string());
|
|
strings.push_back(std::string("3"));
|
|
strings.push_back(std::string("\360\237\222\251"));
|
|
|
|
typename Traits::MinidumpStringListWriterType string_list_writer;
|
|
EXPECT_FALSE(string_list_writer.IsUseful());
|
|
string_list_writer.InitializeFromVector(strings);
|
|
EXPECT_TRUE(string_list_writer.IsUseful());
|
|
|
|
StringFile string_file;
|
|
|
|
ASSERT_TRUE(string_list_writer.WriteEverything(&string_file));
|
|
|
|
const MinidumpRVAList* list =
|
|
MinidumpRVAListAtStart(string_file.string(), strings.size());
|
|
ASSERT_TRUE(list);
|
|
|
|
for (size_t index = 0; index < strings.size(); ++index) {
|
|
EXPECT_EQ(
|
|
Traits::ObservationAtRVA(string_file.string(), list->children[index]),
|
|
Traits::ExpectationForUTF8(strings[index]));
|
|
}
|
|
}
|
|
|
|
TEST(MinidumpStringWriter, MinidumpUTF16StringList) {
|
|
MinidumpStringListTest<MinidumpUTF16StringListWriterTraits>();
|
|
}
|
|
|
|
TEST(MinidumpStringWriter, MinidumpUTF8StringList) {
|
|
MinidumpStringListTest<MinidumpUTF8StringListWriterTraits>();
|
|
}
|
|
|
|
} // namespace
|
|
} // namespace test
|
|
} // namespace crashpad
|