Skip to content

Commit

Permalink
mk: remove library search path from binary
Browse files Browse the repository at this point in the history
[ upstream commit 6b01864 ]

This patch functionally reverts the patch in fixes line to not have any
hardcoded library path in the final binary for the security reasons, in
case this binary distributed to production environment.

RPATH only added in RTE_DEVEL_BUILD case and this binary shouldn't
distributed, but still removing it to be cautious.

Fixes: 8919f73 ("mk: add build directory to library search path")

Suggested-by: Bruce Richardson <bruce.richardson@intel.com>
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
  • Loading branch information
Ferruh Yigit authored and kevintraynor committed Dec 11, 2019
1 parent 65ce5cd commit 7280388
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
1 change: 1 addition & 0 deletions devtools/test-null.sh
Expand Up @@ -8,6 +8,7 @@ build=${1:-build}
coremask=${2:-3} # default using cores 0 and 1

if grep -q SHARED_LIB=y $build/.config; then
export LD_LIBRARY_PATH=$build/lib:$LD_LIBRARY_PATH
pmd='-d librte_pmd_null.so'
fi

Expand Down
4 changes: 0 additions & 4 deletions mk/rte.app.mk
Expand Up @@ -333,10 +333,6 @@ filter-libs = \

LDLIBS := $(call filter-libs,$(LDLIBS))

ifeq ($(RTE_DEVEL_BUILD)$(CONFIG_RTE_BUILD_SHARED_LIB),yy)
LDFLAGS += -rpath=$(RTE_SDK_BIN)/lib
endif

MAPFLAGS = -Map=$@.map --cref

.PHONY: all
Expand Down

0 comments on commit 7280388

Please sign in to comment.