Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 728882 - net-misc/rsync-3.2.0: warning about executable stack
Summary: net-misc/rsync-3.2.0: warning about executable stack
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
: 729146 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-06-20 11:27 UTC by Holger Hoffstätte
Modified: 2020-06-22 10:08 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Add --noexecstack for assembly routines (noexecstack.patch,931 bytes, patch)
2020-06-20 18:29 UTC, Holger Hoffstätte
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Holger Hoffstätte 2020-06-20 11:27:56 UTC
After updating to rsync-3.2.0:

Jun 20 11:31:04 ragnarok kernel: process '/usr/bin/rsync' started with executable stack

This also happens with vanilla upstream git (not sure why), I just noticed it too late for release. Probably just need to add noexecstack to LDFLAGS.

Reproducible: Always

Steps to Reproduce:
1. emerge rsync-3.2.0
2. rsync something
3. should find warning message in syslog (only once).


Expected Results:  
Silence is golden.
Comment 1 Holger Hoffstätte 2020-06-20 12:33:33 UTC
A few more observations:

- adding -z,noexecstack to LDFLAGS indeed fixes it

- echo 1 > /sys/kernel/debug/clear_warn_once clears WARN_ONCE things so that
one can test reproducibly without rebooting :)
Comment 2 Holger Hoffstätte 2020-06-20 14:58:16 UTC
It's due to raw assembly in lib/md5-asm-x86_64.s.
I added --noexecstack to the build and have opened an upstream PR.
Comment 3 Holger Hoffstätte 2020-06-20 18:29:22 UTC
Created attachment 645432 [details, diff]
Add --noexecstack for assembly routines

This is now fixed upstream but of course no longer applies since the author
decided to pile on a bunch of unrelated fixes on top.
This patch works against 3.2.0.
Comment 4 Larry the Git Cow gentoo-dev 2020-06-21 09:31:18 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6e2a72e6b9381c465f0e95914d044ba373f656ae

commit 6e2a72e6b9381c465f0e95914d044ba373f656ae
Author:     Lars Wendler <polynomial-c@gentoo.org>
AuthorDate: 2020-06-21 09:30:29 +0000
Commit:     Lars Wendler <polynomial-c@gentoo.org>
CommitDate: 2020-06-21 09:31:07 +0000

    net-misc/rsync: Revbump fixing stunnel build and removed execstack
    
    Closes: https://bugs.gentoo.org/728882
    Closes: https://bugs.gentoo.org/728898
    Package-Manager: Portage-2.3.101, Repoman-2.3.22
    Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

 net-misc/rsync/files/rsync-3.2.0-noexecstack.patch | 24 +++++++++++
 net-misc/rsync/files/rsync-3.2.0-simd_check.patch  | 11 -----
 .../{rsync-3.2.0.ebuild => rsync-3.2.0-r1.ebuild}  | 48 ++++++++++++++--------
 3 files changed, 56 insertions(+), 27 deletions(-)
Comment 5 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2020-06-22 10:08:57 UTC
*** Bug 729146 has been marked as a duplicate of this bug. ***