5417bff optimized rpath checking to only run readelf once. This breaks in the 
(rare) cases when a binary has both a RPATH and RUNPATH entry.

Move the readelf call out of the check_rpath function so both can be processed 
separately while still only readelf once.

Use simpler (and correct) calculation of $lower from before 5417bff.

Test library from https://github.com/fitzsim/add-runpath-to-rpath-elf under 
AGPL v3.0.

Resolves: #3667
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/3694

-- Commit Summary --

  * Check RPATH and RUNPATH separately in check-rpaths

-- File Changes --

    M scripts/check-rpaths-worker (10)
    A tests/data/misc/libboth-rpath-and-runpath.so (0)
    M tests/rpmbrp.at (10)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/3694.patch
https://github.com/rpm-software-management/rpm/pull/3694.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3694
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
https://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to