feat: Enable PIE support for ASLR and full RELRO - Githubissues

edgexfoundry / app-rfid-llrp-inventory

Apache License 2.0

9 stars 20 forks source link

feat: Enable PIE support for ASLR and full RELRO #309

Closed jackchenjc closed 1 month ago

jackchenjc commented 1 month ago

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/app-rfid-llrp-inventory/blob/main/.github/Contributing.md

PR Checklist

Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

Run the following command to build binary

make build

Verify that Full RELRO and PIE enabled are displayed

checksec --file=./app-rfid-llrp-inventory
RELRO           STACK CANARY      NX            PIE             RPATH      RUNPATH  Symbols     FORTIFY Fortified   Fortifiable FILE
Full RELRO      No canary found   NX enabled    PIE enabled     No RPATH   No RUNPATH   59528) Symbols    No    0       0       ./app-rfid-llrp-inventory

New Dependency Instructions (If applicable)