Page MenuHomePhabricator

[backport] Reject invalid wallets
ClosedPublic

Authored by schancel on Jan 20 2018, 06:27.

Details

Summary

Backport of Bitcoin Core PR#10885: Reject invalid wallets

d84e78e [wallet] Specify wallet name in wallet loading errors (John Newbery)
a6da027 Reject invalid wallet files (João Barbosa)
3ef77a0 Reject duplicate wallet filenames (João Barbosa)

Pull request description:

This PR prevents loading the same wallet more than once in a multi wallet scenario. It also prevents loading with invalid files: non regular files or symlinks.

Depends on D987
Closes T190

Test Plan

make check && ./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
depends-on-D987
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1724
Build 1724: arc lint + arc unit

Event Timeline

schancel retitled this revision from Reject invalid wallets to [backpor] Reject invalid wallets.Jan 20 2018, 06:28
schancel edited the test plan for this revision. (Show Details)
schancel retitled this revision from [backpor] Reject invalid wallets to [backport] Reject invalid wallets.
matiu requested changes to this revision.Jan 21 2018, 14:36
matiu added inline comments.
test/functional/test_framework/test_framework.py
39 ↗(On Diff #2626)

This is already at line 37.

This revision now requires changes to proceed.Jan 21 2018, 14:36

Remove changes to test_framework

src/wallet/wallet.cpp
529 ↗(On Diff #2657)

Should be fs::

deadalnix added inline comments.
src/wallet/wallet.cpp
529 ↗(On Diff #2657)

Yes please.

boost::filesystem:: -> fs::

This revision is now accepted and ready to land.Jan 23 2018, 00:09
This revision was automatically updated to reflect the committed changes.