120fe6aef2e9a8baea2d50cd527d1e9138148369 |
|
21-Jan-2016 |
Steve Fung <stevefung@google.com> |
Merge "crash_reporter: Fix unit tests to use ScopedTempDir"
|
78fcf66c1747ed6cc067e764ac3e39e12d6b103b |
|
20-Jan-2016 |
Steve Fung <stevefung@google.com> |
crash_reporter: Fix unit tests to use ScopedTempDir Some of the unit tests manually create their own temp directories from a relative path. This creates a temp folder in the folder that the test is run from. This is a problem when using test_droid or a non-interactive ADB shell command, since the root directory is not writable. Instead use a ScopedTempDir to handle temporary folders. Bug: 26671657 Change-Id: Iaf71d345565dcb4393e80426784a864f7f8b5929 Test: Unit tests pass when running brillo_WhitelistedGtests
/system/core/crash_reporter/user_collector_test.cc
|
ea05ff9229163e6aacfb7f7dffdd76bff559673c |
|
20-Jan-2016 |
Alex Vakulenko <avakulenko@google.com> |
crash_reporter/metricsd: Update libchrome APIs to r369476 The new libchrome has been ported from Chromium and some APIs have changed. Make necessary changes at call sites. Change-Id: I9e45975cdef43913270b82e3fca2c027e614a464
/system/core/crash_reporter/user_collector_test.cc
|
8ed101bab5d01b1ab0c648586c1d5259404b79c4 |
|
03-Dec-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Fix unit tests - Removed obsolete unit tests. - Fix remaining unit tests for Android. Change-Id: I1665447d375178d2bcf941fec3585c8f96cd2bac
/system/core/crash_reporter/user_collector_test.cc
|
74dc62460b8cdd5bfeac47bfe8e759fc04b55ef8 |
|
13-Oct-2015 |
Alex Vakulenko <avakulenko@google.com> |
system/core: Rename "chromeos" -> "brillo" in include paths and namespaces libchromeos is transitioning to libbrillo and chromeos namespaces and include directory is changing to brillo. Bug: 24872993 Change-Id: I797613a38c7444a113f12e38366a424388477276
/system/core/crash_reporter/user_collector_test.cc
|
773fd3c42883b17b5918c71aead5690e16438248 |
|
10-Oct-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Use the actual GID of the crashing process Rather than assuming the UID and GID of crashing processes is the same, report and use the actual GID that the process was running as. Bug: 24678424 Change-Id: I3cfc415be2feb2863a4f4b850bfd4a3267217a44
/system/core/crash_reporter/user_collector_test.cc
|
6c34c2576eb241f134431b2cc847a6c7d70ef012 |
|
20-Aug-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Relicense as Apache 2. MODULE_LICENSE_BSD and NOTICE were removed as the license is no longer different from the rest of system/core. Bug: 23368756 Change-Id: I31e48eb7c3d153c7c5ec4f694c61474dedac2eeb
/system/core/crash_reporter/user_collector_test.cc
|
ab2ac7d11442975f700bbde5702a96d020d82254 |
|
15-Aug-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Enable core dumps Fix various path/directory locations to their Android equivalents to enable collection of core dumps and generation of mini dumps to work correctly. Also add the init script to initialize crash_reporter. Bug: 22874832 Change-Id: Iffb1529e5259c5da5ba7f6977b2787e738f68a78
/system/core/crash_reporter/user_collector_test.cc
|
8bafb3da8fe46993f995054ec84c75f82e6bb2d8 |
|
07-Aug-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Fix initial compile issues with Android toolchain Fix some issues to get crash_reporter to compile on Android. C++ source: * bits/wordsize.h is used for __WORDSIZE, but is unavailable for most Android toolchain architectures. It instead is available in sys/cdefs.h Flex source: * The Flex compiler toolchain doesn't link against the default fl library using -lfl. Thus there is no yywrap() function defined. Since the default fl library implementation just returns 1 (true), instead add a directive to disable yywrap. * The index(..) function has been deprecated in favor of strchr(..) * Add a directive to disable the "deprecated conversion from string constant to 'char*'" warning in the generated code. * Remove warning suppression for the input() function which doesn't get generated with Flex v2.5.39 and the arguments passed in the build command. Bug: 22874187 Change-Id: I6d7dcf5f801010aeae06db9f0628a6652ee269ee
/system/core/crash_reporter/user_collector_test.cc
|
c8b741428caeb01f7aca3c4c199e5f6894d335f8 |
|
06-Aug-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Remove Chrome collector, CrOS build files, DBus Removed the ChromeOS specific code so that the code compiles. The code removed is not needed when running on Android, and includes: * Chrome collector * ChromeOS build's gyp file * ChromeOS-specific DBus calls * ChromeOS-specific collector path logic * Chrome bypass logic in user collector Bug: 22672752 Change-Id: I3c7d87c971a181d8f73293519318e3602d142927
/system/core/crash_reporter/user_collector_test.cc
|
129bea543b7572bfdf09c6a7b3cebbe3b57ce723 |
|
23-Jul-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Fix header include paths Fix the header include paths to be relative to the crash_reporter base folder. Bug: 22873331 Change-Id: Icdd0495a79cd7679f38c54a84e189bfb3c9d7f1c
/system/core/crash_reporter/user_collector_test.cc
|
6e13952269ad729594f2670b889bd806e9b0f3b3 |
|
05-Feb-2015 |
Steve Fung <stevefung@chromium.org> |
crash: Remove glib from crash_reporter As part of the minimization effort, refactor crash_reporter code to not depend directly on glib. BUG=brillo:87, brillo:88, chromium:435314 TEST=`FEATURES=test emerge-panther libchromeos debugd crash-reporter` TEST=Enabled crash reports; Browsed to chrome://crash; crash files \ generated; `FORCE_OFFICIAL=1 SECONDS_SEND_SPREAD=1 crash_sender` \ /var/log/messages shows crash id, report shows all expected files TEST=`cbuildbot --remote -p chromiumos/platform2 amd64-generic-full` CQ-DEPEND=I00331e0bf29195b41cd84d4495ab47738a5a41de CQ-DEPEND=I9df752d8995773adb56fab34dd97626f3ddf1765 Change-Id: I48b366198a7f89ca55259603cf8470e4d59321bf Reviewed-on: https://chromium-review.googlesource.com/246441 Reviewed-by: Dan Erat <derat@chromium.org> Tested-by: Steve Fung <stevefung@chromium.org> Commit-Queue: Steve Fung <stevefung@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
d257ea1cbeb0d61fe73a2aef8f3f8feca5402a3c |
|
28-Jan-2015 |
Daniel Erat <derat@chromium.org> |
crash-reporter: Clean up testing code. Link all of the crash reporter's tests into a single crash_reporter_test executable instead of building separate binaries that lead to a bunch of duplication. BUG=none TEST=tests pass CQ-DEPEND=I7f6623dd0ae36395efac89bdcfae54926ea6c918 Change-Id: I18e2d929d6545e7d05e88232bdaec089ae620544 Reviewed-on: https://chromium-review.googlesource.com/243940 Reviewed-by: Dan Erat <derat@chromium.org> Commit-Queue: Dan Erat <derat@chromium.org> Trybot-Ready: Dan Erat <derat@chromium.org> Tested-by: Dan Erat <derat@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
d6169a2a4f775da37ffa8b8ba4d767efd0377343 |
|
12-Aug-2014 |
Steve Fung <stevefung@chromium.org> |
crash-reporter: remove gflags dependency We are switching to using chromeos/flag_helper.h instead to standardize the code everywhere. BUG=chromium:402631 TEST=`FEATURES=test emerge-panther crash-reporter` TEST=`test_that -b panther <ip> e:logging_.*` Change-Id: I62b911a54f89d853235d0540460cbea119f66e9e Reviewed-on: https://chromium-review.googlesource.com/212140 Reviewed-by: Mike Frysinger <vapier@chromium.org> Tested-by: Steve Fung <stevefung@chromium.org> Commit-Queue: Steve Fung <stevefung@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
262d798b380a59ef691ede4c6a32aa71ff191295 |
|
18-Sep-2014 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Replace NULL with nullptr. BUG=None TEST=`FEATURES=test emerge-$BOARD crash-reporter` Change-Id: If0804613ee0385752d01f7bbe01902ffef53bd94 Reviewed-on: https://chromium-review.googlesource.com/218870 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
ab6cc90503ca2db976a3cb9c9382a9da85c4b5a2 |
|
05-Sep-2014 |
Ben Chan <benchan@chromium.org> |
Update code to include base/files/file_util.h file_util.h was moved from base to base/files (https://codereview.chromium.org/468253002). This CL updates platform2 code to include base/files/file_util.h instead of base/file_util.h. BUG=chromium:411001 TEST=Trybot run on paladin, release, and chromiumos-sdk builders. Change-Id: I488925b54615e131e508a460dc1a27f88168f936 Reviewed-on: https://chromium-review.googlesource.com/216851 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Tested-by: Ben Chan <benchan@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
7e77690375bc8a896a8de318d69d515e67c7aefe |
|
18-Jun-2014 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Fix coding style and cpplint issues. BUG=chromium:385849 TEST=`FEATURES=test emerge-$BOARD platform2` TEST=`cpplint.py --filter=--build/header_guard,-build/include_order *.h *.cc` Change-Id: I3c24dd9487df50cdb22fc1c7739c9e95e452afae Reviewed-on: https://chromium-review.googlesource.com/204487 Tested-by: Ben Chan <benchan@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
f30c641225e55055641ffe3fd679a96e44f34af6 |
|
23-May-2014 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Update to build against libchrome-271506. BUG=chromium:375032 TEST=`FEATURES=test emerge-$BOARD platform2` Change-Id: I3984dee513690711e277e5fa8939ca5f7910d180 Reviewed-on: https://chromium-review.googlesource.com/201184 Tested-by: Ben Chan <benchan@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
9b1f300139689eb9c7b7a35e91a4fbc1eab93b4e |
|
24-Apr-2014 |
Lei Zhang <thestig@chromium.org> |
crash-reporter: Write out a magic string for Chrome crashes. Also do some much needed code cleanup. BUG=chromium:363660 TEST=emerge platform2 Change-Id: Ica9abfd854e2c77d970851805989c86a6a45fdee Reviewed-on: https://chromium-review.googlesource.com/196764 Commit-Queue: Lei Zhang <thestig@chromium.org> Tested-by: Lei Zhang <thestig@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
bfdf4a8913c805bd74723aa1daf9eeeaa971e590 |
|
14-Mar-2014 |
Mike Frysinger <vapier@chromium.org> |
user_collector: use %P instead of %p The latter will give us the PID as seen in the current PID namespace, but since the crash handler always runs in the top most PID namespace, that is meaningless to us. Instead, use the %P option so we see the PID in the top most namespace. BUG=chromium:351568 TEST=`cbuildbot amd64-generic-full` passes # linux-3.10 TEST=`cbuildbot x86-alex-release` passes # linux-3.4 TEST=`cbuildbot link-release` passes # linux-3.8 CQ-DEPEND=CL:190011 CQ-DEPEND=CL:190020 CQ-DEPEND=CL:190012 CQ-DEPEND=CL:190021 Change-Id: Ic619c4e7514e2824f7ba31b73cb9e11ec4092774 Reviewed-on: https://chromium-review.googlesource.com/190010 Reviewed-by: Ben Chan <benchan@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
a557c1187ff19d422db2a9c951ecd8f7243e79bd |
|
06-Feb-2014 |
Mike Frysinger <vapier@chromium.org> |
update to libbase-242728 BUG=chromium:331128 TEST=`FEATURES=test emerge-x86-alex platform2` works TEST=`cbuildbot chromiumos-sdk` works CQ-DEPEND=CL:185131 Change-Id: Ia03a6ea7aaf6b4ee5d9c512ebf7080c0c28920f1 Reviewed-on: https://chromium-review.googlesource.com/185150 Reviewed-by: Ben Chan <benchan@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
83ce73eac51b219dc1d112683149c486722e65c5 |
|
24-Jan-2014 |
Mike Frysinger <vapier@chromium.org> |
remove unused gflags includes These files don't use gflags, so drop the includes. BUG=chromium:337753 TEST=`FEATURES=test emerge-x86-alex crash-reporter` passes Change-Id: I0a57e45cd16cd37b1361858e3313d59dced4eb5e Reviewed-on: https://chromium-review.googlesource.com/183644 Reviewed-by: Ben Chan <benchan@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
d2db5ff98889fbe0bb75bcf34c3ef31672bb38e4 |
|
09-Oct-2013 |
Mike Frysinger <vapier@chromium.org> |
crash_reporter: clarify chrome crash ignore message We ignore Chrome crashes when it is the kernel that runs us. When Chrome itself runs us, we will take their crashes. The log output here though has long confused developers, so rephrase it slightly. BUG=None TEST=`cros_run_unit_tests -p crash-reporter --board x86-alex` passes TEST=`cbuildbot {amd64,x86}-generic-full` passes (VMTests) Change-Id: Ia13a766134f5a20a1cc4eaab74ac46633d3177eb Reviewed-on: https://chromium-review.googlesource.com/172261 Reviewed-by: Chris Masone <cmasone@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
9f90acaa4d420530d7b4ddd37112518df68e373a |
|
20-Feb-2013 |
Simon Que <sque@chromium.org> |
Replace FilePath declaration with file_path.h include Fixes a clang error. Also use "base::FilePath" since FilePath has been moved to the base namespace. BUG=chromium-os:39046 TEST=The following command passes: USE="chrome_internal" CFLAGS="-clang -print-cmdline" CXXFLAGS="-clang -print-cmdline" emerge-x86-alex crash-reporter Change-Id: I7da82093c5685b5b556cba971b1e1b14ac0f59c4 Signed-off-by: Simon Que <sque@chromium.org> Reviewed-on: https://gerrit.chromium.org/gerrit/43577 Reviewed-by: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
1a8780d309b980914a5e2403bf9addafe07386ec |
|
15-Feb-2013 |
Mike Frysinger <vapier@chromium.org> |
update to libchrome 180609 Purely mechanical changes here. BUG=chromium-os:38933 TEST=`cros_run_unit_tests --board=stumpy -p crash-reporter` passes CQ-DEPEND=CL:43376 Change-Id: I62d5d12556517bedf48d36ada4c651ff1c671fe9 Reviewed-on: https://gerrit.chromium.org/gerrit/43378 Reviewed-by: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
1c57e9e330bc4c6a44955fd649ee9a4badb9e1dc |
|
26-Sep-2012 |
Michael Krebs <mkrebs@chromium.org> |
crash-reporter: Remove some dependency on /proc Make the existence of a crashing process's /proc/<PID>/ directory optional for all of crash_reporter, except for core2md. In particular, have the kernel pass the UID of the crashing process to crash_reporter in case the "status" file is unavailable. BUG=chromium-os:34385 TEST=Ran unittests and autotests CQ-DEPEND=I6048d3eb84f8188bee6a755eaa010510f5d2459b Change-Id: I62df52cab44cf1febc7ed3e55b75bcffa0daf524 Reviewed-on: https://gerrit.chromium.org/gerrit/34078 Commit-Queue: David James <davidjames@chromium.org> Reviewed-by: David James <davidjames@chromium.org> Tested-by: David James <davidjames@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
2f3ed034f378e99f27bd733b6a452df47739117c |
|
22-Aug-2012 |
Michael Krebs <mkrebs@chromium.org> |
crash-reporter: Also ignore renamed Chrome threads At some point Chrome started naming its threads such that crashes report a different executable name. We've been getting a *lot* of crash-reporter errors with names like "supplied_Compositor" in particular, so consider those to be Chrome crashes that should be ignored. Unfortunately, the thread names can be arbitrary. With this CL we check the entire range of possible names. Maybe someday someone can use the core file to determine that a process was originally named "chrome". BUG=chrome-os-partner:12045 TEST=Ran unittests Change-Id: Ia82d619bd1ee4367129640dc6c7be5ce258a68bb Reviewed-on: https://gerrit.chromium.org/gerrit/31084 Commit-Ready: Michael Krebs <mkrebs@chromium.org> Tested-by: Michael Krebs <mkrebs@chromium.org> Reviewed-by: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
57b261cf04f45d67b02f6bea8ad2c37734da86e0 |
|
11-Apr-2012 |
Mike Frysinger <vapier@chromium.org> |
update to newer libbase BUG=chromium-os:25872 TEST=`emerge-x86-alex crash-reporter` works TEST=`cros_run_unit_tests --board=x86-alex -p crash-reporter` passed Change-Id: Iff428b1364f96533d2d2290272823c2c84c48933 Reviewed-on: https://gerrit.chromium.org/gerrit/20008 Commit-Ready: Mike Frysinger <vapier@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
6e709a11031c80c96f47da50b3231074334f462c |
|
29-Feb-2012 |
Ben Chan <benchan@chromium.org> |
Improve error diagnostics in UserCollector. This CL makes the following changes to improve error diagnostics in UserCollector: 1. Add an error type to describe why the crash reporter fails to collects a user crash, which is reported to the crash server along with the "crash_reporter-user-collection" error signature. 2. Perform some preliminary validations of the core file before converting it to a minidump file using core2md. 3. Identify the case when a 32-bit core file is produced on a 64-bit platform as core2md currently cannot handle 32-bit core files on a 64-bit platform. BUG=chromium-os:7871 TEST=Tested the following: 1. emerge-{x86,amd64,arm}-generic crash-reporter 2. FEATURES="test" emerge-{x86,amd64,arm}-generic crash-reporter 3. Run the following autotest tests on x86-mario and stumpy64: - logging_CrashSender - logging_UserCrash Change-Id: Ib50b4cc81a91f7cd75f9440005200d4027ce1f6f Reviewed-on: https://gerrit.chromium.org/gerrit/17166 Reviewed-by: Michael Krebs <mkrebs@chromium.org> Tested-by: Ben Chan <benchan@chromium.org> Commit-Ready: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
f13bb58358b879258c5c0eb48c6e09895d394ef1 |
|
06-Jan-2012 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Validate proc files and log the state of crashed process. This CL makes the following changes to UserCollector: 1. Check if the maps file is empty after copying proc files to the temporary container directory, which prevents the core-to-minidump conversion failure due to unusable maps file. 2. Obtain and log the state of the crashed process, which helps determine if the crashed process is in the zombie state. 3. Refactor the status file processing code. BUG=chromium-os:24820 TEST=Tested the following: 1. Build crash-reporter and run unit tests. 2. Run the following autotest tests on a Cr48: - logging_CrashSender - logging_UserCrash 3. Check that the process state of a crashed process is logged in /var/log/messages. Change-Id: Iebaf9c36c18185c703c18e7028ee4673dd9ebc93 Reviewed-on: https://gerrit.chromium.org/gerrit/13826 Reviewed-by: Ken Mixter <kmixter@chromium.org> Tested-by: Ben Chan <benchan@chromium.org> Commit-Ready: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
4fe30db1405498dd21162436eda081486f400fa8 |
|
05-Aug-2011 |
Michael Krebs <mkrebs@chromium.org> |
crash-reporter: Dump crashes outside of cryptohome during autotests. If the file "/mnt/stateful_partition/etc/collect_chrome_crashes" exists on the target, dump crashes to /var/spool/crash/ instead of /home/chronos/user/crash/. This is so they are easily available even after logout. BUG=chromium-os:18637 TEST=Ran unit and remote tests. Change-Id: I340bbb0c1772123192c8bb87872bcda53fae5524 Reviewed-on: http://gerrit.chromium.org/gerrit/5419 Reviewed-by: Ken Mixter <kmixter@chromium.org> Tested-by: Michael Krebs <mkrebs@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
538ecbf7d5f4011440c45b27e84902f5953b1f82 |
|
27-Jul-2011 |
Michael Krebs <mkrebs@chromium.org> |
Don't ignore chrome crashes when autotesting. If the file "/mnt/stateful_partition/etc/collect_chrome_crashes" exists on the target, allow chrome crashes to be reported by crash reporter. The intention is that the autotest system will create this file. This is a relatively temporary fix to help track down chrome problems, until the crashes can be handled by crash reporter in the field as well. BUG=chromium-os:17987, chromium-os:17898 TEST=Ran unit tests. Change-Id: I68a584a0b861669117d8e97f5687b4c8fc876011 Reviewed-on: http://gerrit.chromium.org/gerrit/4861 Reviewed-by: Ken Mixter <kmixter@chromium.org> Tested-by: Ken Mixter <kmixter@chromium.org>
/system/core/crash_reporter/user_collector_test.cc
|
5d3a1a2c3543db21913f67cf240bad6c137063c9 |
|
16-Mar-2011 |
Ken Mixter <kmixter@google.com> |
crash-reporter: keep ignoring chrome crashes even on dev builds Change-Id: Idd8859958e626dc782e511f35fca5d45a2977f53 BUG=chromium-os:12911 TEST=unit tests and UserCrash Review URL: http://codereview.chromium.org/6673002
/system/core/crash_reporter/user_collector_test.cc
|
a324932e066ccb7ed9d62db9c409ee2c10052554 |
|
03-Mar-2011 |
Ken Mixter <kmixter@google.com> |
crash-reporter: Use standard logging and new libchromeos Process code Change-Id: Ie085d61d4d79c0df38e51debaa044d701a860c89 BUG=none TEST=unit tests / UserCrash / CrashSender Review URL: http://codereview.chromium.org/6517001
/system/core/crash_reporter/user_collector_test.cc
|
d49d362f61c7e702d557e563055b1c541ead5b67 |
|
10-Feb-2011 |
Ken Mixter <kmixter@google.com> |
crash-reporter: when exe symlink read fails send diags and still ignore chrome crashes Change-Id: I0f43d5f61936ebb6420c357e58df45761facd4d0 BUG=6299 TEST=unit tests and UserCrash/CrashSender Review URL: http://codereview.chromium.org/6480009
/system/core/crash_reporter/user_collector_test.cc
|
1b8fe015585c0b966d5240b8e22eba1af8c69e88 |
|
25-Jan-2011 |
Ken Mixter <kmixter@chromium.org> |
crash-reporter: Add diagnostics to help diagnose failures in the wild We add logging messages that are generated during invocation of core2md, but we also enable sending arbitrary system info based on the configuration file. In this case we add the list of running processes, meminfo, and dmesg. Change-Id: Ifdf29b89dd60d56349fa39095d2aa07f6b5e2de2 BUG=chromium-os:6299,chromium-os:9345 TEST=UserCrash, unit tests Review URL: http://codereview.chromium.org/6297004
/system/core/crash_reporter/user_collector_test.cc
|
c49dbd4775986f32b2f09659595f9f28ef7f6b44 |
|
15-Dec-2010 |
Ken Mixter <kmixter@chromium.org> |
crash-reporter: Capture and send recent update_engine logs when it crashes We also disallow more than 4 nested crashes. This way we know if core2md crashes for instance, but not if crash_reporter crashes while handling the core2md crash that came from crash_reporter. Note that the kernel does not know about parent/child process relationships, it just counts concurrent handling, so it is possible that some of many crashing processes will be discarded. See crash report id 473c22ed428d33a9 for an example report with update_engine logs. Change-Id: I9ff9f03a94dc87d2d42840511c2e5e42ee37cea8 BUG=9176 TEST=UserCrash,CrashSender,unit tests Review URL: http://codereview.chromium.org/5814001
/system/core/crash_reporter/user_collector_test.cc
|
2105b49796f2b753b44ccc792fd387dd0dddf54f |
|
10-Nov-2010 |
Ken Mixter <kmixter@chromium.org> |
crash-reporter: Ignore Chrome crashes, relying on Chrome breakpad Change-Id: I08e44d2c8b6be839d18efd2fe6e8b612a61ae723 BUG=8841 TEST=bvts Review URL: http://codereview.chromium.org/4499006
/system/core/crash_reporter/user_collector_test.cc
|
9b346474538a255bc7144ae358bb0ee129163d58 |
|
07-Nov-2010 |
Ken Mixter <kmixter@chromium.org> |
crash-reporter: Avoid writing through symlinks. BUG=7987 TEST=bvts Change-Id: I875adeb5073936e790beb93f6a223a1642131cbd Review URL: http://codereview.chromium.org/4603001
/system/core/crash_reporter/user_collector_test.cc
|
afcf80821c57a189b53b7a66f76d13855d63821e |
|
26-Oct-2010 |
Ken Mixter <kmixter@chromium.org> |
crash-reporter: Generate kernel crash signatures for server-side grouping of similar crashes BUG=5868 TEST=KernelCrash, UserCrash, CrashSender, unittests, and manual inspection of sent report. Change-Id: I31991895c9ac719ac1832d588ae3360500ef0c26 Review URL: http://codereview.chromium.org/4018008
/system/core/crash_reporter/user_collector_test.cc
|
2953c3a87a7b201bbf08555129f0640ede1d0cdb |
|
18-Oct-2010 |
Ken Mixter <kmixter@google.com> |
crash-reporter: Avoid using system to invoke core2md Change-Id: I1fc0b4dd6e4e84cb999ba61fedd43fc74a8fa1ba BUG=6941 TEST=bvts Review URL: http://codereview.chromium.org/3755011
/system/core/crash_reporter/user_collector_test.cc
|
0340316050044e0995b98fea87ed41ea77abb28b |
|
19-Aug-2010 |
Ken Mixter <kmixter@google.com> |
Collect and send kernel crash diagnostics BUG=1512,1914 Review URL: http://codereview.chromium.org/3179006
/system/core/crash_reporter/user_collector_test.cc
|
777484c41cc63e2f97f41a443d1d55f7e6037a5e |
|
24-Jul-2010 |
Ken Mixter <kmixter@google.com> |
Start invoking core2md to implement full system crash reporting BUG=4741 Review URL: http://codereview.chromium.org/3040013
/system/core/crash_reporter/user_collector_test.cc
|
522c061f831325e73e3d46ee2161238ad6fa8b22 |
|
18-Jun-2010 |
Chris Sosa <sosa@chromium.org> |
Change include paths to reflect new directory. TEST=Tested with new ebuild. Review URL: http://codereview.chromium.org/2819009
/system/core/crash_reporter/user_collector_test.cc
|
e4a86037f405415c460ace194fbf16cfd93d4c6f |
|
17-Jun-2010 |
Chris Sosa <sosa@chromium.org> |
Initial commit of crash reporter repo
/system/core/crash_reporter/user_collector_test.cc
|