Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Unified Diff: src/client/linux/handler/minidump_descriptor.cc

Issue 1125153008: [microdump] Add build fingerprint and product info metadata. (Closed) Base URL: http://google-breakpad.googlecode.com/svn/trunk
Patch Set: Addressing changes from https://breakpad.appspot.com/9794002 Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/client/linux/handler/minidump_descriptor.cc
diff --git a/src/client/linux/handler/minidump_descriptor.cc b/src/client/linux/handler/minidump_descriptor.cc
index 8c68bde21dc46222cec8c14b80d7adbe7f6f2bc8..c601d35f04d8a2283386c5e1bb4be1bcad672521 100644
--- a/src/client/linux/handler/minidump_descriptor.cc
+++ b/src/client/linux/handler/minidump_descriptor.cc
@@ -44,7 +44,9 @@ MinidumpDescriptor::MinidumpDescriptor(const MinidumpDescriptor& descriptor)
fd_(descriptor.fd_),
directory_(descriptor.directory_),
c_path_(NULL),
- size_limit_(descriptor.size_limit_) {
+ size_limit_(descriptor.size_limit_),
+ microdump_build_fingerprint_(descriptor.microdump_build_fingerprint_),
+ microdump_product_info_(descriptor.microdump_product_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
// can cause problems in compromised environments.
@@ -65,6 +67,8 @@ MinidumpDescriptor& MinidumpDescriptor::operator=(
UpdatePath();
}
size_limit_ = descriptor.size_limit_;
+ microdump_build_fingerprint_ = descriptor.microdump_build_fingerprint_;
+ microdump_product_info_ = descriptor.microdump_product_info_;
return *this;
}
@@ -82,4 +86,15 @@ void MinidumpDescriptor::UpdatePath() {
c_path_ = path_.c_str();
}
+void MinidumpDescriptor::SetMicrodumpBuildFingerprint(
+ const char* build_fingerprint) {
+ assert(mode_ == kWriteMicrodumpToConsole);
+ microdump_build_fingerprint_ = build_fingerprint;
+}
+
+void MinidumpDescriptor::SetMicrodumpProductInfo(const char* product_info) {
+ assert(mode_ == kWriteMicrodumpToConsole);
+ microdump_product_info_ = product_info;
+}
+
} // namespace google_breakpad
« no previous file with comments | « src/client/linux/handler/minidump_descriptor.h ('k') | src/client/linux/microdump_writer/microdump_writer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698