Merge #10027: Set to nullptr after delete
ClosedPublic
Actions

Authored by jasonbcox on May 6 2019, 21:10.

Details

Reviewers

deadalnix
Fabien
markblundeberg
Mengerian

Group Reviewers

Restricted Project

Commits

rSTAGINGf5919276fb6e: Merge #10027: Set to nullptr after delete
rABCf5919276fb6e: Merge #10027: Set to nullptr after delete

Summary

d93b97f Set to nullptr after delete (practicalswift)

Tree-SHA512: 7201cef4541557ffe31f52ce7527c4b08a2ff5aa1eae5268bdfee5b4843881f8fd115257bef6d1b4dfb71166951950a912ce87aef160ca89c2ca2ae264cfab1b

Backport of Core PR 10027
https://github.com/bitcoin/bitcoin/pull/10027/files
Changes in dbwrapper_tests.cpp have already been supersceded by an out-of-order backport.

Test Plan

ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr10027

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5700
Build 9462: Bitcoin ABC Buildbot (legacy)
Build 9461: arc lint + arc unit

Event Timeline

jasonbcox created this revision.May 6 2019, 21:10

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 6 2019, 21:10

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

deadalnix accepted this revision.May 6 2019, 21:12

This revision is now accepted and ready to land.May 6 2019, 21:12

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B5700: Diff 8424.May 6 2019, 21:23

Closed by commit rABCf5919276fb6e: Merge #10027: Set to nullptr after delete (authored by Wladimir J. van der Laan <laanwj@gmail.com>, committed by jasonbcox). · Explain WhyMay 6 2019, 21:30

This revision was automatically updated to reflect the committed changes.

jasonbcox added a commit: rSTAGINGf5919276fb6e: Merge #10027: Set to nullptr after delete.Sep 18 2019, 23:17

Revision Contents
Changeset List

Path

Size

src/

httpserver.cpp

1 line

Diff 8424

View Options

Merge #10027: Set to nullptr after deleteClosedPublicActions