Page MenuHomePhabricator

Backport of Core PR14728: fix uninitialized read when stringifying an addrLocal
ClosedPublic

Authored by majcosta on Nov 27 2019, 23:02.

Details

Reviewers
jasonbcox
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCeb1893c84752: Backport of Core PR14728: fix uninitialized read when stringifying an addrLocal
Summary

This is a backport of Core PR14728

As above. https://github.com/bitcoin/bitcoin/pull/14728

Test Plan
ninja check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
second-backport
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 8320
Build 14659: Bitcoin ABC Buildbot
Build 14658: arc lint + arc unit

Event Timeline

majcosta created this revision.Nov 27 2019, 23:02
Owners added a reviewer: Restricted Owners Package.Nov 27 2019, 23:02
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 27 2019, 23:02
deadalnix requested changes to this revision.Nov 27 2019, 23:11
deadalnix added inline comments.
src/netaddress.h
34 ↗(On Diff #14475)

Revert

73 ↗(On Diff #14475)

dito

This revision now requires changes to proceed.Nov 27 2019, 23:11
majcosta updated this revision to Diff 14477.Nov 27 2019, 23:37

reverted inline comments

deadalnix accepted this revision.Nov 27 2019, 23:39
This revision is now accepted and ready to land.Nov 27 2019, 23:39
deadalnix edited the summary of this revision. (Show Details)Nov 27 2019, 23:40

Keep in mind to mention PR12354936 in the diff sum up so people can grep for it easily.

You can also indent commands with double space rather than use backticks.

like this
majcosta updated this revision to Diff 14479.Nov 27 2019, 23:42

removed declaration not in our codebase

majcosta edited the test plan for this revision. (Show Details)Nov 27 2019, 23:45