From e7dfafc16e9984d849a781c92154fd086e6e8650 Mon Sep 17 00:00:00 2001 From: Tobias Sargeant Date: Thu, 19 Jan 2017 16:25:10 +0000 Subject: [PATCH] Add API to skip dump if crashing thread doesn't reference a given module (2) Follow-up CL to add relevant code to the copy constructor and assignment operator for MinidumpDescriptor BUG=664460 Change-Id: I71c0ad01d8686a9215a718cebc9d11a215ea342c Reviewed-on: https://chromium-review.googlesource.com/430711 Reviewed-by: Robert Sesek --- src/client/linux/handler/minidump_descriptor.cc | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/client/linux/handler/minidump_descriptor.cc b/src/client/linux/handler/minidump_descriptor.cc index ce09153d..cdb5bf03 100644 --- a/src/client/linux/handler/minidump_descriptor.cc +++ b/src/client/linux/handler/minidump_descriptor.cc @@ -45,6 +45,10 @@ MinidumpDescriptor::MinidumpDescriptor(const MinidumpDescriptor& descriptor) directory_(descriptor.directory_), c_path_(NULL), size_limit_(descriptor.size_limit_), + address_within_principal_mapping_( + descriptor.address_within_principal_mapping_), + skip_dump_if_principal_mapping_not_referenced_( + descriptor.skip_dump_if_principal_mapping_not_referenced_), microdump_extra_info_(descriptor.microdump_extra_info_) { // The copy constructor is not allowed to be called on a MinidumpDescriptor // with a valid path_, as getting its c_path_ would require the heap which @@ -66,6 +70,10 @@ MinidumpDescriptor& MinidumpDescriptor::operator=( UpdatePath(); } size_limit_ = descriptor.size_limit_; + address_within_principal_mapping_ = + descriptor.address_within_principal_mapping_; + skip_dump_if_principal_mapping_not_referenced_ = + descriptor.skip_dump_if_principal_mapping_not_referenced_; microdump_extra_info_ = descriptor.microdump_extra_info_; return *this; }