35574 Commits

Author SHA1 Message Date
fanquake
d3a3522560
Merge bitcoin/bitcoin#28668: [24.2] Final Changes
b01a5bea7dd278ee2e6a7b46119d09860c5580af doc: update release notes for 24.2 (fanquake)
1343f43518d8f57547a5ecce8d6986db182caedd doc: update manual pages for 24.2 (fanquake)
db244c63429b4a8c4c60f66f4b76c0b0dee73365 build: bump version to 24.2 final (fanquake)

Pull request description:

  Final changes for v24.2.
  PR for bitcoincore.org is here: https://github.com/bitcoin-core/bitcoincore.org/pull/993.

  24.2rc1 tag: https://github.com/bitcoin/bitcoin/releases/tag/v24.2rc1
  rc1 bins available for ~1 week: https://bitcoincore.org/bin/bitcoin-core-24.2/test.rc1/.

ACKs for top commit:
  josibake:
    ACK b01a5bea7d
  stickies-v:
    ACK b01a5bea7dd278ee2e6a7b46119d09860c5580af

Tree-SHA512: 339466ace2d3401a690e1bbd7f7636a9905f379870a06a38151a745d0df4f1c9a532acecd38cade608e417343c6ac20563e42eae5fbd42128cb81802e3212340
v24.2
2023-10-23 10:26:40 +01:00
fanquake
b01a5bea7d
doc: update release notes for 24.2 2023-10-17 16:49:43 +01:00
fanquake
1343f43518
doc: update manual pages for 24.2 2023-10-17 16:48:54 +01:00
fanquake
db244c6342
build: bump version to 24.2 final 2023-10-17 16:46:36 +01:00
fanquake
0dfbe96583
Merge bitcoin/bitcoin#28582: [24.2] Final changes for 24.2rc1
d8340dd03bfaee1a888478803ce4f0cfeb0cc98d doc: add release notes for 24.2rc1 (fanquake)
9795e0ba5e92cc4bd53538545aba9b8690041798 doc: update manual pages for 24.2rc1 (fanquake)
248d14cb63ee283becbbbbbdd245b1806dc09bc8 build: bump version to 24.2.rc1 (fanquake)
e78961db591272a26a8f4d1c844b31c8710bd204 doc: add 24.1 release notes (fanquake)

Pull request description:

  Final changes to tag a `24.2rc1`.
  Bumps version numbers, man pages, adds release notes etc.

ACKs for top commit:
  achow101:
    ACK d8340dd03bfaee1a888478803ce4f0cfeb0cc98d
  stickies-v:
    ACK d8340dd03bfaee1a888478803ce4f0cfeb0cc98d

Tree-SHA512: 5cb68e605abcdf9999852605e7c9002a0bfd72bdd078e707eee7beec4f56184505cee8480f7c9e8e27d56815a6bbe3a811396fbb8cac530f065bc04ee1ccbd8a
v24.2rc1
2023-10-11 20:16:41 +02:00
fanquake
d8340dd03b
doc: add release notes for 24.2rc1 2023-10-10 17:35:30 +01:00
fanquake
9795e0ba5e
doc: update manual pages for 24.2rc1 2023-10-06 11:37:36 +01:00
fanquake
248d14cb63
build: bump version to 24.2.rc1 2023-10-06 11:37:36 +01:00
fanquake
e78961db59
doc: add 24.1 release notes 2023-10-06 11:37:36 +01:00
fanquake
1416d09cba
Merge bitcoin/bitcoin#28535: [24.x] Further backports
9077f214f55386af12419235deaff52b23446856 depends: fix unusable memory_resource in macos qt build (fanquake)
dccacf0bf7d5815036c64d4c040b5702ad890cd8 build, macos: Fix `qt` package build with new Xcode 15 linker (Hennadii Stepanov)
43596499b2c2b8cc0a51c0b9db9153047c266627 ci: Switch to `amd64` container in "ARM" task (Hennadii Stepanov)
805f98b79aa9e5ecda70516578296bd0a065a707 ci: Nuke Android APK task, Use credits for tsan (MarcoFalke)
cb5512da2336c9145a670c287f1abecc372906b9 test: ensure old fee_estimate.dat not read on restart and flushed (ismaelsadeeq)
01f8ee48efc1f46563f2841c0a7125b20b0df159 tx fees, policy: read stale fee estimates with a regtest-only option (ismaelsadeeq)
1c98029b3913a99e7bfe563d8ded2a5074e75fa1 tx fees, policy: do not read estimates of old fee_estimates.dat (ismaelsadeeq)
77979e0172d0bc86bfbc60f6a652e26c77722e29 tx fees, policy: periodically flush fee estimates to fee_estimates.dat (ismaelsadeeq)
67b6d99aead0d1b2030bc3e88256d279477894b5 Do not use std::vector = {} to release memory (Pieter Wuille)
defdc1502372863f700720e8d5cde69190371a64 ci: Use podman stop over podman kill (MarcoFalke)
7f1357de5136bd6f80758f1f31e6dba21acb9954 ci: Use podman for persistent workers (MarcoFalke)
0db69a3d500020e11fd67c55732e0d02eb606204 ci: Prune dangling images on RESTART_CI_DOCKER_BEFORE_RUN (MarcoFalke)

