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

Side by Side Diff: src/processor/minidump_processor.cc

Issue 1833123002: Make EXC_BAD_ACCESS / EXC_I386_GPFLT print nicely in the processor (Closed) Base URL: https://chromium.googlesource.com/breakpad/breakpad.git@master
Patch Set: Created 4 years, 9 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006, Google Inc. 1 // Copyright (c) 2006, Google Inc.
2 // All rights reserved. 2 // All rights reserved.
3 // 3 //
4 // Redistribution and use in source and binary forms, with or without 4 // Redistribution and use in source and binary forms, with or without
5 // modification, are permitted provided that the following conditions are 5 // modification, are permitted provided that the following conditions are
6 // met: 6 // met:
7 // 7 //
8 // * Redistributions of source code must retain the above copyright 8 // * Redistributions of source code must retain the above copyright
9 // notice, this list of conditions and the following disclaimer. 9 // notice, this list of conditions and the following disclaimer.
10 // * Redistributions in binary form must reproduce the above 10 // * Redistributions in binary form must reproduce the above
(...skipping 727 matching lines...) Expand 10 before | Expand all | Expand 10 after
738 reason.append("EXC_PPC_BADSPACE"); 738 reason.append("EXC_PPC_BADSPACE");
739 break; 739 break;
740 case MD_EXCEPTION_CODE_MAC_PPC_UNALIGNED: 740 case MD_EXCEPTION_CODE_MAC_PPC_UNALIGNED:
741 reason.append("EXC_PPC_UNALIGNED"); 741 reason.append("EXC_PPC_UNALIGNED");
742 break; 742 break;
743 default: 743 default:
744 reason.append(flags_string); 744 reason.append(flags_string);
745 BPLOG(INFO) << "Unknown exception reason " << reason; 745 BPLOG(INFO) << "Unknown exception reason " << reason;
746 break; 746 break;
747 } 747 }
748 } else if (raw_system_info->processor_architecture ==
749 MD_CPU_ARCHITECTURE_X86 ||
750 raw_system_info->processor_architecture ==
751 MD_CPU_ARCHITECTURE_AMD64) {
752 switch (exception_flags) {
753 case MD_EXCEPTION_CODE_MAC_X86_GENERAL_PROTECTION_FAULT:
754 reason.append("EXC_GPFLT");
Mark Mentovai 2016/03/25 14:52:24 EXC_I386_GPFLT. Fix the change description too.
Ted Mielczarek 2016/03/25 15:46:44 I copied this from: https://chromium.googlesource.
755 break;
756 default:
757 reason.append(flags_string);
758 BPLOG(INFO) << "Unknown exception reason " << reason;
759 break;
760 }
748 } else { 761 } else {
749 reason.append(flags_string); 762 reason.append(flags_string);
750 BPLOG(INFO) << "Unknown exception reason " << reason; 763 BPLOG(INFO) << "Unknown exception reason " << reason;
751 } 764 }
752 break; 765 break;
753 } 766 }
754 break; 767 break;
755 case MD_EXCEPTION_MAC_BAD_INSTRUCTION: 768 case MD_EXCEPTION_MAC_BAD_INSTRUCTION:
756 reason = "EXC_BAD_INSTRUCTION / "; 769 reason = "EXC_BAD_INSTRUCTION / ";
757 switch (raw_system_info->processor_architecture) { 770 switch (raw_system_info->processor_architecture) {
(...skipping 778 matching lines...) Expand 10 before | Expand all | Expand 10 after
1536 char assertion_line[32]; 1549 char assertion_line[32];
1537 snprintf(assertion_line, sizeof(assertion_line), "%u", raw_assertion->line); 1550 snprintf(assertion_line, sizeof(assertion_line), "%u", raw_assertion->line);
1538 assertion_string.append(" at line "); 1551 assertion_string.append(" at line ");
1539 assertion_string.append(assertion_line); 1552 assertion_string.append(assertion_line);
1540 } 1553 }
1541 1554
1542 return assertion_string; 1555 return assertion_string;
1543 } 1556 }
1544 1557
1545 } // namespace google_breakpad 1558 } // namespace google_breakpad
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698