summaryrefslogtreecommitdiff
path: root/src/wallet/wallet2.cpp
Commit message (Collapse)AuthorAgeFilesLines
* addressbook updatesJaquee2016-12-121-4/+11
|
* Merge pull request #1434Riccardo Spagni2016-12-121-0/+17
|\ | | | | | | 5df92877 GUI address book (Jaquee)
| * GUI address bookJaquee2016-12-121-0/+17
| |
* | wallet: bring forward use of the dynamic feemoneromooo-monero2016-12-101-1/+1
|/ | | | | | It was 14 days after v4, it is now one day after it. luigi1111's suggestion
* Merge pull request #1422Riccardo Spagni2016-12-101-1/+1
|\ | | | | | | 9b00d80f wallet2: pass by const ref where possible (moneromooo-monero)
| * wallet2: pass by const ref where possiblemoneromooo-monero2016-12-101-1/+1
| | | | | | | | Reported by kenshi84
* | Merge pull request #1419Riccardo Spagni2016-12-101-2/+7
|\ \ | | | | | | | | | f4a3ce15 Fix sending outputs from a tx with more than one pubkey (moneromooo-monero)
| * | Fix sending outputs from a tx with more than one pubkeymoneromooo-monero2016-12-091-2/+7
| |/ | | | | | | | | | | A bug in cold signing caused a spurious pubkey to be included in transactions, so we need to ensure we use the correct one when sending outputs from one of those.
* | Merge pull request #1415Riccardo Spagni2016-12-081-3/+13
|\ \ | | | | | | | | | 6c44f5c6 wallet: send 0 change to a random address where necessary with rct (moneromooo-monero)
| * | wallet: send 0 change to a random address where necessary with rctmoneromooo-monero2016-12-071-3/+13
| |/ | | | | | | | | | | | | | | | | | | | | If a rct transaction would cause no change to be generated, a zero change output is added, and sent to a randomly generated address. This ensures that no transaction will be sent with just one output, which could cause the receiver to be able to determine which of the inputs in the sent rings is the real one. This is very rare, since it requires the sum of outputs to be equal to the sum of outputs plus the fee, which is now a function of the last few blocks.
* / Remove infinite loop in refresh codeJaquee2016-12-051-68/+42
|/ | | | This reverts commit fd181b03bb58a8b0628d2af8637cf6bb968fc437.
* Merge pull request #1399Riccardo Spagni2016-12-041-2/+1
|\ | | | | | | 73ac3b8e wallet2: avoid possible undefined behavior on empty string (moneromooo-monero)
| * wallet2: avoid possible undefined behavior on empty stringmoneromooo-monero2016-12-041-2/+1
| |
* | Merge pull request #1398Riccardo Spagni2016-12-041-4/+4
|\ \ | | | | | | | | | f4772bae Fix a few minor typos (Pierre Boyer)
| * | Fix a few minor typosPierre Boyer2016-12-041-4/+4
| |/
* | Merge pull request #1385Riccardo Spagni2016-12-041-0/+142
|\ \ | | | | | | | | | | | | | | | 5783dd8c tests: add unit tests for uri parsing (moneromooo-monero) 82ba2108 wallet: add API and RPC to create/parse monero: URIs (moneromooo-monero) d9001b43 epee: add functions to convert from URL format (ie, %XX values) (moneromooo-monero)
| * | wallet: add API and RPC to create/parse monero: URIsmoneromooo-monero2016-11-281-0/+142
| |/
* | Merge pull request #1380Riccardo Spagni2016-12-041-0/+5
|\ \ | | | | | | | | | 657a70e0 wallet: add a getter for the filename path (moneromooo-monero)
| * | wallet: add a getter for the filename pathmoneromooo-monero2016-11-261-0/+5
| |/
* / Improve daemon RPC version handlingmoneromooo-monero2016-11-261-4/+4
|/ | | | | | | Daemon RPC version is now composed of a major and minor number, so that incompatible changes bump the major version, while compatible changes can still bump the minor version without causing clients to unnecessarily complain.
* Merge pull request #1369Riccardo Spagni2016-11-241-2/+4
|\ | | | | | | | | 6d76072 simplewallet: remove double confirmation when submitting signed tx (moneromooo-monero) 92dea04 wallet2: fix wrong change being recorded for cold signed txes (moneromooo-monero)
| * wallet2: fix wrong change being recorded for cold signed txesmoneromooo-monero2016-11-231-2/+4
| |
* | Merge pull request #1361Riccardo Spagni2016-11-241-3/+3
|\ \ | | | | | | | | | | | | | | | | | | 1d9e223 rpc: do not include output indices for pool txes (moneromooo-monero) e227d6e rpc: bump version after RPC changes (moneromooo-monero) 2c0173c Add a get_outs (fully text based) version of get_outs.bin (moneromooo-monero) e05907b rpc: add output indices to gettransactions (moneromooo-monero)
| * | Add a get_outs (fully text based) version of get_outs.binmoneromooo-monero2016-11-221-3/+3
| |/
* | Merge pull request #1358Riccardo Spagni2016-11-241-7/+51
|\ \ | | | | | | | | | 7abfc54 wallet: fix exporting outputs and key images with txes with two pubkeys (moneromooo-monero)
| * | wallet: fix exporting outputs and key images with txes with two pubkeysmoneromooo-monero2016-11-191-7/+51
| |/ | | | | | | | | This also needs to make sure to pick the correct one, in the case where cold signing caused to tx keys to be included.
* | Merge pull request #1352Riccardo Spagni2016-11-241-1/+1
|\ \ | | | | | | | | | d01f5c7 wallet2: fix illegal memory access removing newlines from password (moneromooo-monero)
| * | wallet2: fix illegal memory access removing newlines from passwordmoneromooo-monero2016-11-171-1/+1
| |/
* | Merge pull request #1351Riccardo Spagni2016-11-241-4/+5
|\ \ | | | | | | | | | e6264a2 wallet2: fix decrypting a bit too much in authenticated mode (moneromooo-monero)
| * | wallet2: fix decrypting a bit too much in authenticated modemoneromooo-monero2016-11-171-4/+5
| |/
* / Clamp refresh from height to blockchain height.Dion Ahmetaj2016-11-181-42/+68
|/
* Merge pull request #1318Riccardo Spagni2016-11-171-0/+28
|\ | | | | | | 4fca34d Wallet2: calculate approximate blockchain height on offline creation (Jacob Brydolf)
| * Wallet2: calculate approximate blockchain height on offline creationJacob Brydolf2016-11-131-0/+28
| | | | | | | | Wallet API: add approximateBlockChainHeight()
* | wallet: auto sync outputs and key images in cold signing filesmoneromooo-monero2016-11-161-1/+32
| | | | | | | | | | | | | | | | | | When passing around unsigned and signed transactions, outputs and key images are passed along (outputs are passed along unsigned transactions from the hot wallet to the cold wallet, key images are passed along with signed transations from the cold wallet to the hot wallet), to allow more user friendly syncing between hot and cold wallets.
* | wallet2: fill key image and pubkey maps when importing outputsmoneromooo-monero2016-11-161-0/+2
| |
* | wallet: cast indices to string in logs to be nice to CLANGmoneromooo-monero2016-11-161-4/+4
| |
* | wallet2: try all tx keys when scanning a new transactionmoneromooo-monero2016-11-161-2/+7
| | | | | | | | | | | | The vast majority of transactions will have just one tx pubkey, but a bug with cold wallet signing caused two such keys to be there, with the second one being the real one.
* | wallet2: fill in key image map when importing key imagesmoneromooo-monero2016-11-151-0/+1
|/
* Merge pull request #1316Riccardo Spagni2016-11-111-1/+343
|\ | | | | | | 358e068 Created monero-wallet-rpc, moving functionality from monero-wallet-cli (Lee Clagett)
| * Created monero-wallet-rpc, moving functionality from monero-wallet-cliLee Clagett2016-11-101-1/+343
| |
* | wallet: fix corner case of no recent outputs availablemoneromooo-monero2016-11-091-6/+9
|/ | | | Also clarify related logs
* Merge pull request #1305Riccardo Spagni2016-11-091-18/+24
|\ | | | | | | c80f4d4 wallet: fix output collision detection for view wallets (moneromooo-monero)
| * wallet: fix output collision detection for view walletsmoneromooo-monero2016-11-071-18/+24
| | | | | | | | | | | | | | | | | | | | | | | | View wallets do not have the spend secret key, and are thus unable to derive key images for incoming outputs. Moreover, a previous patch set key images to zero as a means to mark an output as having an unknown key image, so they could be filled in when importing key images at a later time. That later patch caused spurious collisions. We now use public keys to detect duplicate outputs. Public keys obtained from the blockchain are checked to be identical to the ones derived locally, so can't be spoofed.
* | Merge pull request #1306Riccardo Spagni2016-11-081-0/+55
|\ \ | | | | | | | | | 8aba0d4 wallet: encrypt outputs and key images files with the view key (moneromooo-monero)
| * | wallet: encrypt outputs and key images files with the view keymoneromooo-monero2016-11-071-0/+55
| |/ | | | | | | | | | | This key is available to both cold and hot wallet. Authenticated encryption will guard against interception and/or modification of the file.
* | Merge pull request #1285Riccardo Spagni2016-11-081-16/+13
|\ \ | | | | | | | | | a970a4e refresh speedup (luigi1111)
| * | refresh speedupluigi11112016-11-011-16/+13
| | | | | | | | | | | | Compute derivation only once per tx, instead of once per output. Approx 33% faster while using 75% as much CPU on my machine. Note old functions in cryptonote_core (lookup_acc_outs and is_out_to_acc) are still used by tests.
* | | Merge pull request #1276Riccardo Spagni2016-11-081-4/+40
|\ \ \ | |_|/ |/| | | | | | | | | | | 18f66f4 wallet: use the dynamic per kB fee (moneromooo-monero) e6deb8a rpc: add a dynamic fee estimation RPC call (moneromooo-monero) 82dbba1 core: dynamic fee algorithm from ArticMine (moneromooo-monero)
| * | wallet: use the dynamic per kB feemoneromooo-monero2016-10-311-4/+40
| | |
* | | wallet: fix bad amounts/fees againmoneromooo-monero2016-11-021-0/+12
| |/ |/| | | | | | | | | | | | | | | | | | | m_amount_out was sometimes getting initialized with the sum of an transaction's outputs, and sometimes with the sum of outputs that were not change. This caused confusion and bugs. We now always set it to the sum of outputs. This reverts an earlier fix for bad amounts as this used the other semantics. The wallet data should be converted automatically in a percentage of cases that I'm hesitant to estimate. In any case, restoring from seed or keys or rebuilding the cache will get it right.