Pull request description:

  Backports to the 24.x branch. Currently:
  * https://github.com/bitcoin/bitcoin/pull/27622
  * https://github.com/bitcoin/bitcoin/pull/27777
  * https://github.com/bitcoin/bitcoin/pull/27834
  * https://github.com/bitcoin/bitcoin/pull/27844
  * https://github.com/bitcoin/bitcoin/pull/27886
  * https://github.com/bitcoin/bitcoin/pull/28452
  * https://github.com/bitcoin/bitcoin/pull/28543
  * https://github.com/bitcoin/bitcoin/pull/28571

ACKs for top commit:
  stickies-v:
    ACK 9077f214f5

Tree-SHA512: abaafc9a048b67b494993134fd332457ea52695ec007b963c283f962ec40c3b6b3a7e98407481be55d3271a595088a0281cc84b79dad4f24d260381ea0153076
2023-10-06 11:36:51 +01:00
fanquake
9077f214f5
depends: fix unusable memory_resource in macos qt build
See https://codereview.qt-project.org/c/qt/qtbase/+/482392.

Github-Pull: #28571
Rebased-From: 848eec09363d1ba8198376eb9654b1a69e3541aa
2023-10-03 15:57:10 +01:00
Hennadii Stepanov
dccacf0bf7
build, macos: Fix qt package build with new Xcode 15 linker
Github-Pull: #28543
Rebased-From: 79ef528511f0cbbe0a7097ef031f2964aaccfe5c
2023-10-03 10:34:53 +01:00
Hennadii Stepanov
43596499b2
ci: Switch to amd64 container in "ARM" task
Tee `arm_container` does not support 32-bit mode anymore.
See: https://github.com/bitcoin/bitcoin/issues/27879

Github-Pull: #27886
Rebased-From: 016fe6d8280768917081894dfca233c2f06e78d9
2023-10-02 16:49:14 +01:00
MarcoFalke
805f98b79a
ci: Nuke Android APK task, Use credits for tsan
Github-Pull: #27834
Rebased-From: fa22538e481fa2c4f0b5d6f91166335e60b67fe9
2023-10-02 13:05:51 +01:00
ismaelsadeeq
cb5512da23
test: ensure old fee_estimate.dat not read on restart and flushed
This commit adds tests to ensure that old fee_estimates.dat files
are not read and that fee_estimates are periodically flushed to the
fee_estimates.dat file.

Additionaly it tests the -regtestonly option -acceptstalefeeestimates.

Github-Pull: #27622
Rebased-From: d2b39e09bc6a5982fc5cf4b538b7fdb0e3cae576
2023-09-26 16:26:55 +01:00
ismaelsadeeq
01f8ee48ef
tx fees, policy: read stale fee estimates with a regtest-only option
If -acceptstalefeeestimates option is passed stale fee estimates can now
be read when operating in regtest environments.

Additionally, this commit updates all declarations of the CBlockPolicyEstimator
class to include a the second constructor variable.

Github-Pull: #27622
Rebased-From: cf219f29f3c5b41070eaab9a549a476f01990f3a
2023-09-26 16:26:55 +01:00
ismaelsadeeq
1c98029b39
tx fees, policy: do not read estimates of old fee_estimates.dat
Old fee estimates could cause transactions to become stuck in the
mempool. This commit prevents the node from using stale estimates
from an old file.

