Change Print method of MinidumpLinuxMaps and MinidumpLinuxMapsList to print

contents of /proc/<pid>/maps instead of just the files mapped to memory.

Review URL: https://codereview.chromium.org/1273123002

git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@1481 4c0a9323-5329-0410-9bdc-e9ce6186880e
This commit is contained in:
Liu.andrew.x@gmail.com 2015-08-11 16:05:48 +00:00
parent 47527e48e5
commit 6a1a906773
3 changed files with 5 additions and 1 deletions

View file

@ -45,6 +45,9 @@ struct MappedMemoryRegion {
// "[heap]" and "[stack]" are used to represent the location of the process' // "[heap]" and "[stack]" are used to represent the location of the process'
// heap and stack, respectively. // heap and stack, respectively.
string path; string path;
// The line from /proc/<pid>/maps that this struct represents.
string line;
}; };
// Parses /proc/<pid>/maps input data and stores in |regions|. Returns true // Parses /proc/<pid>/maps input data and stores in |regions|. Returns true

View file

@ -4004,7 +4004,7 @@ void MinidumpLinuxMaps::Print() {
BPLOG(ERROR) << "MinidumpLinuxMaps cannot print invalid data"; BPLOG(ERROR) << "MinidumpLinuxMaps cannot print invalid data";
return; return;
} }
std::cout << GetPathname() << std::endl; std::cout << region_.line << std::endl;
} }
// //

View file

@ -94,6 +94,7 @@ bool ParseProcMaps(const std::string& input,
// Pushing then assigning saves us a string copy. // Pushing then assigning saves us a string copy.
regions.push_back(region); regions.push_back(region);
regions.back().path.assign(line + path_index); regions.back().path.assign(line + path_index);
regions.back().line.assign(line);
} }
regions_out->swap(regions); regions_out->swap(regions);