• R/O
  • HTTP
  • SSH

thirdparty-breakpad: Listed des commits

Breakpad, a crash reporter, from Google.

Original home: https://chromium.googlesource.com/breakpad/breakpad/

Révision l'heure Auteur
2e26639 2016-04-14 23:32:20 Ted Mielczarek

Fix DWARF handling of inlined functions in namespaces

Currently an inlined function in a namespace in DWARF will
be given a name comprised of just `namespace::`. This is due
to a logic error in ComputeQualifiedName, where it doesn't
handle an empty `unqualified_name` properly.

We apparently have a fair number of these in our Mac builds,
an example of the DWARF that's being mishandled looks like:
0x117eda40: TAG_namespace [5] *
AT_name( "js" )
AT_decl_file( "../../dist/include/js/Utility.h" )
AT_decl_line( 35 )

0x11808500: TAG_subprogram [251] *
AT_low_pc( 0x0000000002f12110 )
AT_high_pc( 0x0000000002f1216b )
AT_APPLE_omit_frame_ptr( 0x01 )
AT_frame_base( rsp )
AT_abstract_origin( {0x0000000011800a4f}"_ZN2js40TraceManuallyBarrieredGenericPointerEdgeEP8JSTracerPPNS_2gc4CellEPKc" )
AT_MIPS_linkage_name( "_ZN2js40TraceManuallyBarrieredGenericPointerEdgeEP8JSTracerPPNS_2gc4CellEPKc" )
AT_name( "TraceManuallyBarrieredGenericPointerEdge" )
AT_decl_file( "/builds/slave/rel-m-rel-m64_bld-000000000000/build/js/src/gc/Marking.cpp" )
AT_decl_line( 547 )
AT_external( 0x01 )
AT_APPLE_optimized( 0x01 )
AT_inline( DW_INL_inlined )

This turned a few instances of this in the file I was testing on into
`<name omitted>`, which seems to just be a symptom of the
"DW_AT_abstract_origin comes later in the file" issue. (Which is probably
also worth fixing given that it occurs some 29k times when dumping
symbols from Firefox's XUL binary, but it's a separate issue.)


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

fee47f4 2016-04-14 01:15:15 Ted Mielczarek

Add some new stream types to MDStreamType

I ran minidump_dump on a dump from Firefox on my Windows 10 machine
and noticed some streams that Breakpad didn't have names for.
Looking in minidumpapiset.h in the Windows 10 SDK finds these values
in MINIDUMP_STREAM_TYPE. There are also struct definitions for the
stream data for some of them (all but JavaScriptData), but I don't have
a particular need for those currently.


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

0203b0c 2016-04-13 03:55:52 Ted Mielczarek

Add travis CI config

This will let us setup travis-ci on the Breakpad GitHub mirror.

R=vapier@chromium.org, mark@chromium.org

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

205b5ac 2016-04-13 03:55:18 Ted Mielczarek

Fix a dependency issue in automake

Doing a `make -jN check` from a fresh build breaks (and has probably been
broken for a while). linux_client_unittest_shlib is missing $(TEST_LIBS)
from its _DEPENDENCIES. The automake manual says if _DEPENDENCIES are not
specified they'll be computed from _LDADD, but we are specifying it and just
leaving out $(TEST_LIBS).


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

2f1b7ad 2016-04-12 02:05:44 Mike Frysinger

tweak IGNORE_RET behavior a bit

GCC will still warn about unused return value with the form:
if (write(...));

Instead, change the semi-colon to an empty set of braces.

TEST=build+test still works

b0e5f26 2016-04-08 11:37:45 David Yen

Added an option (-i) to have dump_syms output header information only.

It is often helpful to check if a particular symbol file dumped by
dump_syms actually matches a version of a binary file we have. The
symbol output contains an ID which can be used to see if it matches
the binary file. Unfortunately, this ID is internally calculated
and not a standard hash of the binary file. Being able to output the
header information only will allow users to determine whether their
symbol file is up to date or not.


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

Patch from David Yen <dyen@chromium.org>.

32901f6 2016-04-06 08:37:13 Yunxiao Ma

Remove unreferenced local variable which breaks build.

Depending on compiler's setting, the unreferenced local variable may
cause build break.

modified: src/processor/minidump.cc


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

