[23.x] qt: Avoid crash on startup if int specified in settings.json #24511
pull achow101 wants to merge 2 commits into bitcoin:23.x from achow101:23.x-settings-json-crash changing 7 files +180 −1-
achow101 commented at 4:04 pm on March 9, 2022: memberBackport of #24498
-
test: Add tests for GetArg methods / settings.json type coercion
Just add tests. No changes to application behavior. Tests will be updated in the next commit changing & improving current behavior. Include a Qt test for GUI startup crash reported by Rspigler in https://github.com/bitcoin/bitcoin/issues/24457 caused by GetArg behavior that happens if settings.json contains an integer value for any of the configuration options which GUI settings can currently clash with (-dbcache, -par, -spendzeroconfchange, -signer, -upnp, -natpmp, -listen, -server, -proxy, -proxy, -onion, -onion, -lang, and -prune). Github-Pull: bitcoin/bitcoin#24498 Rebased-From: 84b0973e35dae63cd1b60199b481e24d54e58c97
-
qt: Avoid crash on startup if int specified in settings.json
Fix GUI startup crash reported by Rspigler in https://github.com/bitcoin/bitcoin/issues/24457 that happens if settings.json contains an integer value for any of the configuration options which GUI settings can currently clash with (-dbcache, -par, -spendzeroconfchange, -signer, -upnp, -natpmp, -listen, -server, -proxy, -proxy, -onion, -onion, -lang, and -prune). Fix is a one-line change in ArgsManager::GetArg. Github-Pull: bitcoin/bitcoin#24498 Rebased-From: 5b1aae12ca4a99c6b09349981a4902717a6a5d3e
-
fanquake added the label Backport on Mar 9, 2022
-
fanquake added this to the milestone 23.0 on Mar 9, 2022
-
jonatack commented at 4:36 pm on March 9, 2022: memberACK 7e1b968f61a1309dfd0fc4f7cd5a2d964bfc4f30
-
ryanofsky approved
-
ryanofsky commented at 6:07 pm on March 9, 2022: memberACK confirmed conflict-free backport with no changes using script (https://github.com/bitcoin/bitcoin/pull/11592#pullrequestreview-73922693)
-
fanquake merged this on Mar 10, 2022
-
fanquake closed this on Mar 10, 2022
-
laanwj referenced this in commit 04fdd644b4 on May 25, 2022
-
DrahtBot locked this on Mar 10, 2023
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-27 12:12 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-27 12:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me