Skip to content

Commit

Permalink
Merge pull request #701 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 9248015 + e0a7d63 commit 2822fc2
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
.PHONY: build test unittest lint clean prepare update 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"

MICROSERVICES=cmd/device-mqtt

ARCH=$(shell uname -m)
Expand All @@ -18,6 +23,14 @@ GOFLAGS=-ldflags "-X github.com/edgexfoundry/device-mqtt-go.Version=$(VERSION) \
-X github.com/edgexfoundry/device-sdk-go/v3/internal/common.SDKVersion=$(SDKVERSION)" \
-trimpath -mod=readonly

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

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

build: $(MICROSERVICES)

build-nats:
Expand Down

0 comments on commit 2822fc2

Please sign in to comment.