summaryrefslogtreecommitdiff
path: root/src/wallet/wallet2.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7997luigi11112021-11-091-9/+21
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 29c03e2 wallet_api: enable set_strict_default_file_permissions (tobtoht) 1c26d34 epee: add missing header (selsta) 1d451a6 depends: update zeromq package to 4.3.4 (selsta) 5180610 cmake: use CMAKE_CURRENT_LIST_DIR for git version (selsta) 5aae00a epee: avoid ADL selecting C++14 std::quoted (selsta) cf31136 cmake: set cpp14 standard (selsta) 945665f fix median overflow bug (koe) 908ecb3 LMDB: fix deadlock in resized detection (Howard Chu) a492c4e blockchain: fix pedantic assert (selsta) 1e663e8 Increase timeout for the mining test 4 times (to 240s) (mj-xmr) 4760f8b Wallet2: Update 'approximate_testnet_rolled_back_blocks' (rbrunner7) 76b6120 cmake: set required C/C++ standard to 11 (selsta) 3067987 Fix parallel make of OpenSSL (Howard Chu) 1219c74 add singapore.node.xmr.pm to seed nodes (lza_menace) 689dd52 net_node: add seed node (selsta) 95936a3 depends: update unwind, fix build with ubuntu 21.04 (selsta) e2c802f Only INFO log for short seed words with ALLOW_SHORT_WORDS active (rbrunner7) 14df8ad epee: don't log raw packet data (moneromooo-monero) 5fa7208 Mac: Enable on ARM, by not enforcing x86_64 (mj-xmr) fe6ab93 daemon: allow proxy configuration (anon) 52dcc03 bootstrap_daemon: proxy configuration support (xiphon) a6df656 wallet2: chunk get_outs.bin calls to avoid sanity limits (moneromooo-monero)
| * Wallet2: Update 'approximate_testnet_rolled_back_blocks'rbrunner72021-10-071-1/+1
| |
| * wallet2: chunk get_outs.bin calls to avoid sanity limitsmoneromooo-monero2021-10-061-8/+20
| |
* | wallet2: fix key encryption when changing ask-password from 0/1 to 2moneromooo-monero2021-10-191-20/+3
| | | | | | | | | | we reuse the wallet_keys_unlocker object, which does the right thing in conjunction with other users of decrypt/encrypt (ie, refresh).
* | Merge pull request #7993luigi11112021-10-171-1/+1
|\ \ | | | | | | | | | c599b56 Decrease the 'recent spend window' in gamma re-select to 15 blocks (j-berman)
| * | Decrease the "recent spend window" in gamma re-select to 15 blocksj-berman2021-10-041-1/+1
| |/ | | | | | | | | - combined with patching integer truncation (#7798), this gets the algorithm marginally closer to mirroring empirically observed output ages - 50 was originally chosen assuming integer truncation would remain in the client for that client release version. But patching integer truncation causes the client to select more outputs in the 10-100 block range, and therefore the benefit of choosing a larger recent spend window of 50 has less merit - 15 seems well-suited to cover the somewhat sizable observable gap in the early window of blocks
* / Fix precision of average_output_timej-berman2021-09-121-7/+1
|/ | | | The fix as suggested by <jberman> on IRC. Before the fix, it would truncate 1.9 to 1 skewing the output selection.
* Merge pull request #7894luigi11112021-08-261-4/+5
|\ | | | | | | c0ce4d5 Wallet2: fix optimize-coinbase for p2pool payouts (SChernykh)
| * Wallet2: fix optimize-coinbase for p2pool payoutsSChernykh2021-08-261-4/+5
| | | | | | | | | | | | | | | | | | | | RefreshOptimizeCoinbase was an optimization to speed up scanning of coinbase transactions before RingCT (tx version 2) where they split miner reward into multiple denominations, all to the same wallet. When RingCT was introduced, all coinbase transactions became 1 output only, so this optimization does nothing now. With p2pool, this optimization will skip scanning p2pool payouts because they use more than 1 output in coinbase transaction. Fix it by applying this optimization only to pre-RingCT transactions (version < 2).
* | Merge pull request #7825luigi11112021-08-261-9/+36
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ccfed60 cmake: test is a reserved keyword (selsta) f0fa959 cmake: fix ccache detection (anon) 8dbe2e5 Daemon: Update average block size table (rbrunner7) 94bad34 wallet2: Don't auto lock device on process parsed blocks (tobtoht) 3c23aca ledger: don't lock for software device (tobtoht) 49beb8d ledger: use software device if we have view key (tobtoht) d188b5f wallet_api: getPassword (tobtoht) a395f36 depends, docker: fix bintray links (selsta) 0fecd7e blocks: fix cmake syntax (selsta) 76824bf Stop adding more outputs than bulletproof allows (Alex Opie) 67ba733 unit_tests: fix wipeable_string parse_hexstr test with latest gtest (xiphon) 402ba04 cmake: use CMAKE_CURRENT_LIST_DIR in FindLibUSB (selsta) 1565bcb epee: include public openssl header in cmake (selsta) 2e9af2a cmake: treat warnings as error in compiler flag tests (selsta) db564ef wallet_api: fix typo in exportKeyImages (selsta) 8507917 depends: add getmonero package mirror (selsta) 8716d2a cmake: fix non portable code (selsta)
| * | wallet2: Don't auto lock device on process parsed blockstobtoht2021-08-071-2/+1
| | |
| * | Stop adding more outputs than bulletproof allowsAlex Opie2021-08-011-7/+35
| | | | | | | | | | | | | | | | | | If more outputs are requested, they are split across multiple transactions. #7322
* | | Merge pull request #7822luigi11112021-08-261-0/+31
|\ \ \ | |_|/ |/| | | | | 99bee1c Apply gamma distr from chain tip when selecting decoys (j-berman)
| * | Apply gamma distr from chain tip when selecting decoysj-berman2021-08-191-0/+31
| |/ | | | | | | - matches the paper by Miller et al to apply the gamma from chain tip, rather than after unlock time - if the gamma produces an output more recent than the unlock time, the algo packs that output into one of the first 50 spendable blocks, respecting the block density factor
* | Merge pull request #7848luigi11112021-08-201-1/+2
|\ \ | | | | | | | | | adcf440 Make sure node returns to wallet that real output is unlocked (j-berman)
| * | Make sure node returns to wallet that real output is unlockedj-berman2021-08-111-1/+2
| |/
* | Merge pull request #7846luigi11112021-08-201-0/+6
|\ \ | | | | | | | | | 2285c66 Protect client from divide by 0 caused by integer truncation (j-berman)
| * | Protect client from divide by 0 caused by integer truncationj-berman2021-08-191-0/+6
| |/
* | Merge pull request #7792luigi11112021-08-111-3/+6
|\ \ | | | | | | | | | 9b84043 fix #7784 - deinit wallet in wallet dtor (Dusan Klinec)
| * | fix #7784 - deinit wallet in wallet dtorDusan Klinec2021-07-141-3/+6
| |/
* / wallet: rephrase error message on invalid device addressDusan Klinec2021-07-141-1/+1
|/
* wallet2: Fix rescan_bc keep_ki optionNathan Dorfman2021-04-221-21/+26
|
* fix serialization being different on macmoneromooo-monero2021-03-051-5/+15
| | | | | | | | | | | | On Mac, size_t is a distinct type from uint64_t, and some types (in wallet cache as well as cold/hot wallet transfer data) use pairs/containers with size_t as fields. Mac would save those as full size, while other platforms would save them as varints. Might apply to other platforms where the types are distinct. There's a nasty hack for backward compatibility, which can go after a couple forks.
* restrict public node checks a littlemoneromooo-monero2020-12-221-0/+1
| | | | | do not include blocked hosts in peer lists or public node lists by default, warn about no https on clearnet and about untrusted peers likely being spies
* wallet2: set propagation timeout to current max timeoutselsta2020-12-031-1/+2
|
* wallet2: check imported multisig curve points are in main subgroupCrypto City2020-11-241-0/+14
|
* wallet2: wait for propagation timeout before marking tx as failedxiphon2020-10-181-2/+6
|
* fix a couple bugs found by OSS-fuzzmoneromooo-monero2020-09-241-0/+2
| | | | | - index out of bounds when importing outputs - accessing invalid CLSAG data
* Merge pull request #6793luigi11112020-09-151-1/+1
|\ | | | | | | de9a9fb update error message 'No unlocked balance in the specified account' (woodser)
| * update error message "No unlocked balance in the specified account"woodser2020-09-011-1/+1
| |
* | wallet2: adapt to deterministic unlock timeTheCharlatan2020-09-151-10/+21
| |
* | Merge pull request #6815luigi11112020-09-141-3/+8
|\ \ | | | | | | | | | ef4325f wallet2: fix tx sanity check triggering on pre-rct outputs (monermooo-monero)
| * | wallet2: fix tx sanity check triggering on pre-rct outputsmoneromooo-monero2020-09-121-3/+8
| | |
* | | Fix typo in command line argument descriptionReinaldulin2020-09-091-1/+1
| | |
* | | Merge pull request #6798luigi11112020-09-071-1/+1
|\ \ \ | |/ / |/| | | | | 4e44306 enable CLSAG support for Trezor client (ph4r05)
| * | enable CLSAG support for Trezor clientDusan Klinec2020-09-041-1/+1
| |/
* | Merge pull request #6757luigi11112020-09-031-15/+15
|\ \ | |/ |/| | | 6a37da8 threadpool: guard against exceptions in jobs, and armour plating (moneromooo-monero)
| * threadpool: guard against exceptions in jobs, and armour platingmoneromooo-monero2020-09-011-15/+15
| | | | | | | | | | Those would, if uncaught, exit run and leave the waiter to wait indefinitely for the number of active jobs to reach 0
* | Bind signature to full address and signing modeSarang Noether2020-08-281-24/+40
| |
* | wallet: allow signing a message with spend or view keymoneromooo-monero2020-08-281-18/+81
| |
* | Integrate CLSAGs into moneromoneromooo-monero2020-08-271-30/+64
| | | | | | | | They are allowed from v12, and MLSAGs are rejected from v13.
* | Merge pull request #6746Alexander Blair2020-08-271-2/+2
|\ \ | | | | | | | | | 20f603c6b Fix broken multisig pubkey sorting (Jason Rhinelander)
| * | Fix broken multisig pubkey sortingJason Rhinelander2020-08-051-2/+2
| |/ | | | | | | | | | | | | | | | | | | The sort predicate is a boolean ordered-before value, but these are returning the memcmp value directly, and thus returns true whenever the pubkeys aren't equal. This means: - it isn't actually sorting. - it can (and does) segfault for some inputs.
* | Merge pull request #6752Alexander Blair2020-08-271-11/+21
|\ \ | | | | | | | | | | | | 85899230d simplewallet: allow setting tx keys when sending to a subaddress (moneromooo-monero) e916201f1 wallet2: fix setting tx keys when another is already set (moneromooo-monero)
| * | simplewallet: allow setting tx keys when sending to a subaddressmoneromooo-monero2020-08-101-1/+11
| | | | | | | | | | | | The tx key derivation is different then
| * | wallet2: fix setting tx keys when another is already setmoneromooo-monero2020-08-081-10/+10
| |/ | | | | | | | | insert doesn't actually insert if another element with the same key is already in the map
* | Merge pull request #6731Alexander Blair2020-08-271-1/+1
|\ \ | | | | | | | | | db5d6e460 wallet2: fix wrong name when checking RPC cost (moneromooo-monero)
| * | wallet2: fix wrong name when checking RPC costmoneromooo-monero2020-07-311-1/+1
| |/
* | Merge pull request #6690Alexander Blair2020-08-271-45/+196
|\ \ | | | | | | | | | 7175dcb10 replace most boost serialization with existing monero serialization (moneromooo-monero)
| * | replace most boost serialization with existing monero serializationmoneromooo-monero2020-08-171-45/+196
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reduces the attack surface for data that can come from malicious sources (exported output and key images, multisig transactions...) since the monero serialization is already exposed to the outside, and the boost lib we were using had a few known crashers. For interoperability, a new load-deprecated-formats wallet setting is added (off by default). This allows loading boost format data if there is no alternative. It will likely go at some point, along with the ability to load those. Notably, the peer lists file still uses the boost serialization code, as the data it stores is define in epee, while the new serialization code is in monero, and migrating it was fairly hairy. Since this file is local and not obtained from anyone else, the marginal risk is minimal, but it could be migrated later if needed. Some tests and tools also do, this will stay as is for now.