| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
|
|
1137142 Change C-cast to static_cast in net_peerlist.h (Jeffrey)
175b411 Change C-style-casts to static_cast in time_helper.h (Jeffrey)
b49ed59 Remove the only 4 non-UTF8 characters in codebase (Jeffrey)
1f25aa2 Factor out move_it_backward from misc_language.h (Jeffrey)
7764d69 Move copyable_atomic into connection_context (Jeffrey)
801568d Refactor out to_nonconst_iterator.h (Jeffrey)
87ec36c Refactor out pragma_comp_defs (Jeffrey)
441c860 Merge functionality of misc_os_dependent into time_helper.h (Jeffrey)
40f02f9 Add Include statements (Jeffrey)
12b1b74 Trimming Fat (Jeffrey
690ce56 Boring Old Deletes (Jeffrey)
|
|
Thanks @mj-xmr: https://github.com/monero-project/monero/pull/8211#discussion_r823870855
|
|
At the request of @mj-xmr: https://github.com/monero-project/monero/pull/8211#discussion_r822868321
|
|
Relevant commit in old PR:
1b798a7042070cc8063bd341ebf7025da554b632
|
|
Relevant commit from old PR:
330df2952cb2863a591158b984c0fb7f652887ac
|
|
Relevant commit from old PR:
bd0a5119957d3ef9130a0b82599e1696995ef235
|
|
Relevant commit on old PR:
2499269696192ce30dd125ddee90a80d4326dff9
|
|
Relevant commits on the old cleanup PR:
36933c7f5c7778e2d7fbfea5361c11fb41070467
21e43de0f300ee47b7e597098908601bf591950b
3c678bb1cedfd7b865ac2e7aaf014de4bfb3eb3d
|
|
Actions:
1. Remove unused functions from misc_os_dependent.h
2. Move three remaining functions, get_gmt_time, get_ns_count, and get_tick_count into time_helper.h
3. Remove unused functions from time_helper.h
4. Refactor get_ns_count and get_internet_time_str and get_time_interval_string
5. Remove/add includes as needed
Relevant commits on the old PR:
a9fbe52b02ffab451e90c977459fea4642731cd1
9a59b131c4ed1be8afe238fff3780fe203c65a46
7fa9e2817df9b9ef3f0290f7f86357939829e588
|
|
|
|
Remove unused include statements or unused definitions.
|
|
Here lies dozens of unused files. This commit is ONLY file deletions except
for the removing of a couple of #includes and removing filenames from CmakeLists
where appropriate.
|
|
4db6aed CMake: Add missing headers via monero_find_all_headers macro (mj-xmr)
|
|
b030f20 Fee changes from ArticMine (moneromooo-monero)
9f786f0 epee: allow copying a rolling_median_t object (moneromooo-monero)
|
|
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
|
|
|
|
6abe3fe tests: integrated_address: remove missing payment id assertion (reemuru)
|
|
The integrated address functional test fails in the workflows due
to an assertion for missing payment id that is no longer needed.
Remove the assertion and update the assertion count.
Fixes 7dcfccb: ("wallet_rpc_server: fix make_integrated_address with no payment id")
|
|
|
|
1ce9e9c Remove dead code from parserse_base_utils and fix unit tests (Jeffrey)
|
|
17772ef Eliminate dependence on boost::interprocess #8223 (Jeffrey)
|
|
d1f1947 wallet_rpc_server: support regex for get_accounts tag (reemuru)
|
|
7dcfccb wallet_rpc_server: fix make_integrated_address with no payment id (moneromooo-monero)
|
|
591c174 Remove footgun doc comment in miner TX validation (Luke Parker)
|
|
0d6fce8 Compil. time: cryptonote_core minus portable_storage header (mj-xmr)
|
|
da9aa1f Copyright: Update to 2022 (mj-xmr)
|
|
3c82958 wallet2: update stagenet rollback blocks (selsta)
|
|
3f88fdd simplewallet: add option for exporting tx keys (reemuru)
|
|
ae0a840 wallet2: decrease the amount of data exchanged for output export (moneromooo-monero)
|
|
cf44349 account info text alignment (im)
|
|
83bb027 Make RPC server functions that read db thread safe (j-berman)
|
|
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)
|
|
354e1d9 Make the wallet name optional when locked. (Norman Moeschter)
|
|
- grab an lmdb db_rtxn_guard to ensure consistent data from the db
- fixed on_getblockhash error resp when requested height >= blockchain height
- left functions that read shared memory untouched for now
|
|
|
|
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
|
|
|
|
|
|
|
|
https://suyash67.github.io/homepage/assets/pdfs/bulletproofs_plus_audit_report_v1.1.pdf
|
|
|
|
|
|
* 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
|
|
In this repo, `boost::interprocess` was being used soley to make `uint32_t` operations atomic. So I replaced each instance of
`boost::interprocess::ipcdetail::atomic(...)32` with `std::atomic` methods. I replaced member declarations as applicable. For example,
when I needed to change a `volatile uint32_t` into a `std::atomic<uint32_t>`. Sometimes, a member was being used a boolean flag, so
I replaced it with `std::atomic<bool>`.
You may notice that I didn't touch `levin_client_async.h`. That is because this file is entirely unused and will be deleted in PR monero-project#8211.
Additional changes from review:
* Make some local variables const
* Change postfix operators to prefix operators where value was not need
|
|
|
|
f22051d replace erciccione's seednode with one on haveno's infrastructure (erciccione)
|
|
9bde2db Fixes: Don't require --rpc-login with --rpc-access-control-origins #8168 (Jeffrey)
|
|
f68f927 Undefined behaviour fixes (SChernykh)
|
|
14eb42d systemd: remove outdated torsocks example (selsta)
|
|
5e201fe rpc: add explicit restricted flag to /get_info (tobtoht)
|
|
e5000a9 Balance includes unconfirmed transfers to self (woodser)
|
|
de4413b workflows: don't build PRs if only docs and READMEs are being changed (erciccione)
|
|
This commit adds a 'regexp' boolean field to the get_accounts
request. The flag is set to false by default and maintains backwards
compatibility. When set to true the user can search tags by regular
expression filters. An additional error message was added for failed
regular expression searches. Bump minor version to 25.
|
|
|
|
|
|
This comment suggests this check is unnecessary, when it is completely necessary as miner TXs can have multiple outputs *which is a statement directly, and incorrectly, contradicted by this comment*. While I don't ever see someone removing this code and getting their edits merged into Monero, someone inexperienced who thinks they're cleaning old code may break their own work, and then there's really just zero benefit to keeping this around.
|
|
|
|
|
|
|
|
b8beac3 Fix typos in docs (Robert Theis)
|
|
fb5b2b3 support authentication in monero-wallet-rpc set_daemon (woodser)
|
|
c36ea26 fix ge_p3_is_point_at_infinity(), which is evaluating field elements that haven't been reduced by the field order (koe)
|
|
e08abaa multisig key exchange update and refactor (koe)
|
|
06e6c8b crypto: fix constant randomx exceptions in large page allocator (moneromooo-monero)
|
|
|
|
This commit adds an option to export transaction
keys when performing export_transfers all. By passing
option=with_keys a new column 'tx key' will be
populated with respective keys.
|
|
|
|
This PR removes the requirement for --rpc-login to be specified if --rpc-access-control-origins is.
This will allow public nodes to serve cross-origin requests. You can still use --rpc-login with
--rpc-access-control-origins, but it is no longer mandatory.
Original Issue: #8168
|
|
If allocating large pages fails, we don't try again.
This has the obvious drawback of not being able to use large pages
if they fail once.
|
|
Fixes issues reported in #8120
|
|
|
|
|
|
|
|
|
|
haven't been reduced by the field order
|
|
2d0a518 Update Copyright year in README.md (IceCruelStuff)
|
|
1051276 Update net_node.inl (Gingeropolous)
|
|
e73e1e7 Use CMD in Dockerfile (Kevin Thomas)
|
|
4446cc8 wallet inits cache if file and blob missing (woodser)
|
|
505b382 cmake: enable build on riscv{64, 32} (Xeonacid)
|
|
6366187 Fixed warnings for Clang (used as CMAKE_CXX compiler) (SerHack)
|
|
fef0280 Fix docker instructions to account for non-root user (Kevin Thomas)
|
|
6a46f8c Add copy commands for compiled binaries (Seth For Privacy)
52612e7 Add archive details to post-build section (Seth For Privacy)
03a256e Add git clone and cd commands for initial prep (Seth For Privacy)
cd76011 Add note on adding SSH keys for Github (Seth For Privacy)
9d02531 Update DOCKRUN.md (Seth For Privacy)
eeb0be4 Bump versions in DOCKRUN.md and add log commands (Seth For Privacy)
|
|
a02b342 Avoid unnecessary 'Invalid hashing blob' error message (rbrunner7)
|
|
361f621 Fix typos (Dimitris Apostolou)
|
|
|
|
|
|
|
|
According to Dockerfile best practices
(https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#entrypoint)
the best use for ENTRYPOINT is to set the image’s main command and then
use CMD as the default flags. This makes it easier to change the flags
passed to monerod when running the Docker image.
|
|
|
|
tested on Arch Linux RISC-V.
|
|
|
|
e9156d7 functional_tests: update uri test with external pid (selsta)
|
|
|
|
Since the non-root user 'monero' is specified in the Dockerfile, the
bind mount when running the container should be set to this user's home
directory be default, not the root user's home directory.
|
|
|
|
Also moved the tail of in-progress logs to two distinct and precise tail commands, was a good recommendation by @hyc.
|
|
|
|
|
|
|
|
|
|
6e6ad80 Fix Android OpenSSL build (Howard Chu)
|
|
53be72c make_uri disallows standalone payment ids (woodser)
|
|
7bc261b cmake: find readline on macOS ARM (selsta)
|
|
1efe624 device_ledger: fix incorrect macro (selsta)
|
|
a491175 cmake: fix incorrect else syntax (selsta)
|
|
94dc19a cmake: remove including non-existent directory (selsta)
|
|
3a90be4 rpc: don't set error code as status string (selsta)
|
|
|
|
depends didn't have target-specific overrides for _build_env
so ANDROID_NDK_HOME wasn't being set at build time.
This fix supersedes the manual edit of Configuration/15-android.conf.
Resolves #8095
|
|
4571d36 Add simple script for running gitian-build (Howard Chu)
|
|
e8e507f rpc: fix DoS vector in get_output_distribution (moneromooo-monero)
|
|
22b891c Update RandomX (Jethro Grassie)
|
|
a4ebec9 Add python method (Howard Chu)
0221b01 Add calcpow RPC (Howard Chu)
|
|
82828bf cryptonote_core: fix unused lambda warning (selsta)
|
|
a30928f depends: bump hidapi to 0.11.0 (selsta)
|
|
|
|
Calculate PoW hash for a block candidate
|
|
|
|
|
|
|
|
The current code does work by accident, but it might break if
someone uses ASSERT_SW in a different place, or if variables
get renamed.
|
|
e0a5dce Update RandomX to v1.1.10 (tevador)
|
|
fbe8358 device: fix ledger error strings, add human friendly messages (selsta)
|
|
187633c slow_hash: don't blow out Mac stack on ARM64 (Howard Chu)
|
|
bbe3b27 tx_pool: full tx revalidation on fork boundaries (moneromooo-monero)
|
|
|
|
|
|
|
|
|
|
|
|
8f21001 depends: openssl 1.1.1k -> 1.1.1l (selsta)
|
|
|
|
|
|
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.
|
|
548cfc8 crypto: fix typo in ifdef (selsta)
|
|
47e7a43 wallet_api: enable set_strict_default_file_permissions (tobtoht)
|
|
a15466d wallet2: remove 2 unused variables (selsta)
|
|
b0ec9f8 epee: avoid ADL selecting C++14 std::quoted (selsta)
|
|
2cad563 wallet2: fix key encryption when changing ask-password from 0/1 to 2 (moneromooo-monero)
|
|
|
|
|
|
|
|
e106bdc epee: add missing header (selsta)
|
|
1f81f5f protocol: fix spurious rejection of downloaded blocks (moneromooo-monero)
|
|
9c61110 cmake: use CMAKE_CURRENT_LIST_DIR for git version (selsta)
|
|
69c09ad Add README to release archives (Howard Chu)
4881154 README v12 fork occurred on 2019-11-30 as expected (Howard Chu)
|
|
ace4669 simplewallet: fix incoming_transfers index error (selsta)
|
|
75d05b9 download: fix leak (moneromooo-monero)
|
|
49441d4 depends: update zeromq package to 4.3.4 (selsta)
|
|
797ca97 socks: ignore any exceptions in shutdown/close (selsta)
|
|
This will prevent people spending old pre-rct outputs using a
stranger's node, which may be a good thing
|
|
|
|
|
|
|
|
|
|
when kicking a peer for inactivity, clear the set of requested blocks,
or next time we requests blocks from it, we'll probably reject the
incoming blocks due to missing the previous requested blocks
|
|
|
|
|
|
we reuse the wallet_keys_unlocker object, which does the right thing
in conjunction with other users of decrypt/encrypt (ie, refresh).
|
|
|
|
9973edd ARMv8: detect AES support dynamically (Howard Chu)
|
|
84c5257 Decrease the 'recent spend window' in gamma re-select to 15 blocks (j-berman)
|
|
b00e21d cmake: set cpp14 standard (selsta)
|
|
5e63f6c Ledger: member 'mode' was shadowing that of base class (mj-xmr)
|
|
ad9fde4 UB: virtual methods in destructor at Trezor (mj-xmr)
|
|
eed7e79 UB: Not calling virtual method in destructor of WalletImpl (mj-xmr)
|
|
514f8fa LMDB: protection against UB, by not calling virtual methods from destructor (mj-xmr)
|
|
7caab8d utils/health: separate Clang Tidy tests for C & C++ (mj-xmr)
|
|
852f319 Doxygen: Exclude build directories (mj-xmr)
|
|
76542ea Fix precision of average_output_time (j-berman)
|
|
Also add docs/ANONYMITY_NETWORKS.md
Even though there is no "docs" dir in the archive...
|
|
Not just an estimated date.
Also fix whitespace alignment.
Note for future: always use backslash-asterisk if adding this
notation, otherwise it screws up rendering on some MD formatters
|
|
9cd7aa6 cmake: don't optimize debug build (selsta)
|
|
48a0997 blockchain: fix pedantic assert (selsta)
|
|
bc167a1 cmake: fix copy/paste typo (selsta)
|
|
2e8936f rpc: Fix get_transactions failing when not found (Nathan Dorfman)
|
|
5f54dea LMDB: fix deadlock in resized detection (Howard Chu)
|
|
313edb2 Doc: building gtest on Debian and Ubuntu (ralphmapper)
|
|
ae41178 gitian: cleanup more unused packages (Howard Chu)
4ea7832 gitian: omit ccache from gitian builds (Howard Chu)
|
|
9154883 node_server: fix race condition (anon)
8922f96 node_server: add race condition demo (anon)
|
|
13a8a57 trezor: try empty passphrase first (Dusan Klinec)
|
|
|
|
|
|
A shared_ptr as by value capture will keep the object alive
|
|
|
|
|
|
|
|
1346516 workflows: add missing dependency, fix mac build (selsta)
|
|
- combined with patching integer truncation (#7798), this gets the algorithm marginally closer to mirroring empirically observed output ages
- 50 was originally chosen assuming integer truncation would remain in the client for that client release version. But patching integer truncation causes the client to select more outputs in the 10-100 block range, and therefore the benefit of choosing a larger recent spend window of 50 has less merit
- 15 seems well-suited to cover the somewhat sizable observable gap in the early window of blocks
|
|
|
|
|
|
These appear to be leftovers inherited from Bitcoin's builder
|
|
It's just wasted overhead since the build VMs are always deleted
and recreated fresh for each run.
|
|
|
|
It's 2021, we can easily switch to C++14 without any troubles.
|
|
4afd9a7 cmake: set required C/C++ standard to 11 (selsta)
|
|
5617a29 unbound: remove leftovers (selsta)
|
|
a95e6e0 Dockerfile: use depends (selsta)
|
