Age | Commit message (Collapse) | Author |
|
mainline-permission-release
Change-Id: I16732bc942de4c220b58120deb2b172f9e8fda0d
|
|
am: ff243c3ea8
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1607974
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I157e45e0dd07ca464faedd7fa3c3a95daf0935a7
|
|
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1607974
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ia190891c28bee9470888de47601cf81a6b954fd0
|
|
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1607974
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I5ab7de37f54935efc130da6d3b776c94aeb3fe38
|
|
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1607974
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I96349124bbc35c1a96d55dc5da236438b0f6708c
|
|
|
|
Bug: 181174838
Bug: 178702846
Bug: 181203470
Test: m out/soong/.intermediates/external/google-breakpad/breakpad_client/linux_bionic_arm64_static/obj/external/google-breakpad/src/client/linux/minidump_writer/linux_core_dumper.o
Change-Id: I47f18a0de28c9b3db9a34a4a93e8e4ef80af102f
|
|
61ebad0fd0 am: 95b3c38624 am: d04303a5eb
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1588975
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I9d9427709cb6caedf023ecbc32915345f6cca9ca
|
|
61ebad0fd0 am: 95b3c38624
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1588975
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ib3d71a0a2ad3bdaceba921d67476e5d4ee534e3a
|
|
61ebad0fd0
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1588975
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I7c52b116a7f21a20145668787e575d693df51130
|
|
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1588975
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I496269f1d28f559791d32ea8d68f0ece59e99ccc
|
|
Added SPDX-license-identifier-APSL-2.0
SPDX-license-identifier-Apache-2.0 SPDX-license-identifier-Artistic
SPDX-license-identifier-BSD SPDX-license-identifier-FSFAP
SPDX-license-identifier-ISC SPDX-license-identifier-MIT
SPDX-license-identifier-Unicode-DFS legacy_notice
legacy_unencumbered
to:
Android.bp
Added SPDX-license-identifier-BSD
to:
android/google_breakpad/Android.mk
android/sample_app/jni/Android.mk
Bug: 68860345
Bug: 151177513
Bug: 151953481
Test: m all
Exempt-From-Owner-Approval: janitorial work
Change-Id: I748afbce03f889c37576012387694a670b267eac
|
|
am: 7a24447c4d
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1556522
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Icd60d91a45edc0d8bdf037c0ab949e8ffe0e1f67
|
|
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1556522
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I739d34539bf804a8be06ff4471f7ac4b6b05f8c4
|
|
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1556522
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I1e2cda35b5ca3922ef877caa6e21de19a553d1a6
|
|
Original change: https://android-review.googlesource.com/c/platform/external/google-breakpad/+/1556522
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ie36ff4d1208ebf4baba1d082645aba656df5583d
|
|
See: http://go/android-license-checking-v2-lsc
Test: m all
Bug: 151953481
Bug: 151177513
Bug: 67772237
Exempt-From-Owner-Approval: janitorial work
Change-Id: I52c8df8ba7f3fd42098fb57c66bc20d487fc4558
|
|
merge-from-upstream-20190420 am: 22f554e9e4 am: ba746a803a am: 8cf44142ee am: b2a028d1fa am: d6c43e2d4b
Change-Id: I2f7724f26868bf0681a74ebe22905f423f58fec8
|
|
merge-from-upstream-20190420 am: 22f554e9e4 am: ba746a803a am: 8cf44142ee am: b2a028d1fa
Change-Id: I8de86155d76a115b13d190ae84bd1635e2c98652
|
|
merge-from-upstream-20190420 am: 22f554e9e4 am: ba746a803a am: 8cf44142ee
Change-Id: I0ef77cf5aabc77e513993bee8cc99f6df6190647
|
|
merge-from-upstream-20190420 am: 22f554e9e4 am: ba746a803a
Change-Id: I3d1432e94f27dc0a0fcbc06e54ae15531a210d16
|
|
merge-from-upstream-20190420 am: 22f554e9e4
Change-Id: I3c2395037444eff220d8ee818ed6cccf26edd8d4
|
|
merge-from-upstream-20190420
* Fixed build for host
* Added required METADATA and MODULE_LICENSE_BSD files
Bug: 154443614
Test: mm -j40
Change-Id: I5f9bb7b9ac14170b7e44e3b3dcfb4c37619fbd15
|
|
Change-Id: Iaa79c5c7c70ca953256cd39cc9f6f4f139c696af
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2153567
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
Bug: 1071277
Change-Id: Ic65704a20c55fea83b5633650ee2c769ac227a56
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2153333
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
Change-Id: Ib815b0757539145c005d828080b92cbfa971a21b
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2141226
Reviewed-by: Ivan Penkov <ivanpe@chromium.org>
|
|
These are left over from when this file was written in C.
In C++, true and false are keywords, so this is no longer necessary.
In fact, redefining a keyword is not actually permitted in ISO C++
(https://eel.is/c++draft/macro.names#2), and is rejected by MSVC.
Change-Id: I2f4a6eae7f8dd4d91b8ee6588c7ae5e8f20bd0d0
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2105831
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
__WORDSIZE is an internal libc definition. Instead, we can use
ULONG_MAX from limits.h, whose value corresponds to the machine's
native word size.
This allows us to remove the fallback definition of __WORDSIZE in
the Android compatibility headers.
Bug: google-breakpad:631
Change-Id: I7b9e6f3b2121f78ccad9e32bf26acac518aefd8f
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2107100
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
https://chromium.googlesource.com/linux-syscall-support/+log/f70e2f1641e2..fd00dbbd0c06
$ git log f70e2f164..fd00dbbd0 --date=short --no-merges --format='%ad %ae %s'
2020-03-10 mforney Avoid conflict with getdents64 defined by libc
2019-01-26 vapier futex: change to 6 args
Created with:
roll-dep src/src/third_party/lss
Bug: google-breakpad:631
Change-Id: Icc248237dcc7bdc3f7b74cd8556efcaf3b4b1c31
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2106526
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
getcontext is also not available on musl libc, so generalize
breakpad_getcontext so it can be used as a fallback for non-Android
platforms as well.
On x86_64 and i386, ucontext_t uses an Android-specific offset for
storage of FP registers, since its sigset_t differs in size. So,
make the definition of MCONTEXT_FPREGS_MEM and UCONTEXT_FPREGS_MEM_OFFSET
conditional on whether we are building for Android.
On glibc and musl, signal.h and asm/sigcontext.h can't be included
together, so in breakpad_context_unittest.cc, only compare the libc
and kernel _fpstate when on Android.
Bug: google-breakpad:631
Change-Id: If81d73c4101bae946e9a3655b8d1c40a34ab6c38
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2102135
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
Bug: google-breakpad:725
Change-Id: I8c5f9a828e1bb6a2a5603fbfaa127f6dfe64a4b4
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/1803770
Reviewed-by: Mark Mentovai <mark@chromium.org>
|
|
The crash address from the microdump was never checked against
anything. Instead, the test was checking the value of a constant.
On 32-bit systems, an intptr_t cannot represent kCrashAddress
(0xDEADDEAD), causing a failure when the crash address is parsed
from the microdump. Instead, use uintptr_t, which matches the type
of kCrashAddress.
Change-Id: Ib5612743803609f7801dcfb98deaa8779e362025
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2100816
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
Since a03ad96dbd, sys/signal.h is no longer required.
Change-Id: I64e6235e1bffb3728d6050389bad6e7a99389741
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2097666
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
Change-Id: I5c5e5c85ec49a76e5f4c7c6a0f662398e6cdc6ee
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2097681
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
We are already in a LINUX_HOST conditional block, so there is no need
to check for it again.
Change-Id: Ifbe283ffa47373c98d2653f1014445d1a435c715
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2093144
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
The _libc_fpstate struct tag is an implementation detail of glibc,
and musl uses a different struct tag, _fpstate.
Unfortunately, the public type fpregset_t is a pointer type, so is
not suitable for allocating storage, or referring to constant
storage. Instead, we can use std::remove_pointer<fpregset_t>::type
to refer to the pointed-to type, regardless of the struct tag.
Bug: google-breakpad:631
Change-Id: Iaf47f15b2d834dd8de839431f65a481e9b0c7f9e
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2096171
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
__WORDSIZE is an internal libc definition. Instead, we can use
ULONG_MAX from limits.h, whose value corresponds to the machine's
native word size.
Bug: google-breakpad:631
Change-Id: If69caf578286d678585d1510c01562b969b5061f
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2097352
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
sys/poll.h and sys/signal.h just redirect to the standardized
location for these headers, poll.h and signal.h.
On musl libc, including the incorrect header path results in a
warning, and since breakpad is built with -Werror, this is an error.
In exception_handler.cc, signal.h is already included earlier, so
we can drop the sys/signal.h include.
Bug: google-breakpad:631
Change-Id: If36d326453e3267d38a5b92ed1301f828e46befe
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2097344
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
pthread_yield is not a standard POSIX function, and is not available
in musl libc. The man page says to "Use the standardized sched_yield(2)
instead"[0].
On glibc, pthread_yield is exactly equivalent to sched_yield[1].
On bionic, pthread_yield is also not available, so on Android, the
tests define a wrapper that just calls sched_yield. This wrapper
is no longer necessary if we just use sched_yield in the first
place.
[0] http://man7.org/linux/man-pages/man3/pthread_yield.3.html
[1] https://sourceware.org/git/?p=glibc.git;a=blob;f=nptl/pthread_yield.c
Bug: google-breakpad:631
Change-Id: Ie4c6be8c17cdc2f5396a7fe972fa51a97573b049
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2097340
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
This reverts commit 0c0e24f709288a129d665ec27d6f089189318385.
Reason for revert: This is causing breakage on CrOS due to unhandled
enums and dwarf5 support is in limbo, so pull this back out for now.
Bug: google-breakpad:812
Change-Id: I22dd5d87efb8bc83596d51d15a6808c41afb3fd2
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2095872
Reviewed-by: Mark Mentovai <mark@chromium.org>
|
|
This reverts commit dbd454dbe47e584571388fc3533193416bdce67f.
Reason for revert: The parent CL is causing breakage on CrOS due to
unhandled enums. Before we can revert that, we need to revert this.
Bug: google-breakpad:812
Change-Id: I7c2446f3cd8ed9f6411e90dbdd2434bc463b2f6c
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2095798
Reviewed-by: Mark Mentovai <mark@chromium.org>
|
|
Previously, dump_syms produced warnings whenever a DW_AT_specification
or DW_AT_abstract_origin attribute was a forward reference. 47cd498384fd
allowed those attributes to carry forward references, removing the
warnings altogether. It was not correct to remove the warnings entirely.
References that do not point to valid DIEs should still produce
warnings, whether a back reference or a forward reference.
This reintroduces those warnings as appropriate.
Bug: google-breakpad:813
Test: dumper_unittest SimpleCU.UnknownAbstractOrigin,Specifications.BadOffset
Change-Id: Ie7222c7a1886bab31423f27e2fbcce93e69625b3
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2090103
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
- "sym-upload-v2" protocol now supports specifying a symbol file "type".
- Known types are "breakpad" (default option, previously this was only
effectively the only option), "elf", "pe", "macho", "debug_only", "dwp",
"pdb", and "dsym".
- When type other than breakpad is specified, sym_upload tool requires
the code_file and debug_id value (that it otherwise would have gotten
from the text of the Breakpad symbol file).
- This ultimately means that sym_upload can be used to upload native
symbol files now, in addition to Breakpad symbol files.
Change-Id: I3a331ba16f199d1d0025df735716ba5de298f522
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2078670
Reviewed-by: Mark Mentovai <mark@chromium.org>
|
|
- Didn't used to support statically linked libcurl, now it does (like
HttpUpload does).
Change-Id: Ic014548225b129f0c1c9ffe6a671f5bd2352b6e6
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2068947
Reviewed-by: Ivan Penkov <ivanpe@chromium.org>
Reviewed-by: Joshua Peraza <jperaza@chromium.org>
|
|
Change-Id: I58612eb70173d79c2aeb755d8c7f5677cb3a8fb8
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2067608
Reviewed-by: Nelson Billing <nbilling@google.com>
|
|
Previously, the processor truncated the upper of two
overlapping module ranges to compensate for incorrect
reporting of module ranges by the Breakpad client.
Crashpad correctly reports module load ranges, so
this truncation strategy is no longer necessary.
However, when partitioned libraries are used, the base
library may have a range which encompasses the other
partitions. When this is combined with the truncate
upper merge strategy, the base library's executable
segment is truncated causing symbolization failures.
This patch changes Android's merge strategy to truncate
the lower range (which is still the base library, but
this strategy truncates from the high end of the
library's range, instead of its base).
Bug: b/149845120
Change-Id: Ic75ecd3e919432690740eb21ebd4265fc0bbaa86
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2067952
Reviewed-by: Mark Mentovai <mark@chromium.org>
|
|
Change-Id: I6a49e9f4a699fa6f5f8e9f0fc86afb4cb342a442
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/1422400
Reviewed-by: Mark Mentovai <mark@chromium.org>
Reviewed-by: Ivan Penkov <ivanpe@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
* Renamed convert_UTF.c to convert_UTF.cc
* Enabled to use C++17 for [[clang::fallthrough]] defined in src/common/macros.h
Patch by Hiro Komatsu
Change-Id: I5de7f7dd4c8bf231a004144a5c82828c59ddcfd6
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2026761
Reviewed-by: Mark Mentovai <mark@chromium.org>
|
|
https://chromium.googlesource.com/linux-syscall-support/+log/8048ece6c16c..f70e2f1641e2
$ git log 8048ece6c..f70e2f164 --date=short --no-merges --format='%ad %ae %s'
2020-01-22 mforney Avoid conflict with pread64/pwrite64 defined by libc
2019-12-05 jperaza arm, ppc, mips32: fix fallocate and add tests
2019-11-13 jperaza Add sigtimedwait and sigprocmask wrappers
Created with:
roll-dep src/src/third_party/lss
Change-Id: I63e1e0722a338875f53884565ea4c2706ec67037
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2020986
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
Stackwalk can't recover caller's register X30($LR) value by STACK CFI info.
This will lead unwinding stop immediately when fallback to frame pointer unwinder.
This PR will use try to use second last frame to recover register X30($LR) by frame pointer.
And we will give up correction if STACK CFI info doesn't agree with frame pointer info.
Bug: https://bugs.chromium.org/p/google-breakpad/issues/detail?id=808
Change-Id: I50649e3398e268b02ff297e83db21d05705c2a2d
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/1992641
Reviewed-by: Joshua Peraza <jperaza@chromium.org>
|