diff options
author | kmixter@chromium.org <kmixter@chromium.org@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-12-08 22:26:20 +0000 |
---|---|---|
committer | kmixter@chromium.org <kmixter@chromium.org@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-12-08 22:26:20 +0000 |
commit | b5dfa2834d37a669cce3ca62cb7a56b713d368a2 (patch) | |
tree | ac4713c932d871f6f0c5f67926f4917d3f697045 /src/client/linux/minidump_writer/minidump_writer.cc | |
parent | Propagate failure if http uploads fail with http error codes. (diff) | |
download | breakpad-b5dfa2834d37a669cce3ca62cb7a56b713d368a2.tar.xz |
Reuse code and fix inconsistent array boundaries.
R=ted.mielczarek
Review URL: http://breakpad.appspot.com/237001
git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@740 4c0a9323-5329-0410-9bdc-e9ce6186880e
Diffstat (limited to 'src/client/linux/minidump_writer/minidump_writer.cc')
-rw-r--r-- | src/client/linux/minidump_writer/minidump_writer.cc | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/client/linux/minidump_writer/minidump_writer.cc b/src/client/linux/minidump_writer/minidump_writer.cc index bb365a1e..405ec4e8 100644 --- a/src/client/linux/minidump_writer/minidump_writer.cc +++ b/src/client/linux/minidump_writer/minidump_writer.cc @@ -1216,12 +1216,8 @@ class MinidumpWriter { bool WriteProcFile(MDLocationDescriptor* result, pid_t pid, const char* filename) { - char buf[80]; - memcpy(buf, "/proc/", 6); - const unsigned pid_len = my_int_len(pid); - my_itos(buf + 6, pid, pid_len); - buf[6 + pid_len] = '/'; - memcpy(buf + 6 + pid_len + 1, filename, my_strlen(filename) + 1); + char buf[NAME_MAX]; + dumper_.BuildProcPath(buf, pid, filename); return WriteFile(result, buf); } |