summaryrefslogtreecommitdiff
path: root/tests/unit_tests
Commit message (Collapse)AuthorAgeFilesLines
* unit_tests: fix strtoul unit testjeffro2562024-03-071-1/+0
|
* Merge pull request #9158luigi11112024-02-241-5/+34
|\ | | | | | | 33e3f72 serialization: fix infinite loops and clean up dispatching (jeffro256)
| * serialization: fix infinite loops and clean up dispatchingjeffro2562024-02-081-5/+34
| | | | | | | | Resolves #8687
* | Merge pull request #9145luigi11112024-02-241-0/+1
|\ \ | | | | | | | | | f5b8634 Add <cstdint> to aligned test (Lee *!* Clagett)
| * | Add <cstdint> to aligned testLee *!* Clagett2024-01-291-0/+1
| |/
* | Merge pull request #9142luigi11112024-02-241-42/+37
|\ \ | | | | | | | | | 98ee46f Disable/fix ports with I2P (Lee Clagett)
| * | Disable/fix ports with I2PLee Clagett2024-01-301-42/+37
| |/
* / wallet: feature: transfer amount with fee includedjeffro2562024-02-201-0/+57
|/ | | | | | | | | | | | | | | | | To transfer ~5 XMR to an address such that your balance drops by exactly 5 XMR, provide a `subtractfeefrom` flag to the `transfer` command. For example: transfer 76bDHojqFYiFCCYYtzTveJ8oFtmpNp3X1TgV2oKP7rHmZyFK1RvyE4r8vsJzf7SyNohMnbKT9wbcD3XUTgsZLX8LU5JBCfm 5 subtractfeefrom=all If my walet balance was exactly 30 XMR before this transaction, it will be exactly 25 XMR afterwards and the destination address will receive slightly less than 5 XMR. You can manually select which destinations fund the transaction fee and which ones do not by providing the destination index. For example: transfer 75sr8AAr... 3 74M7W4eg... 4 7AbWqDZ6... 5 subtractfeefrom=0,2 This will drop your balance by exactly 12 XMR including fees and will spread the fee cost proportionally (3:5 ratio) over destinations with addresses `75sr8AAr...` and `7AbWqDZ6...`, respectively. Disclaimer: This feature was paid for by @LocalMonero.
* Merge pull request #9080luigi11112024-01-181-0/+7
|\ | | | | | | 47d8899 Fix missing checks for IsObject in ZMQ jsonrpc reading (Lee Clagett)
| * Fix missing checks for IsObject in ZMQ jsonrpc readingLee Clagett2023-11-281-0/+7
| |
* | add a test for the long term weight cacheBoog9002023-10-021-0/+35
| |
* | wallet2: fix `store_to()` and `change_password()`jeffro2562023-08-232-0/+267
|/ | | | | | | | Resolves #8932 and: 2. Not storing cache when new path is different from old in `store_to()` and 3. Detecting same path when new path contains entire string of old path in `store_to()` and 4. Changing your password / decrypting your keys (in this method or others) and providing a bad original password and getting no error and 5. Changing your password and storing to a new file
* common: do not use DNS to determine if address is localtobtoht2023-05-252-0/+51
| | | | Co-authored-by: j-berman <justinberman@protonmail.com>
* verRctNonSemanticsSimpleCached: fix fragilityJeffrey Ryan2023-03-172-0/+427
|
* Merge pull request #8649luigi11112023-01-111-0/+11
|\ | | | | | | 2c24322 DNSResolver: fix not handling hostnames without dot characters [release] (Jeffrey Ryan)
| * DNSResolver: fix not handling hostnames without dot characters [release]Jeffrey Ryan2022-11-211-0/+11
| | | | | | | | Unrelated, but similar code-wise to #8643. There is a check in `DNSResolver` which automatically fails to resolve hostnames which do not contain the `.` character. This PR removes that check.
* | Merge pull request #8644luigi11112023-01-111-0/+35
|\ \ | | | | | | | | | 38d4811 p2p: fix exclusive node DNS resolution for certain hosts [release] (Jeffrey Ryan)
| * | p2p: fix exclusive node DNS resolution for certain hosts [release]Jeffrey Ryan2022-11-171-0/+35
| |/ | | | | | | | | | | | | | | Fixes #8633. The function `append_net_address` did not parse hostname + port addresses (e.g. `bar:29080`) correctly if the hostname did not contain a `'.'` character. @vtnerd comments 1 clear up 2nd conditional statement
* / Fix dandelion++ fluff/stem bug with local txesLee Clagett2022-10-251-15/+84
|/
* Merge pull request #8588luigi11112022-09-261-0/+1
|\ | | | | | | 802c4bb Move update_checkpoints() to a later stage (SChernykh)
| * Move update_checkpoints() to a later stageSChernykh2022-09-221-0/+1
| | | | | | | | update_checkpoints() makes a few DNS requests and can take up to 20-30 seconds to complete (3-6 seconds on average). It is currently called from core::handle_incoming_block() which holds m_incoming_tx_lock, so it blocks all incoming transactions and blocks processing while update_checkpoints() is running. This PR moves it to until after a new block has been processed and relayed, to avoid full monerod locking.
* | add an option to force-update multisig key exchange under some circumstanceskoe2022-09-211-24/+64
|/
* multisig: fix #8537 seed restore (suggestions by @UkoeHB)j-berman2022-09-011-1/+1
| | | | | | | | - spend secret key is no longer the sum of multisig key shares; no need to check that is the case upon restore. - restoring a multisig wallet from multisig info means that the wallet must have already completed all setup rounds. Upon restore, set the number of rounds completed accordingly.
* Merge pull request #8426luigi11112022-07-151-2/+252
|\ | | | | | | | | | | a82fba4 address PR comments (j-berman) 3be1dbd connection: fix implementation (anon) 724ff21 connection: add segfault and deadlocks demo (anon)
| * address PR commentsj-berman2022-07-081-5/+5
| |
| * connection: add segfault and deadlocks demoanon2022-07-051-2/+252
| |
* | multisig: fix critical vulnerabilities in signinganon2022-06-303-20/+21
|/
* Merge pull request #8340luigi11112022-05-261-0/+22
|\ | | | | | | | | 08080df unit_tests: add more sha256sum test cases (Jeffrey Ryan) a66a52d common: update sha256sum to use OpenSSL 3.0 API (Jeffrey Ryan)
| * unit_tests: add more sha256sum test casesJeffrey Ryan2022-05-181-0/+22
| |
* | Don't exclusively drop tor/i2p outgoing cxns in idle loopj-berman2022-05-181-2/+2
|/
* Merge pull request #8302luigi11112022-05-102-2/+2
|\ | | | | | | 41da2fe Update copyright to 2022 for Hardfork files (Akrit)
| * Update copyright to 2022 for Hardfork filesAkrit2022-04-292-2/+2
| | | | | | | | Update Makefile and LICENSE
* | Merge pull request #8220luigi11112022-05-101-8/+7
|\ \ | |/ |/| | | 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-8/+7
| | | | | | | | have completed the multisig address
* | Add Include statementsJeffrey2022-04-181-0/+1
| |
* | Fee changes from ArticMinemoneromooo-monero2022-04-102-0/+188
| | | | | | | | | | | | 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
* | epee: allow copying a rolling_median_t objectmoneromooo-monero2022-04-101-0/+18
| |
* | Merge pull request #8228luigi11112022-04-061-64/+71
|\ \ | | | | | | | | | 1ce9e9c Remove dead code from parserse_base_utils and fix unit tests (Jeffrey)
| * | Remove dead code from parserse_base_utils and fix unit testsJeffrey2022-03-301-64/+71
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove `match_string()`, `match_number()`, and `match_word()` * Remove `match_word_with_extrasymb()` and `match_word_til_equal_mark()` * Adapt unit test for `match_number()` to `match_number2()` * Adapt unit test for `match_string()` to `match_string2()` Note: the unit tests were testing for the old version of the functions, and the interfaces for these functions changed slightly, so I had to also edit the tests. As of writing, this PR has no merge conflicts with #8211 Additional changes during review: * Explicitly set up is_[float/signed]_val to be changed before each call * Structify the tests and fix uninitialized variables
* | Merge pull request #8197luigi11112022-04-0673-73/+94
|\ \ | | | | | | | | | da9aa1f Copyright: Update to 2022 (mj-xmr)
| * | Copyright: Update to 2022mj-xmr2022-03-0473-73/+94
| |/
* | plug bulletproofs plus into consensusmoneromooo-monero2022-04-051-1/+1
| |
* | Bulletproofs+Sarang Noether2022-04-042-0/+170
|/
* Merge pull request #7877luigi11112022-03-021-31/+201
|\ | | | | | | e08abaa multisig key exchange update and refactor (koe)
| * multisig key exchange update and refactorkoe2022-02-221-31/+201
| |
* | tx_pool: full tx revalidation on fork boundariesmoneromooo-monero2021-11-011-5/+10
| | | | | | | | | | | | avoids mining txes after a fork that are invalid by this fork's rules, but were valid by the previous fork rules at the time they were verified and added to the txpool.
* | Merge pull request #7874luigi11112021-10-112-36/+337
|\ \ | |/ |/| | | | | 9154883 node_server: fix race condition (anon) 8922f96 node_server: add race condition demo (anon)
| * node_server: fix race conditionanon2021-09-201-36/+77
| |
| * node_server: add race condition demoanon2021-09-201-0/+260
| |
* | Merge pull request #7923luigi11112021-09-231-2/+4
|\ \ | | | | | | | | | 5eaedb5 unit_tests: fix use after free in serialization test (moneromooo-monero)