diff options
| author | João Barbosa <joao.paulo.barbosa@gmail.com> | 2017-07-27 00:57:02 +0100 |
|---|---|---|
| committer | João Barbosa <joao.paulo.barbosa@gmail.com> | 2017-07-28 11:23:43 +0100 |
| commit | a6da027d83e48f05c933149ff89c9b9ad5ced915 (patch) | |
| tree | 4a1b394b8304dc6f1163e8e6ed96e4a19bf08fe6 /test/functional | |
| parent | 3ef77a0c1288df524fdf0c90ca70c986f473b787 (diff) | |
| download | bitcoin-a6da027d83e48f05c933149ff89c9b9ad5ced915.tar.xz | |
Reject invalid wallet files
Diffstat (limited to 'test/functional')
| -rwxr-xr-x | test/functional/multiwallet.py | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/test/functional/multiwallet.py b/test/functional/multiwallet.py index 5844d99139..c60d345acb 100755 --- a/test/functional/multiwallet.py +++ b/test/functional/multiwallet.py @@ -6,6 +6,8 @@ Verify that a bitcoind node can load multiple wallet files """ +import os + from test_framework.test_framework import BitcoinTestFramework from test_framework.util import assert_equal, assert_raises_jsonrpc @@ -23,6 +25,14 @@ class MultiWalletTest(BitcoinTestFramework): # should not initialize if there are duplicate wallets self.assert_start_raises_init_error(0, self.options.tmpdir, ['-wallet=w1', '-wallet=w1'], 'Duplicate -wallet filename') + # should not initialize if wallet file is a directory + os.mkdir(os.path.join(self.options.tmpdir, 'node0', 'regtest', 'w11')) + self.assert_start_raises_init_error(0, self.options.tmpdir, ['-wallet=w11'], 'Invalid -wallet file') + + # should not initialize if wallet file is a symlink + os.symlink(os.path.join(self.options.tmpdir, 'node0', 'regtest', 'w1'), os.path.join(self.options.tmpdir, 'node0', 'regtest', 'w12')) + self.assert_start_raises_init_error(0, self.options.tmpdir, ['-wallet=w12'], 'Invalid -wallet file') + self.nodes[0] = self.start_node(0, self.options.tmpdir, self.extra_args[0]) w1 = self.nodes[0] / "wallet/w1" |