lp:~itachi-san/telegram-desktop/breakpad

Branch merges

This import branch has no branches proposed for merge into it.

No branches dependent on this one.

Related source package recipes

1 recipe using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

2199. By Liryna <email address hidden> on 2024-09-04

[windows] dump_syms - Support ARM64 PDB

Change-Id: I6a9cb5c5253abcd34cb72a035d280b7fd8a50372
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5836085
Reviewed-by: Nelson Billing <email address hidden>

2198. By abrook <email address hidden> on 2024-09-04

Fix bugs with alternative section naming on mac, and unpropagated
section information.

Change-Id: I6f4e112a83aeee1e4cd525c1d1d0b9213ef2d09b
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5838157
Reviewed-by: Joshua Peraza <email address hidden>

2197. By Greg Thompson <email address hidden> on 2024-08-16

[windows] Silence MSVC warning C4100 when building without zlib support

Also correct the indentation of the Deflate function to match the
surrounding code.

Bug: b/354710760
Change-Id: I17e64b2f6390f673169a783e63505eb9df233a79
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5771532
Reviewed-by: Joshua Peraza <email address hidden>

2196. By Sean Carpenter <email address hidden> on 2024-07-30

Adds Support for EC Zephyr Binaries

The text section header for EC Zephyr binaries is `text` instead of the
standard `.text`. As a result the file_id hash function was failing when
attempting to generate a file id by hashing the contents of a Zephyr
binaries `text` section.

This change adds support for Zephyr binaries by having said hash
function also check the `text` section in addition to `.text`.

Test output: https://paste.googleplex.com/4708788542373888#l=20

Three tests are listed as failing in the test output. This is a known
unrelated issue caused by this commit: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5636127

TEST=make check && dump_syms -a ~/some.zephyr.elf
BUG=b:328511413

Change-Id: I88882305a4c9ca18082ba2690ba1b22da03bd19d
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5732693
Reviewed-by: Forest Mittelberg <email address hidden>
Reviewed-by: Joshua Peraza <email address hidden>

2195. By Tom Burgin <email address hidden> on 2024-07-26

Revert "[windows] Silence MSVC warning C4100 when building without zlib support"

This reverts commit ee94d7f24bf1ac756c8db20b6d9e052acaa17e0c.

Reason for revert: Breaks Windows builds of Chromium.

https://ci.chromium.org/ui/p/chromium/builders/try/win-rel/695756/overview

Original change's description:
> [windows] Silence MSVC warning C4100 when building without zlib support
>
> Bug: b/354710760
> Change-Id: I0a5771795ca456cd07ad26038f005553875df641
> Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5735099
> Reviewed-by: Ivan Penkov <email address hidden>
> Reviewed-by: Ivan Penkov <email address hidden>

Bug: b/354710760
Change-Id: Ib49e06944934d467beaefb91411235eed020381a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5742385
Reviewed-by: Ivan Penkov <email address hidden>

2194. By Tom Burgin <email address hidden> on 2024-07-26

upload_system_symbols: break into go packages

This will make reusing this code easier. Since there are multiple
packages now, the build needs updating. A separate CL in Chromium will
follow this change.

Change-Id: I8d95d13ea61d32684d0ef845c3d1da2cd6d4cd7f
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5742415
Reviewed-by: Mark Mentovai <email address hidden>
Reviewed-by: Leonard Grey <email address hidden>

2193. By Greg Thompson <email address hidden> on 2024-07-25

[windows] Silence MSVC warning C4100 when building without zlib support

Bug: b/354710760
Change-Id: I0a5771795ca456cd07ad26038f005553875df641
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5735099
Reviewed-by: Ivan Penkov <email address hidden>
Reviewed-by: Ivan Penkov <email address hidden>

2192. By Greg Thompson <email address hidden> on 2024-07-19

[windows] Include-what-you-use fixes in http_upload.{cc,h}

Bug: b/353869880
Change-Id: I7dd2d432032099c77410ce67a0fce6c7a71b473f
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5720428
Reviewed-by: Ivan Penkov <email address hidden>

2191. By Sean Carpenter <email address hidden> on 2024-07-17

Adds NDS32 Architecture Support to dump_syms

NDS32 is a relatively uncommon architecture that is used on a
significant chunk of our ChromeOS EC Microprocessors. NDS32 symbol table
generation was already supported, but would fail due to "Unrecognized ELF
machine architecture". This change fixes that.

TEST=dump_syms -a some.nds32.elf
BUG=b:328511413

Change-Id: I3e9ae900be483c957a824166d24e2d5db240b64a
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5718928
Reviewed-by: Forest Mittelberg <email address hidden>
Reviewed-by: Ivan Penkov <email address hidden>

2190. By Greg Thompson <email address hidden> on 2024-07-12

[Windows] PDBSourceLineWriter improvements

COM calls that return a BSTR via an out param pass ownership of the
string's underlying memory to the caller. It is the caller's
responsibility to free the memory via SysFreeString. Be sure to use
CComBSTR in all such cases, so that this happens upon destruction of the
CComBSTR. This CL fixes three BSTR memory leaks.

The inline_origins_ member maps an inline origin's name to a unique
numerical identifier. Simplify the container by mapping a name only to
an id rather than a name to a struct holding the name and id.

These origins are printed ordered by id. Rather than using a std::set to
create a container sorted by id, leverage the fact that ids are ints
that are assigned sequentially to create the reverse mapping in a
std::vector. This greatly reduces heap usage, and reduces complexity to
O(N).

Combined, these changes reduce commit charge by 26% and runtime by 10%
when processing chrome.dll for a 32-bit win-dcheck build.

Change-Id: Ib8d6540c74622500989f1dc06f705d6846be303f
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5682242
Reviewed-by: Ivan Penkov <email address hidden>