enable ASLR and DEP for bitcoind.exe via linker flags #1752

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:bitcoind_ASLR_DEP changing 2 files +6 −4

Diapolo commented at 7:26 AM on August 29, 2012: none
- this is already active for bitcoin-qt.exe
As no one responded in the IRC chan I'll open this as pull. I'm not sure if that pull needs further modifications to the makefiles, so please be gentle :).
laanwj commented at 9:27 AM on August 29, 2012: member

Looks fine. Let's wait for the pulltester to see whether the linker args are really passed.
laanwj commented at 7:20 PM on August 29, 2012: member
This will not work yet: LDFLAGS is not passed when linking. You need to add $(LDFLAGS) to the lines that build bitcoind.exe and test_bitcoin.exe, i.e. in makefile.mingw-linux these are
```
    i586-mingw32msvc-g++ $(CFLAGS) -o $@ $(LIBPATHS) $^ $(LIBS)
```
and
```
    i586-mingw32msvc-g++ $(CFLAGS) -o $@ $(LIBPATHS) $^ -lboost_unit_test_framework-mt-s $(LIBS)
```
enable ASLR and DEP for bitcoind.exe via linker flags
- this is already active for bitcoin-qt.exe
a4d4403a75
Diapolo commented at 8:31 PM on August 29, 2012: none

@laanwj Can you verify tha I did what you suggested ;)?
laanwj commented at 5:34 AM on August 30, 2012: member

Yes, looks OK now.
laanwj referenced this in commit f78fd6f397 on Aug 30, 2012
laanwj merged this on Aug 30, 2012
laanwj closed this on Aug 30, 2012
owlhooter referenced this in commit d1aeac1b2e on Oct 10, 2018
DrahtBot locked this on Sep 8, 2021

Contributors