aboutsummaryrefslogtreecommitdiff
path: root/src/wallet/wallet2.cpp
Commit message (Collapse)AuthorAgeFilesLines
...
| * | wallet2: do not assume imported outputs must be non emptymoneromooo-monero2022-09-061-2/+2
| | |
| * | wallet2: prevent importing outputs in a hot walletmoneromooo-monero2022-09-061-0/+2
| | |
| * | wallet2: fix missing subaddress indices in "light" exported outputsmoneromooo-monero2022-09-061-1/+3
| |/
* / Fix missing semi-colon in error messagej-berman2022-09-061-1/+1
|/ | | | Co-authored-by: woodser <woodser@protonmail.com>
* Merge pull request #8433luigi11112022-07-141-9/+14
|\ | | | | | | db8181a wallet2: force using output distribution for ringct outs (tobtoht)
| * wallet2: force using output distribution for ringct outstobtoht2022-06-251-9/+14
| | | | | | | | Co-authored-by: j-berman <justinberman@protonmail.com>
* | derive multisig tx secret keys from an entropy source plus the tx inputs' ↵koe2022-07-131-0/+4
| | | | | | | | key images
* | Merge pull request #8149luigi11112022-07-131-88/+210
|\ \ | | | | | | | | | c7b2944 multisig: fix critical vulnerabilities in signing (anon)
| * | multisig: fix critical vulnerabilities in signinganon2022-06-301-88/+210
| |/
* | Merge pull request #8408luigi11112022-07-061-37/+0
|\ \ | | | | | | | | | 09a597b wallet2: don't use DNS to obtain segregation heights (tobtoht)
| * | wallet2: don't use DNS to obtain segregation heightstobtoht2022-06-281-37/+0
| |/
* | Merge pull request #8404luigi11112022-07-061-26/+1
|\ \ | | | | | | | | | 50ccc7e wallet2: remove obsolete rpc version check (tobtoht)
| * | wallet2: remove obsolete rpc version checktobtoht2022-06-241-26/+1
| |/
* | Merge pull request #8394luigi11112022-07-061-1/+2
|\ \ | | | | | | | | | 4278a3a cryptonote_basic: catch crypto api errors (moneromooo-monero)
| * | cryptonote_basic: catch crypto api errorsmoneromooo-monero2022-06-201-1/+2
| |/
* | Merge pull request #8388luigi11112022-07-051-7/+11
|\ \ | | | | | | | | | 04c0da2 Chunk /gettransactions to avoid hitting restricted RPC limit (tobtoht)
| * | Chunk /gettransactions to avoid hitting restricted RPC limittobtoht2022-06-271-7/+11
| |/
* / Improve consistency between on_money_received and on_money_received_unconfirmedLuke Parker2022-06-011-3/+4
|/ | | | | | unconfirmed solely uses a - b, and received now accepts b so it can provide more detailed logs on what occurred (printing a - b, yet with a and b).
* Merge pull request #8331luigi11112022-05-261-9/+5
|\ | | | | | | fde7c96 wallet2: fix spurious reorg detection with untrusted nodes (moneromooo-monero)
| * wallet2: fix spurious reorg detection with untrusted nodesmoneromooo-monero2022-05-151-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When forced to deal with an untrusted node, a wallet will quantize its current height to disguise the real height to the adversary, to try and minimize the daemon's ability to distinguish returning wallets. Daemons will thus return more blocks than the wallet needs, starting from earlier in the chain. These extra blocks will be disregarded by the wallet, which had already scanned them. However, for the purposes of reorg size detection, the wallet assumes all blocks the daemon sends are different, which is only correct if the wallet hasn't been coy, which is only the case for trusted daemons (which you should use). This causes an issue when the size of this "fake reorg" is above the sanity check threshold at which the wallet refuses a reorg. To fix this, the reorg size check is moved later on, when the reorg is about to actually happen, after the wallet has checked which blocks are actually different from the ones it expects.
* | Merge pull request #8328luigi11112022-05-261-1/+8
|\ \ | | | | | | | | | 2979474 disable multisig by default (moneromooo-monero)
| * | disable multisig by defaultmoneromooo-monero2022-05-171-1/+8
| |/ | | | | | | | | | | | | | | | | There are vulnerabilities in multisig protocol if the parties do not trust each other, and while there is a patch for it, it has not been throroughly reviewed yet, so it is felt safer to disable multisig by default for now. If all parties in a multisig setup trust each other, then it is safe to enable multisig.
* | Merge pull request #8308luigi11112022-05-161-6/+1
|\ \ | | | | | | | | | f63f7dc wallet2: fix a couple unused variable warnings (selsta)
| * | wallet2: fix a couple unused variable warningsselsta2022-05-021-6/+1
| | |
* | | wallet2: speedup large tx construction: reserve vector memorymoneromooo-monero2022-05-131-1/+7
| | | | | | | | | | | | 2.8 seconds -> 2.6 seconds on a test case
* | | wallet2: speedup large tx construction: batch ringdb lookupsCrypto City2022-05-131-4/+35
| | | | | | | | | | | | 3.3 seconds -> 2.8 seconds on a test case
* | | wallet2: speedup large tx construction: batch ringdb updatesCrypto City2022-05-131-2/+14
| | | | | | | | | | | | 5.2 seconds -> 4.1 seconds on a test case
* | | wallet2: speedup large tx construction: cache public key validitymoneromooo-monero2022-05-131-23/+31
| |/ |/| | | | | 5.9 second -> 5.2 seconds on a test case
* | Merge pull request #8277luigi11112022-05-101-3/+1
|\ \ | | | | | | | | | baee2c0 Preserve commitment format inside transactions (kayabaNerve)
| * | Preserve commitment format inside transactionsLuke Parker2022-04-211-3/+1
| | |
* | | Merge pull request #8220luigi11112022-05-101-3/+6
|\ \ \ | |_|/ |/| | | | | 0d6ecb1 multisig: add post-kex verification round to check that all participants have completed the multisig address (koe)
| * | multisig: add post-kex verification round to check that all participants ↵koe2022-04-291-3/+6
| | | | | | | | | | | | have completed the multisig address
* | | Merge pull request #8178luigi11112022-04-251-0/+4
|\ \ \ | |_|/ |/| | | | | 5d388eb Bump ring size to 16 for v15 & remove set default in wallet cli (j-berman)
| * | Bump ring size to 16 for v15 & remove set default in wallet clij-berman2022-04-181-0/+4
| | |
* | | Merge pull request #8254luigi11112022-04-191-1/+1
|\ \ \ | | | | | | | | | | | | 9df7179 wallet2: use BP+ for cold signing (moneromooo-monero)
| * | | wallet2: use BP+ for cold signingmoneromooo-monero2022-04-101-1/+1
| | | | | | | | | | | | | | | | reported by ukoehb
* | | | Add view tags to outputs to reduce wallet scanning timej-berman2022-04-181-128/+177
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements view tags as proposed by @UkoeHB in MRL issue https://github.com/monero-project/research-lab/issues/73 At tx construction, the sender adds a 1-byte view tag to each output. The view tag is derived from the sender-receiver shared secret. When scanning for outputs, the receiver can check the view tag for a match, in order to reduce scanning time. When the view tag does not match, the wallet avoids the more expensive EC operations when deriving the output public key using the shared secret.
* | | Fee changes from ArticMinemoneromooo-monero2022-04-101-32/+62
|/ / | | | | | | | | | | https://github.com/ArticMine/Monero-Documents/blob/master/MoneroScaling2021-02.pdf with a change to use 1.7 instead of 2.0 for the max long term increase rate
* | Merge pull request #8197luigi11112022-04-061-1/+1
|\ \ | | | | | | | | | da9aa1f Copyright: Update to 2022 (mj-xmr)
| * | Copyright: Update to 2022mj-xmr2022-03-041-1/+1
| |/
* | Merge pull request #8195luigi11112022-04-061-4/+4
|\ \ | | | | | | | | | 3c82958 wallet2: update stagenet rollback blocks (selsta)
| * | wallet2: update stagenet rollback blocksselsta2022-03-011-4/+4
| | |
* | | Merge pull request #8179luigi11112022-04-061-7/+125
|\ \ \ | | | | | | | | | | | | ae0a840 wallet2: decrease the amount of data exchanged for output export (moneromooo-monero)
| * | | wallet2: decrease the amount of data exchanged for output exportmoneromooo-monero2022-03-291-7/+125
| | |/ | |/|
* | | Merge pull request #7170luigi11112022-04-051-38/+51
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bd27deb Bulletproofs+ (SarangNoether) b7713cc Precompute initial transcript hash (SarangNoether) b535d66 Updates from security audit (SarangNoether) a0d80b1 plug bulletproofs plus into consensus (moneromooo-monero) 75bd004 ringct: a few minor optimizations from review (moneromooo-monero) a345060 ringct: port some of vtnerd's review changes from BP+ to BP (moneromooo-monero) 4c94cfe store outPk/8 in the tx for speed (moneromooo-monero) 5acdd0e bulletproofs+: some minor cleanup from vtnerd's review (moneromooo-monero)
| * | | store outPk/8 in the tx for speedmoneromooo-monero2022-04-051-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | It avoids dividing by 8 when deserializing a tx, which is a slow operation, and multiplies by 8 when verifying and extracing the amount, which is much faster as well as less frequent
| * | | plug bulletproofs plus into consensusmoneromooo-monero2022-04-051-37/+48
| | | |
* | | | Merge pull request #7153luigi11112022-04-051-0/+7
|\ \ \ \ | |/ / / |/| | | | | | | 354e1d9 Make the wallet name optional when locked. (Norman Moeschter)
| * | | Make the wallet name optional when locked.Norman Moeschter2022-03-101-0/+7
| |/ /
* | | Merge pull request #8158luigi11112022-03-181-0/+13
|\ \ \ | |/ / |/| | | | | e5000a9 Balance includes unconfirmed transfers to self (woodser)