Patch from Yunxiao Ma <yxma@google.com>.

4635927 2016-04-06 04:45:30 Yunxiao Ma

Rename stdio.h wrapper file to stdio_wrapper.h.

Some projects will get build break because the comipler is confused when
searches for the standard stdio.h. Rename the wrapper file to avoid that.

renamed: src/common/stdio.h -> src/common/stdio_wrapper.h
modified: src/processor/minidump.cc
modified: src/processor/dump_context.cc
modified: src/processor/logging.cc
modified: src/processor/minidump.cc
modified: src/processor/minidump_processor.cc
modified: src/processor/stackwalk_common.cc
modified: src/processor/symbolic_constants_win.cc

R=mark@chromium.org, labath@google.com

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

Patch from Yunxiao Ma <yxma@google.com>.

6c8f80a 2016-04-05 22:34:20 Ted Mielczarek

Switch the Linux minidump writer to use MDCVInfoELF for CV data.

This preserves full build ids in minidumps, which are useful for
tracking down the right version of system libraries from Linux

The default build id produced by GNU binutils' ld is a 160-bit SHA-1
hash of some parts of the binary, which is exactly 20 bytes:

The bulk of the changes here are to change the signatures of the
FileID methods to use a wasteful_vector instead of raw pointers, since
build ids can be of arbitrary length.

The previous change that added support for this in the processor code
preserved the return value of `Minidump::debug_identifier()` as the
current `GUID+age` treatment for backwards-compatibility, and exposed
the full build id from `Minidump::code_identifier()`, which was
previously stubbed out for Linux dumps. This change keeps the debug ID
in the `dump_syms` output the same to match.

R=mark@chromium.org, thestig@chromium.org

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

c184848 2016-04-05 10:35:26 Patrick Linehan

sample_app: enable C++11 for Android builds

Also add the -D__STDC_LIMIT_MACROS definition as we rely on UINTPTR_MAX.

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

6ce752d 2016-04-02 07:50:46 Veljko Mihailovic

Support processing microdump for mips architecture

Based on changes for ARM, ARM64 and X86, the support for
MIPS and MIPS64 is added in microdump.

TEST=microdump_stackwalk ~/microdump-mips32.dmp symbols/
BUG=microdump_stackwalk failing for mips architectures

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

d9c5322 2016-04-01 23:11:10 Sebastien Marchand

Add the TID to the CallStack.


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

c77c51f 2016-03-31 05:46:21 Li Yan

Refactor sym_upload in tools to extract code into common/linux, and minor fixes
to code calling libcurl. This change may be used to build a tool to dump and
upload symbols with multi-thread.



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

d091e51 2016-03-30 04:32:47 Ted Mielczarek

Make EXC_BAD_ACCESS / EXC_I386_GPFLT print nicely in the processor

Currently EXC_BAD_ACCESS doesn't support EXC_I386_GPFLT as
exception_flags for pretty-printing in the processor, but this happens
for a lot of things:
(search for EXC_I386_GPFLT).

And we get a lot of these in the wild:

This patch makes them show up with a nice name instead of the current
"EXC_BAD_ACCESS / 0x0000000d".

Additionally, this patch fixes some other cases where x86-64 wasn't being handled in the same way as x86, and fixes some x86-specific exception flags to be stringified with I386 in the output.


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

512cac3 2016-03-24 02:17:41 David Yen

Have dump_syms output the full symbol table.

Some of the symbols in the stack trace are not found in the .dynsym
section but were located in the full symbol table .symtab section
instead. This was causing some of our stack traces to be incomplete or
point to incorrect function names.

Since we only output function names, there are actually not that many
more symbols located in .symtab that aren't in .dynsym. It is better to
simply output all symbols found so our stack traces are complete.

R=mark@chromium.org, thestig@chromium.org

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

Patch from David Yen <dyen@chromium.org>.

85b27e4 2016-03-23 03:03:09 Ted Mielczarek

Fix the Linux Starter Guide docs

The auto-conversion from googlecode wiki format apparently wasn't perfect.
This also fixes up some of the links to point to the new Breakpad repo.


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

ebba180 2016-03-12 09:37:46 Ivan Penkov

Explicitly call non-sized delete on dynamically sized memory for correct behavior under sized-delete.

