edgexfoundry / device-onvif-camera

Owner: Device WG
Apache License 2.0
29 stars 37 forks source link

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

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/edgex-go/blob/main/.github/Contributing.md

PR Checklist

Please check if your PR fulfills the following requirements:

Testing Instructions

Run the following command to build binary

make build

Verify that Full RELRO and PIE enabled are displayed

checksec --file=./cmd/device-onvif-camera
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   54522) Symbols    No    0       0       ./cmd/device-onvif-camera

New Dependency Instructions (If applicable)

codecov-commenter commented 1 month ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 32.76%. Comparing base (c03763b) to head (a83730c).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #465 +/- ## ======================================= Coverage 32.76% 32.76% ======================================= Files 21 21 Lines 1923 1923 ======================================= Hits 630 630 Misses 1217 1217 Partials 76 76 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.