openssl: avoid config file load/race #6438

pull theuni wants to merge 1 commits into bitcoin:master from theuni:openssl-config changing 1 files +8 −0

theuni commented at 10:09 PM on July 14, 2015: member

Should fix #6436. See here for more info. Also the source is self-explanatory.

Since we don't use any loadable functionality, I don't see how this could hurt. Presumably qt tries to load available modules, but the fallbacks should always work.

One last data point: The depends build is trying to load /home/ubuntu/build/bitcoin/depends/x86_64-unknown-linux-gnu/etc/openssl/openssl.cnf which obviously won't be found in the end-user's environment. So this functionality has actually been disabled for a long time anyway.
openssl: avoid config file load/race d4b1d5a8ba
laanwj commented at 7:05 AM on July 15, 2015: member

utACK, also needs backport to 0.11.
laanwj added the label Bug on Jul 15, 2015
jonasschnelli commented at 8:23 AM on July 15, 2015: contributor

Fixes #6436
paveljanik commented at 8:24 AM on July 15, 2015: contributor

ACK
jonasschnelli commented at 8:24 AM on July 15, 2015: contributor

utACK
laanwj merged this on Jul 15, 2015
laanwj closed this on Jul 15, 2015
laanwj referenced this in commit ed789ceade on Jul 15, 2015
laanwj referenced this in commit 253143860d on Jul 15, 2015
laanwj commented at 1:52 PM on July 15, 2015: member

Backported to 0.11 as 253143860da9983d127251814f0665a5454e5030
laanwj referenced this in commit 7e66e9c97b on Jul 17, 2015
laanwj commented at 7:33 AM on July 17, 2015: member

Bacported to 0.10 as d4b1d5a8baf18e4c8d62846360c0c1c0c9331d70
reddink referenced this in commit 46ff200759 on May 27, 2020
DrahtBot locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#6436 Core dumped when running with -prune=N option