Github-Pull: #27622
Rebased-From: 3eb241a141defa564c94cb95c5bbaf4c5bd9682e
2023-09-26 16:26:55 +01:00
ismaelsadeeq
77979e0172
tx fees, policy: periodically flush fee estimates to fee_estimates.dat
This reduces chances of having old estimates in fee_estimates.dat.

Github-Pull: #27622
Rebased-From: 5b886f2b436eaa8c2b7de58dc4644dc6223040da
2023-09-26 16:26:55 +01:00
Pieter Wuille
67b6d99aea
Do not use std::vector = {} to release memory
Github-Pull: #28452
Rebased-From: 3fcd7fc7ff563bdc0e2bba66b4cbe72d898c876e
2023-09-26 16:26:55 +01:00
MarcoFalke
defdc15023
ci: Use podman stop over podman kill
This should avoid a race where the kill is not done when spinning up the
new container. podman stop waits 10 seconds by default.

Github-Pull: #27844
Rebased-From: faaa62754e84417baa917f20db379db78146687d
2023-09-26 16:26:55 +01:00
MarcoFalke
7f1357de51
ci: Use podman for persistent workers
Github-Pull: #27777
Rebased-From: fa123077bc3f39aa0969d883e2d799a054cd4543
2023-09-26 16:25:05 +01:00
MarcoFalke
0db69a3d50
ci: Prune dangling images on RESTART_CI_DOCKER_BEFORE_RUN
Github-Pull: #27777
Rebased-From: fa9c65a74cf18e9c75cd3472112d5197532ac2f2
2023-09-26 16:24:41 +01:00
fanquake
0df8f98d65
Merge bitcoin/bitcoin#28410: [24.x] depends: xcb-proto 1.15.2
66b2b109ebed522817716f73f9490d62d3d9af04 depends: xcb-proto 1.15.2 (fanquake)

Pull request description:

  Resolves build failures under Python 3.12, i.e building on rawhide:
  ```bash
  make[3]: Nothing to be done for 'install-exec-am'.
   /usr/bin/mkdir -p '/bitcoin/depends/work/staging/aarch64-unknown-linux-gnu/xcb_proto/1.14.1-4a91ac9dc41/bitcoin/depends/aarch64-unknown-linux-gnu/lib/python3.12/site-packages/xcbgen'
   /usr/bin/install -c -m 644 __init__.py error.py expr.py align.py matcher.py state.py xtypes.py '/bitcoin/depends/work/staging/aarch64-unknown-linux-gnu/xcb_proto/1.14.1-4a91ac9dc41/bitcoin/depends/aarch64-unknown-linux-gnu/lib/python3.12/site-packages/xcbgen'
  Traceback (most recent call last):
    File "<string>", line 2, in <module>
  ModuleNotFoundError: No module named 'imp'
  make[3]: *** [Makefile:271: install-pkgpythonPYTHON] Error 1
  ```

  `imp` was removed in 3.12: https://docs.python.org/3/library/imp.html.

  Github-Pull: #28097
  Rebased-From: 7cb88c8b46723d306b96953a6a60c90a4ab211e3

  Also being backported for 25.x in #28047.

ACKs for top commit:
  hebasto:
    ACK 66b2b109ebed522817716f73f9490d62d3d9af04

Tree-SHA512: 5364c5ced3004bc08a9c8f96e2b3abae3e31b3f0e7b1d10f83ff997f19b314d4d14c2a780ed02e793771c23091f8f02137cf7f5404627f2bd218c8726ae3228f
2023-09-05 15:39:58 +01:00
fanquake
66b2b109eb
depends: xcb-proto 1.15.2
Resolves build failures under Python 3.12, i.e building on rawhide:
```bash
make[3]: Nothing to be done for 'install-exec-am'.
 /usr/bin/mkdir -p '/bitcoin/depends/work/staging/aarch64-unknown-linux-gnu/xcb_proto/1.14.1-4a91ac9dc41/bitcoin/depends/aarch64-unknown-linux-gnu/lib/python3.12/site-packages/xcbgen'
 /usr/bin/install -c -m 644 __init__.py error.py expr.py align.py matcher.py state.py xtypes.py '/bitcoin/depends/work/staging/aarch64-unknown-linux-gnu/xcb_proto/1.14.1-4a91ac9dc41/bitcoin/depends/aarch64-unknown-linux-gnu/lib/python3.12/site-packages/xcbgen'
Traceback (most recent call last):
  File "<string>", line 2, in <module>
ModuleNotFoundError: No module named 'imp'
make[3]: *** [Makefile:271: install-pkgpythonPYTHON] Error 1
```

