Skip to content

Commit bf2d5d8

Browse files
ismaildscho
authored andcommitted
Don't let ld strip relocations
This is the first step for enabling ASLR (Address Space Layout Randomization) support. We want to enable ASLR for better protection against exploiting security holes in Git. The problem fixed by this commit is that `ld.exe` seems to be stripping relocations which in turn will break ASLR support. We just make sure it's not stripping the main executable entry. Signed-off-by: İsmail Dönmez <[email protected]> Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 78ce6bc commit bf2d5d8

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

config.mak.uname

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -544,9 +544,11 @@ else
544544
prefix = /usr/
545545
ifeq (MINGW32,$(MSYSTEM))
546546
prefix = /mingw32
547+
BASIC_LDFLAGS += -Wl,--pic-executable,-e,_mainCRTStartup
547548
endif
548549
ifeq (MINGW64,$(MSYSTEM))
549550
prefix = /mingw64
551+
BASIC_LDFLAGS += -Wl,--pic-executable,-e,mainCRTStartup
550552
else
551553
COMPAT_CFLAGS += -D_USE_32BIT_TIME_T
552554
BASIC_LDFLAGS += -Wl,--large-address-aware

0 commit comments

Comments
 (0)