| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |\
| |
| |
| | |
6177e49 Added logging for dropped local txes with no i2p/tor connections (vtnerd)
|
| | | |
|
| |\ \
| | |
| | |
| | | |
62ab7fd p2p: plug tor to clearnet association vector (moneromooo-monero)
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
During the handshake for an incoming connection, the peer id is checked against the local node's peer id only for the specific zone of the incoming peer, in order to avoid linking public addresses to tor addresses:
https://github.com/monero-project/monero/blob/5d7ae2d2791c0244a221872a7ac62627abb81896/src/p2p/net_node.inl#L2343
However, on handshakes for outgoing connections, all zones are checked:
https://github.com/monero-project/monero/blob/5d7ae2d2791c0244a221872a7ac62627abb81896/src/p2p/net_node.inl#L1064
If an attacker wanted to link a specific tor node to a public node, they could potentially connect to as many public nodes as possible, get themselves added to the peer whitelist, maybe stuff some more attacker-owned addresses into the greylist, then disconnect, and for any future incoming connections, respond with the tor node's id in an attempt to link the public/tor addresses.
|
| |\ \
| | |
| | |
| | | |
ac1a34c build: prepare v0.15.0.5 update (selsta)
|
| | |/ |
|
| |\ \
| | |
| | |
| | | |
d438e6a wallet: fix exceptions getting the hash of a pruned tx (moneromooo-monero)
|
| | |/ |
|
| |/ |
|
| | |
|
| |\
| |
| |
| | |
74b0ca4 v0.15.0.1 version bump (moneromooo-monero)
|
| | | |
|
| |/
|
|
| |
Don't leave stdout/stderr dangling on a fork.
|
| | |
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
912ff6abe simplewallet: plug a timing leak (moneromooo-monero)
cc2fc0bc3 epee: allow a random component in once_a_time timeouts (moneromooo-monero)
c0f504787 wallet: reuse cached height when set after refresh (moneromooo-monero)
f98d9673e wallet2: fix is_synced checking target height, not height (moneromooo-monero)
98cdc8492 wallet: fix another facet of "did I get some monero" information leak (moneromooo-monero)
69b8aa5a2 wallet2: do not send an unnecessary last getblocks.bin call on refresh (moneromooo-monero)
2ca057402 wallet2: do not repeatedly ask for pool txes sent to us (moneromooo-monero)
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
As reported by Tramèr et al, timing of refresh requests can be used
to see whether a password was requested (and thus at least one output
received) since this will induce a delay in subsequent calls.
To avoid this, we schedule calls at a given time instead of sleeping
for a set time (which would make delays additive).
To further avoid a scheduled call being during the time in which a
password is prompted, the actual scheduled time is now randomized.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Refreshing sets cached height, which is otherwise got by calling
get_info. Since get_info is called upon needing to display a prompt
after a command has finished, it can be used to determine how much
time a given command took to run if the cache timeout lapses while
the command runs. Refreshing caches the height as a side effect, so
get_info will never be called as a result of displaying a prompt
after refreshing (and potentially leaking how much time it took to
process a set of transactions, therefore leaking whether we got
some monero in them).
|
| | |
| |
| |
| |
| |
| | |
Target height would be appropriate for the daemon, which syncs
off other daemons, but the wallet syncs off the daemon it's
connected to, and its target is the daemon's current height.
|
| | |
| |
| |
| |
| |
| |
| | |
We get new pool txes before processing any tx, pool or not.
This ensures that if we're asked for a password, this does not
cause a measurable delay in the txpool query after the last
block query.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The "everything refreshed" state was detected when a refresh call did
not return any new blocks. This can be detected without that extra
"empty" call by comparing the claimed node height to the height of
the last block retrieved. Doing this avoids that last call, saves
some bandwidth, and makes the common refresh case use only one call
rather than two.
As a side effect, it prevents an information leak reported by
Tramèr et al: if the wallet retrieves a set of blocks which includes
an output sent to the refreshing wallet, the wallet will prompt the
user for the password to decode the amount and calculate the key
image for the new output, and this will delay subsequent calls to
getblocks.bin, allowing a passive adversary to note the delay and
deduce when the wallet receives at least one output.
This can still happen if the wallet downloads more than 1000 blocks,
since this will be split in several calls, but then the most the
adversary can tell is which 1000 block section the user received
some monero (the adversary can estimate the heights of the blocks
by calculating how many "large" transfers are done, which will be
sections of blocks, the last of which will usually be below 1000,
but the size of the data should allow the actual number of blocks
sent to be determined fairly accurately).
This timing trick still be used via the subsequent scan for incoming
txes in the txpool, which will be fixed later.
|
| | |
| |
| |
| |
| |
| |
| |
| | |
This lets a passive attacker with access to the network link
between node and wallet perform traffic analysis to deduce
when an idle wallet receives a transaction.
Reported by Tramèr et al.
|
| | |
| |
| |
| | |
e4d1674e8 0.15.0.0 release engineering (Riccardo Spagni)
|
| |\ \
| | |
| | |
| | | |
d31024c2e cryptonote: untangle dependency from miner to blockchain (moneromooo-monero)
|
| | |/
| |
| |
| | |
It causes link errors at least on mac
|
| |\ \
| |/
|/|
| | |
0e3b823a1 daemon: always use bootstrap daemon (if set) in '--no-sync' mode (xiphon)
|
| | | |
|
| |\ \
| | |
| | |
| | | |
86ac20f blockchain: fix unwanted error when probing the pool for a tx (moneromooo-monero)
|
| | | | |
|
| |/ / |
|
| |\ \
| | |
| | |
| | | |
5734686 rpc: Include tag in get_info version string (ndorf)
|
| | | | |
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | | |
296ec7c device: bounds checking in Ledger send_secret/receive_secret (xiphon)
|
| | | | | |
|
| |\| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
d25acd7 Add hmac over encrypted value during transaction (clashm)
34f28a7 Add display address (clashm)
235b94e Revert PR #5835 (export view key) (clashm)
32febd2 Fix debug feature (clashm)
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | | |
174a6ac tx_pool: fix divide by 0 in log (moneromooo-monero)
|
| | | |_|/
| |/| |
| | | |
| | | | |
Coverity 205394
|
| |\ \ \ \
| | | | |
| | | | |
| | | | | |
a4dc575 rpc: add a flush_cache RPC (moneromooo-monero)
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This allows flushing internal caches (for now, the bad tx cache,
which will allow debugging a stuck monerod after it has failed to
verify a transaction in a block, since it would otherwise not try
again, making subsequent log changes pointless)
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | | |
643dcb9 net: link with libzmq (moneromooo-monero)
|
| | | |/ / /
| |/| | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | | |
65e8a89 Change monerod --proxy to --tx-proxy (vtnerd)
|
| | | | | | | |
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | | |
d60c1b6 changed 'batttery' to 'battery' (jakehemmerle)
|
| | |/ / / / / |
|