`imp` was removed in 3.12: https://docs.python.org/3/library/imp.html.

Github-Pull: #28097
Rebased-From: 7cb88c8b46723d306b96953a6a60c90a4ab211e3
2023-09-05 12:42:16 +01:00
Andrew Chow
a537cb0b87
Merge bitcoin/bitcoin#27755: [24.x] rpc: Fix invalid bech32 handling
c2e9214effe9abecae6f81cb10158f9661065da3 rpc: Fix invalid bech32 handling (MarcoFalke)

Pull request description:

  Backports https://github.com/bitcoin/bitcoin/pull/27727 to 24.x. Not a clean cherry-pick.

ACKs for top commit:
  achow101:
    ACK c2e9214effe9abecae6f81cb10158f9661065da3
  stickies-v:
    ACK c2e9214effe9abecae6f81cb10158f9661065da3

Tree-SHA512: 8fd5b4c0016e9f30061e327b5f75320e4ed27b1a7b61eafabefe87d351ab13c970e969ea85b2f4cab44c1b0f28214ee7c5b85e3e0549345b57f1fa724e3c814c
2023-06-01 17:43:07 -04:00
MarcoFalke
c2e9214eff
rpc: Fix invalid bech32 handling
Github-Pull: #27727
Rebased-From: eeee55f9288740747b6e8d806ce8177fd92635cf
2023-05-25 15:13:32 +01:00
fanquake
3116ccd790
Merge bitcoin/bitcoin#27660: [24.1] Final Changes
89a5a416deac060fed8c21d4381c8f59da4f4187 doc: update release notes for 24.1 (fanquake)
b36dff9fa8b14f6894d901e4fd8d413cbcb73f08 doc: update manual pages for 24.1 (fanquake)
8af2d2f7628c15af20f566aa29abc228ed32f845 build: bump version to 24.1 final (fanquake)

Pull request description:

  Final changes for `v24.1`.
  PR for bitcoincore.org is here: https://github.com/bitcoin-core/bitcoincore.org/pull/968.

ACKs for top commit:
  josibake:
    ACK 89a5a416de
  hebasto:
    ACK 89a5a416deac060fed8c21d4381c8f59da4f4187, I have reviewed the code and it looks OK.
  stickies-v:
    ACK 89a5a416deac060fed8c21d4381c8f59da4f4187

Tree-SHA512: 6eb7d30790b50cfc7232cf7c69b5c8b1fbb943cc715d275ff4d8c2ade93d82bf2bc659b82a06b8453e2e0382e9d375aee2c71d0fee44285f9eb8a029980a29d3
v24.1
2023-05-16 10:27:30 +01:00
fanquake
89a5a416de
doc: update release notes for 24.1 2023-05-15 10:23:08 +01:00
fanquake
b36dff9fa8
doc: update manual pages for 24.1 2023-05-15 10:21:28 +01:00
fanquake
8af2d2f762
build: bump version to 24.1 final 2023-05-15 10:17:53 +01:00
fanquake
16ab124e5f
Merge bitcoin/bitcoin#27627: [24.x] qt: 24.1rc3 translations update
a86b45cafaabd8eb4957b84a0d70484a597eacc7 qt: 24.1rc3 translations update (Hennadii Stepanov)

