| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| ... | |
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This plugs a privacy leak from the wallet to the daemon,
as the daemon could previously see what input is included
as a transaction input, which the daemon hadn't previously
supplied. Now, the wallet requests a particular set of
outputs, including the real one.
This can result in transactions that can't be accepted if
the wallet happens to select too many outputs with non standard
unlock times. The daemon could know this and select another
output, but the wallet is blind to it. It's currently very
unlikely since I don't think anything uses non default
unlock times. The wallet requests more outputs than necessary
so it can use spares if any of the returns outputs are still
locked. If there are not enough spares to reach the desired
mixin, the transaction will fail.
|
| | |
|
| | |
|
| |
|
|
|
|
| |
It's not really needed, it used to be an optimization for when
that code was not using the db and needed to recalculate things
fast on startup.
|
| |
|
|
|
|
|
|
|
|
|
| |
Compilation of bitmonero on Arch with gcc 6.1 results in the following
error:
/home/mwo/bitmonero/tests/unit_tests/hardfork.cpp: In member function ‘virtual void TestDB::set_hard_fork_version(uint64_t, uint8_t)’:
/home/mwo/bitmonero/tests/unit_tests/hardfork.cpp:132:5: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation]
if (versions.size() <= height) versions.resize(height+1); versions[height] = version;
This can be fixed by simply unfolding this line into three lines.
|
| |\
| |
| |
| |
| |
| |
| |
| | |
9ef8c7b tests: fix tests broken by the removal of the block reward accumulation loop (moneromooo-monero)
a6e717e cn_deserialize: deserialize tx_extra too (moneromooo-monero)
3eff37f unit_tests: add a write_varint/read_varint test (moneromooo-monero)
7a66387 unit_tests: fix UNBOUND_LIBRARIES/UNBOUND_LIBRARY typo (moneromooo-monero)
d6bce4b core: move tx_extra parsing errors to log level 1 (moneromooo-monero)
|
| | | |
|
| | | |
|
| |\ \
| |/
|/| |
|
| | |\ |
|
| | | |
| | |
| | |
| | |
| | | |
drop obsolete remove_output()
fix get_output_key(global), fix crash in blockchain_dump
|
| |/ / |
|
| | |
| |
| |
| |
| | |
These can be compiled out of libunbound, leading to failure
to check DNSSEC validity.
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This is a list of existing output amounts along with the number
of outputs of that amount in the blockchain.
The daemon command takes:
- no parameters: all outputs with at least 3 instances
- one parameter: all outputs with at least that many instances
- two parameters: all outputs within that many instances
The default starts at 3 to avoid massive spamming of all dust
outputs in the blockchain, and is the current minimum mixin
requirement.
An optional vector of amounts may be passed, to request
histogram only for those outputs.
|
| | |
| |
| |
| |
| |
| | |
Ain't nobody got time for link/cmake skullduggery.
This reverts commit fff238ec94ac6d45fc18c315d7bc590ddfaad63d.
|
| |/
|
|
| |
Useful for debugging users' logs
|
| |
|
|
| |
Match changed BlockchainDB function declaration.
|
| |\
| |
| |
| | |
5ab33ca unit_tests: update test openalias address (moneromooo-monero)
|
| | |
| |
| |
| | |
It was recently changed
|
| |/
|
|
|
| |
This avoids the need to define that variable in every program
which uses epee.
|
| | |
|
| | |
|
| |\
| |
| |
| |
| |
| |
| | |
7658ac0 blockchain: revert handle_get_objects adding block id on tx not found (moneromooo-monero)
3a0f4d8 berkeleydb: fix delete/free mismatch (moneromooo-monero)
1642be2 minor bugfixes and refactoring (Thomas Winget)
098dcf2 unit_tests: fix mnemonics unit test testing invalid seeds (moneromooo-monero)
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Some word triplets, such as "mugged names nail", are not valid
results from any 32 bit value. If used to decode a 32 bit value,
the result will therefore encode to a different word triplet.
Fix this by using random words converted from an actual random
bitstring, ensuring we always get valid triplets.
|
| |/ |
|
| | |
|
| |
|
|
| |
Needed to add the corresponding (dummy) method to unit test hardfork
|
| |
|
|
| |
open isn't actually called in those tests
|
| | |
|
| | |
|
| |
|
|
|
| |
The dummy blockchain class needed to have the newly added
is_read_only virtual function.
|
| | |
|
| |
|
|
|
| |
A couple stopped passing when the hard fork code was made
to reject incoming hard fork versions it did not know about.
|
| | |
|
| |
|
|
|
| |
This would fail, as the post hard fork settings would yield
different data, and the test expects pre hard fork data.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
Also add some more tests, and rename some instances of
"version" and "add" for clarity.
NOTE: the starting height values are sometimes wrong.
I suspect this is due to the hard fork reorg code being
buggy, since they're good when syncing after the fact.
However, they're not actually used by the consensus code,
so I'm ignoring this for now, but this needs debugging.
|
| |
|
|
| |
version 2
|
| |
|
|
|
| |
It returns the ideal version for a given height, which is
based on the minimum height for a fork, disregarding votes
|
| |
|
|
| |
And setup the first fork to not vote
|
| | |
|
| |
|
|
| |
git history's here if needed to get any of this back
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Using major version would cause older daemons to reject those
blocks as they fail to deserialize blocks with a major version
which is not 1. There is no such restriction on the minor
version, so switching allows older daemons to coexist with
newer ones till the actual fork date, when most will hopefully
have updated already.
Also, for the same reason, we consider a vote for 0 to be a
vote for 1, since older daemons set minor version to 0.
|
| | |
|