The code as it stands allocates a chunk of memory of arbitrary size and places an object into it. It stores a pointer to that object and memory into a list telling the compiler that it is a pointer to a char. When the compiler deletes the objects in the list it thinks that the list contains pointers to chars - not pointers to arbitrarily sized regions of memory.

This is fixing an issue that will reproduces when the following optimization (C++ sized dealocation) is enabled: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2013/n3536.html

The fix is to explicitly call the non-sized delete operator, and the library code that supports malloc/free/new/delete will figure out the size of the block of memory from the pointer being passed in.

Patch provided by Darryl Gove.


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

1396934 2016-03-10 22:51:20 Jon Turney

Fix format warning in omap.cc

omap.cc(212): warning C4473: 'fprintf' : not enough arguments passed for format string

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

11e1713 2016-03-10 08:16:29 Patrick Linehan

Include necessary source files in Android build config.

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

8915f7b 2016-03-08 17:46:00 Primiano Tucci

Don't merge the mappings if their exec bit are different.


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

4d69050 2016-03-08 11:35:43 Dominik Laskowski

Add an optional root prefix to Linux dumpers

The Linux dumpers use absolute paths for shared libraries referenced by
dumps, so they fail to locate them if the crash originated in a chroot.
This CL enables callers to specify a root prefix, which is prepended to
mapping paths before opening them.

TEST=make check
Review URL: https://codereview.chromium.org/1761023002/

f3d28e9 2016-02-27 11:50:50 Maria Mandlis

Handle multiple microdumps in system log.

Properly handle microdump processing, when the system_log file contains an incomplete microdump section at the top. The processor will process the first complete microdump section.


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

f97e47e 2016-02-18 23:13:56 Maria Mandlis

Support processing microdumps for x86 architecture.


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

25f913e 2016-02-18 01:36:19 Birunthan Mohanathas

Try loading msdiaNNN.dll if CoCreateInstance(CLSID_DiaSource) fails

Because tools/windows/symupload/symupload.cc uses `nullptr` (which
requires VS2010), the CLSID comparison is only performed for msdia100.dll
and later. When compiling with an older (or future) CLSID_DiaSource, we
retain the existing behaviour (i.e. fail if CoCreateInstance fails).


7b25f70 2016-02-17 20:20:58 Ted Mielczarek

Fix buffer overrun in MinidumpModule::debug_identifier with MDCVInfoELF

358c3b4 2016-02-17 14:48:38 Mike Frysinger

linux_dumper_unittest_helper: move to a check-only binary

There's no need for this binary outside of local tests, so stop
building/installing it by default.

TEST=`make check` passe

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

dee15c2 2016-02-17 04:46:04 Ivan Penkov

Fixing a flaky Linux exploitability unittest.


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

e132514 2016-02-13 05:50:16 Ted Mielczarek

Ensure Linux minidump writer flushes minidump header early.

If the Linux minidump writer crashes while writing a dump, the dump
might contain some useful information, but the header will be empty
because TypedMDRVA's destructor flushes the data, and the header var
doesn't go out of scope until the end of the `Dump` method. This
fixes that problem by putting the header in a shorter block scope.

We've seen this problem in some Android dumps in the wild, like:


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

d7c0bd0 2016-02-13 05:17:11 Veljko Mihailovic

Fix for linux make check build failure

Linux make check is failing for mips, mips64, arm, arm64
with error:
"fatal error: mach/arm/vm_types.h: No such file or directory" in case of arm,
"../src/third_party/mac_headers/mach/machine/vm_types.h:37:2: error: #error architecture not supported" in case of mips/mips64

This was partially fixed in https://codereview.chromium.org/1645673002/.

Here excluding src/common/mac/macho_reader_unittest for hosts other than x86/x86-64.

BUG=make check failure for linux mips
TEST=make check pass

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

df280bb 2016-02-12 03:04:04 Maria Mandlis

Parse additional line introduced in the microdump format and containing the GPU infromation in the following format:

The GPU version, vendor and renderer are extracted during microdump parsing and populated in the appropriate fields in the SystemInfo struct.

This is to match the changes introduced in crrev.com/1343713002 and crrev.com/1334473003


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

Afficher sur ancien navigateur de dépôt.