Pull request description:

  This PR pulls the recent translations from the [Transifex.com](https://www.transifex.com/bitcoin/bitcoin) using the [`bitcoin-maintainer-tools/update-translations.py`](https://github.com/bitcoin-core/bitcoin-maintainer-tools/blob/main/update-translations.py) tool.

  According to our [Release Process docs](https://github.com/bitcoin/bitcoin/blob/master/doc/release-process.md#before-every-release-candidate), it is supposed to be merged before `v24.1rc3` tagging.

  Will keep this PR updated regularly until merging.

  The Dutch translation `bitcoin_nl.ts` has been damaged, therefore its changes were rejected manually.

ACKs for top commit:
  jarolrod:
    ACK a86b45cafaabd8eb4957b84a0d70484a597eacc7

Tree-SHA512: 64bdd4b1d2ec6d925d9972081492d33ebbab6a19cde1c2e5d1022c89f58390618be843de5bec3344b92dda54192352c86ab1e354001f638f32d961a09500576f
v24.1rc3
2023-05-11 16:19:58 +01:00
Andrew Chow
ac5b9f37de
Merge bitcoin/bitcoin#27614: [24.1] Backports for rc3
97f5e28830dfa5786cdc5df62e245e03fa393e19 doc: update release notes for 24.1rc3 (fanquake)
7e9c7ae810994bddd83b53ff5804b02345e4bc2f doc: update manual pages for v24.1rc3 (fanquake)
abb9fa0c81107b4db792ee10b93aed618a42684e build: bump version to v24.1rc3 (fanquake)
128da6e41fc2acc5bd1f4e2bddf2953f2fa62a68 net_processing: Boost inv trickle rate (Anthony Towns)
a9a861af2b4b1c0a17b8abda34cb741170e12d7d txmempool: have CompareDepthAndScore sort missing txs first (Anthony Towns)
ec7cd33c9c70af5903bf5dbf1056d42868de6f43 p2p: Avoid prematurely clearing download state for other peers (Suhas Daftuar)

Pull request description:

  Backports for rc3. Currently:
  * https://github.com/bitcoin/bitcoin/pull/27608
  * https://github.com/bitcoin/bitcoin/pull/27610

ACKs for top commit:
  josibake:
    ACK 97f5e28830
  dergoegge:
    ACK 97f5e28830dfa5786cdc5df62e245e03fa393e19
  achow101:
    ACK 97f5e28830dfa5786cdc5df62e245e03fa393e19
  glozow:
    ACK 97f5e28830dfa5786cdc5df62e245e03fa393e19
  brunoerg:
    ACK 97f5e28830dfa5786cdc5df62e245e03fa393e19
  hebasto:
    ACK 97f5e28830dfa5786cdc5df62e245e03fa393e19, commits were backported locally, got zero diff.

Tree-SHA512: 09572285ed1e8169d7e77d12ec438586dab54c86064de85d0e743564e601686f884bf74f2bf8ed1be73bddcd7db6da4277c6dd6b9732e7eca383e108f8f37d58
2023-05-11 10:57:23 -04:00
Hennadii Stepanov
a86b45cafa
qt: 24.1rc3 translations update 2023-05-11 15:38:27 +01:00
fanquake
97f5e28830
doc: update release notes for 24.1rc3 2023-05-11 14:24:20 +01:00
fanquake
7e9c7ae810
doc: update manual pages for v24.1rc3 2023-05-11 14:24:20 +01:00
fanquake
abb9fa0c81
build: bump version to v24.1rc3 2023-05-11 14:24:19 +01:00
Anthony Towns
128da6e41f
net_processing: Boost inv trickle rate
If transactions are being added to the mempool at a rate faster than 7tx/s
(INVENTORY_BROADCAST_PER_SECOND) then peers' inventory_to_send queue can
become relatively large. If this happens, increase the number of txids
we include in an INV message (normally capped at 35) by 5 for each 1000
txids in the queue.

This will tend to clear a temporary excess out reasonably quickly; an
excess of 4000 invs to send will be cleared down to 1000 in about 30
minutes, while an excess of 20000 invs would be cleared down to 1000 in
about 60 minutes.

Github-Pull: #27610
Rebased-From: 5b3406094f2679dfb3763de4414257268565b943
2023-05-11 14:24:19 +01:00
Anthony Towns
a9a861af2b
txmempool: have CompareDepthAndScore sort missing txs first
We use CompareDepthAndScore to choose an order of txs to inv. Rather
than sorting txs that have been evicted from the mempool at the end
of the list, sort them at the beginning so they are removed from
the queue immediately.

Github-Pull: #27610
Rebased-From: 228e9201efb5574b1b96bb924de1d2e8dd1317f3
2023-05-11 14:24:19 +01:00
Suhas Daftuar
ec7cd33c9c
p2p: Avoid prematurely clearing download state for other peers
Github-Pull: #27608
Rebased-From: 52e52071e01f4e98d87a47e1d5f3c5c3cc6dbaf4
2023-05-11 14:23:45 +01:00
fanquake
36c15d7d5c
Merge bitcoin/bitcoin#27486: [24.1] Bump version to 24.1rc2
03a16a1da09f45721c799c719e48350dfe12ae88 doc: update release notes for 24.1rc2 (fanquake)
f344c66053f1f1c24c0dac3ecc9570bea5c7fa0a doc: update manual pages for v24.1rc2 (fanquake)
4451e899881f2ae6b46c13ff87618d45a6fa982f build: bump version to v24.1rc2 (fanquake)

Pull request description:

  rc1 (tagged > a month ago) was mostly a no-op, due to lack of binaries. While some are up now, [albeit at the wrong URL](https://bitcoincore.org/bin/bitcoin-24.1rc1/), there seems no point continuing the rc1 cycle, when we already have additional changes backported (#27474), and we've also received no bug reports/feedback from anyone who did test.

  So bump the version, and cut an rc2.

ACKs for top commit:
  achow101:
    ACK 03a16a1da09f45721c799c719e48350dfe12ae88
  gruve-p:
    ACK 03a16a1da0
  hebasto:
    ACK 03a16a1da09f45721c799c719e48350dfe12ae88, I have reviewed the code and it looks OK.

Tree-SHA512: f49072149ecabb02b034b1c4d7319a80f93a8c7a29a2b7ec27dff0c257742d08d48fbf266399ce71769cec992902b7d53fc26bb5ffc8681728dc8685cbba25d9
v24.1rc2
2023-04-19 09:19:49 +01:00
fanquake
03a16a1da0
doc: update release notes for 24.1rc2 2023-04-18 15:42:30 +01:00
fanquake
f344c66053
doc: update manual pages for v24.1rc2 2023-04-18 15:36:28 +01:00
fanquake
4451e89988
build: bump version to v24.1rc2 2023-04-18 15:31:44 +01:00
fanquake
15a24781d0
Merge bitcoin/bitcoin#27474: [24.x] Additional backports for 24.1
dc711fbd32653b09e196f72942106114a32353f4 doc: update 24.1 release notes (fanquake)
fc8c1a8deb80913ff353c878f494c9eaf28061c0 doc: fix/improve warning helps in {create,load,unload,restore}wallet (Jon Atack)
3a26b19df25ca99a9a58ae5398f6f423ac074368 bugfix: rest: avoid segfault for invalid URI (pablomartin4btc)
c40b1da2fd64bb10f120f85966b44f0d2bb315f8 depends: fix compiling bdb with clang-16 on aarch64 (fanquake)
0bac52d5cfaf1a3beb99b780ed7446e2faba877f Don't return OutputType::UNKNOWN in ParseOutputType (Pttn)

Pull request description:

  Backports:
  * https://github.com/bitcoin/bitcoin/pull/27279 (only f73782a9032a462a71569e9424db9bf9eeababf3)
  * https://github.com/bitcoin/bitcoin/pull/27462
  * https://github.com/bitcoin/bitcoin/pull/27468
  * https://github.com/bitcoin/bitcoin/pull/27473

ACKs for top commit:
  stickies-v:
    ACK dc711fbd32653b09e196f72942106114a32353f4
  hebasto:
    re-ACK dc711fbd32653b09e196f72942106114a32353f4
  jonatack:
    ACK dc711fbd32653b09e196f72942106114a32353f4

Tree-SHA512: 72c673be82689e3c3a1c2564a1fdd6afe0b357b7aa8bec9524fe6999804fbccf310da0b074e647af14b753e5e695024e268fe4f69aa58747f541f7f429ebede6
2023-04-18 15:28:33 +01:00
fanquake
dc711fbd32
doc: update 24.1 release notes 2023-04-18 12:14:05 +01:00
Jon Atack
fc8c1a8deb
doc: fix/improve warning helps in {create,load,unload,restore}wallet
- clarify that there can be multiple warning messages
- specify the correct wallet action
- describe the use of newlines as delimiters

Github-Pull: #27279
Rebased-From: f73782a9032a462a71569e9424db9bf9eeababf3
2023-04-18 12:12:49 +01:00
pablomartin4btc
3a26b19df2
bugfix: rest: avoid segfault for invalid URI
`evhttp_uri_parse` can return a nullptr, for example when the URI
contains invalid characters (e.g. "%").
`GetQueryParameterFromUri` passes the output of `evhttp_uri_parse`
straight into `evhttp_uri_get_query`, which means that anyone calling
a REST endpoint in which query parameters are used (e.g. `rest_headers`)
can cause a segfault.

This bugfix is designed to be minimal and without additional behaviour change.

Github-Pull: #27468
Rebased-From: 11422cc5720c8d73a87600de8fe8abb156db80dc
2023-04-18 11:43:59 +01:00
fanquake
c40b1da2fd
depends: fix compiling bdb with clang-16 on aarch64
Compiling bdb with clang-16 on aarch64 (hardware) currently fails:
```bash
make -C depends/ bdb CC=clang CXX=clang++
...
checking for mutexes... UNIX/fcntl
configure: WARNING: NO SHARED LATCH IMPLEMENTATION FOUND FOR THIS PLATFORM.
configure: error: Unable to find a mutex implementation
```

Looking at config.log we've got:
```bash
configure:18704: checking for mutexes
configure:18815: clang -o conftest -pipe -std=c11 -O2 -Wno-error=implicit-function-declaration -Wno-error=format-security    -I/bitcoin/depends/aarch64-unknown-linux-gnu/include -D_GNU_SOURCE -D_REENTRANT   -L/bitcoin/depends/aarch64-unknown-linux-gnu/lib conftest.c  -lpthread >&5
conftest.c:45:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
main() {
^
int
conftest.c:50:2: warning: call to undeclared library function 'exit' with type 'void (int) __attribute__((noreturn))'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
        exit (
        ^
conftest.c:50:2: note: include the header <stdlib.h> or explicitly provide a declaration for 'exit'
1 warning and 1 error generated.
```

Clang-16 changed `-Wimplicit-function-declaration` and `-Wimplicit-int`
warnings into errors, see:
https://releases.llvm.org/16.0.0/tools/clang/docs/ReleaseNotes.html#potentially-breaking-changes.

> The -Wimplicit-function-declaration and -Wimplicit-int warnings now
> default to an error in C99, C11, and C17. As of C2x, support for implicit
> function declarations and implicit int has been removed, and the
> warning options will have no effect. Specifying -Wimplicit-int in
> C89 mode will now issue warnings instead of being a noop.

Github-Pull: #27462
Rebased-From: f8b8458276983f8fc1e2a47c4d00c1e30633067d
2023-04-18 09:29:55 +01:00
Pttn
0bac52d5cf
Don't return OutputType::UNKNOWN in ParseOutputType
Fixes https://github.com/bitcoin/bitcoin/issues/27472

Signed-off-by: Pttn <28868425+Pttn@users.noreply.github.com>

Github-Pull: #27473
Rebased-From: 0d6383fda04a99726654945a737bbb1369e0e44a
2023-04-17 15:09:27 +01:00
fanquake
9e05de1d70
Merge bitcoin/bitcoin#27247: [24.x] Bump version to v24.1rc1
932a609312cb7b72bf5f23112763ab933a54389c doc: add initial release notes for v24.1 (fanquake)
cc4e3158c9e7840151fe7b6209ba5f7ea5f103c3 doc: update manual pages for v24.1rc1 (fanquake)
787affb9ea04651a682acf8d09342d6e843fdd0f doc: update version in bips.md to v24.1 (fanquake)
5077e02263cf47749583f22dc2bc609878d8290a build: bump version to v24.1rc1 (fanquake)

Pull request description:

  Bump the version number to v24.1rc1.
  Regenerate the man pages.
  Update the version number in bips.md.
  Move the v24.0.1 release notes to doc/release-notes.
  Add initial release notes for v24.1.

  Merged changes to the 24.x branch since v24.0.1:
  - https://github.com/bitcoin/bitcoin/pull/26457
  - https://github.com/bitcoin/bitcoin/pull/26735
  - https://github.com/bitcoin/bitcoin/pull/26878
  - https://github.com/bitcoin/bitcoin/pull/26880

ACKs for top commit:
  achow101:
    ACK 932a609312cb7b72bf5f23112763ab933a54389c

Tree-SHA512: b90fd7c8f22c8fb096864e47cb79eaf5878524739a3b5c1d495c8c196b70d08c7b95fbfb1dfcdddf507bd8a72a5d133ecbe6ae898bbe70931f404afd0807b707
v24.1rc1
2023-03-13 15:47:43 +01:00