Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libbacktrace: Fix uninitialized variable #39509

Merged
merged 1 commit into from
Feb 5, 2017
Merged

Conversation

petrochenkov
Copy link
Contributor

Fixes #39468

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+

Nice find!

@bors
Copy link
Contributor

bors commented Feb 3, 2017

📌 Commit d14b268 has been approved by alexcrichton

@alexcrichton
Copy link
Member

Nominating for a beta backport

@alexcrichton alexcrichton added beta-nominated Nominated for backporting to the compiler in the beta channel. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. labels Feb 3, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 5, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 5, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 5, 2017
bors added a commit that referenced this pull request Feb 5, 2017
Rollup of 12 pull requests

- Successful merges: #39439, #39472, #39481, #39491, #39501, #39509, #39514, #39519, #39526, #39528, #39530, #39538
- Failed merges:
@bors bors merged commit d14b268 into rust-lang:master Feb 5, 2017
@alexcrichton
Copy link
Member

Accepted for backport in libs triage today

@alexcrichton alexcrichton added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Feb 6, 2017
@brson brson removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Feb 11, 2017
bors added a commit that referenced this pull request Feb 11, 2017
[beta] Backporting PRs to beta

This is a backport of the following PRs:

* #39478
* #39509
* #39517
* #39526
* #39599
* #39624
* #39710
@petrochenkov petrochenkov deleted the rb2 branch March 16, 2017 19:43
alexcrichton added a commit to alexcrichton/libbacktrace that referenced this pull request Jul 25, 2017
In rust-lang/rust#39468 it was discovered that this could cause a crash in
libbacktrace due to freeing uninitialized memory, and this specific instance was
fixed in rust-lang/rust#39509
dscho pushed a commit to dscho/libbacktrace that referenced this pull request Jul 18, 2022
In rust-lang/rust#39468 it was discovered that this could cause a crash in
libbacktrace due to freeing uninitialized memory, and this specific instance was
fixed in rust-lang/rust#39509
johnsonjh added a commit to johnsonjh/libbacktrace that referenced this pull request Oct 18, 2022
… of uninitialized bytes in pecoff.c; 3) Fix a use of undefined memory in pecoff.c

1) Note: as we target MINGW here, we still want to look up the symbols via
the DWARF method (the native Windows way would be to call the
SymFromAddr() function, but that would require .pdb files which MINGW
does not produce).
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>

2) This updates the local declaration of `str_size` to always be 4 bytes instead of
platform-dependent as its initialization later on only fills in 4 bytes instead
of all the bytes of `size_t`.
Originally reported as rust-lang/rust#28447 this was fixed in
rust-lang/rust#30908

3) In rust-lang/rust#39468 it was discovered that this could cause a crash in
libbacktrace due to freeing uninitialized memory, and this specific instance was
fixed in rust-lang/rust#39509
dscho pushed a commit to dscho/libbacktrace that referenced this pull request Apr 8, 2024
In rust-lang/rust#39468 it was discovered that this could cause a crash in
libbacktrace due to freeing uninitialized memory, and this specific instance was
fixed in rust-lang/rust#39509
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants