Skip to content

Commit

Permalink
Merge pull request #1623 from jackchenjc/edgex-go-issue-4916
Browse files Browse the repository at this point in the history
feat: Enable PIE support for ASLR and full RELRO
  • Loading branch information
cloudxxx8 authored Oct 14, 2024
2 parents 5bbd138 + a719ca8 commit 186fda1
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions app-service-template/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,11 @@

.PHONY: build test clean docker

# change the following boolean flag to enable or disable the Full RELRO (RELocation Read Only) for linux ELF (Executable and Linkable Format) binaries
ENABLE_FULL_RELRO:="true"
# change the following boolean flag to enable or disable PIE for linux binaries which is needed for ASLR (Address Space Layout Randomization) on Linux, the ASLR support on Windows is enabled by default
ENABLE_PIE:="true"

GOTESTFLAGS?=-race

# VERSION file is not needed for local development, In the CI/CD pipeline, a temporary VERSION file is written
Expand All @@ -36,6 +41,14 @@ GOFLAGS=-ldflags "-s -w -X github.com/edgexfoundry/app-functions-sdk-go/v3/inter
#GIT_SHA=$(shell git rev-parse HEAD)
GIT_SHA=no-sha

ifeq ($(ENABLE_FULL_RELRO), "true")
GOFLAGS += -ldflags "-bindnow"
endif

ifeq ($(ENABLE_PIE), "true")
GOFLAGS += -buildmode=pie
endif

# CGO is enabled by default and causes docker builds to fail due to no gcc,
# but is required for test with -race, so must disable it for the builds only
build: tidy
Expand Down

0 comments on commit 186fda1

Please sign in to comment.