Skip to content

Commit e133fbb

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 5fa12c0 commit e133fbb

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
@@ -576,10 +576,12 @@ else
576576
ifeq (MINGW32,$(MSYSTEM))
577577
prefix = /mingw32
578578
HOST_CPU = i686
579+
BASIC_LDFLAGS += -Wl,--pic-executable,-e,_mainCRTStartup
579580
endif
580581
ifeq (MINGW64,$(MSYSTEM))
581582
prefix = /mingw64
582583
HOST_CPU = x86_64
584+
BASIC_LDFLAGS += -Wl,--pic-executable,-e,mainCRTStartup
583585
else
584586
COMPAT_CFLAGS += -D_USE_32BIT_TIME_T
585587
BASIC_LDFLAGS += -Wl,--large-address-aware

0 commit comments

Comments
 (0)