Skip to content

Commit

Permalink
Merge pull request #78 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 c25e0a7 + 3ed6416 commit 5252636
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 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-uart

.PHONY: $(MICROSERVICES)
Expand All @@ -14,6 +19,14 @@ GOFLAGS=-ldflags "-X github.com/edgexfoundry/device-uart.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 5252636

Please sign in to comment.