From: Stephen Smalley <sds@tycho.nsa.gov>
To: Laurent Bigonville <bigon@debian.org>
Cc: SELinux <selinux@tycho.nsa.gov>, Daniel J Walsh <dwalsh@redhat.com>
Subject: Re: [PATCH 1/2] Explicitly link libselinux against -lpthread
Date: Mon, 04 Nov 2013 08:09:46 -0500 [thread overview]
Message-ID: <52779C9A.4050306@tycho.nsa.gov> (raw)
In-Reply-To: <1383434030-5853-1-git-send-email-bigon@debian.org>
On 11/02/2013 07:13 PM, Laurent Bigonville wrote:
> From: Laurent Bigonville <bigon@bigon.be>
>
> libselinux is using pthread functions internally without explicitly
> linking against it.
>
> If the executable is itself not linked against libpthread, this could
> lead to some weird ld.so assertions, see:
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=728529
This seems to effectively revert a portion of:
commit c32da69e016061c1a06ec08298aae8c995fbea31
Author: Dan Walsh <dwalsh@redhat.com>
Date: Wed Oct 9 16:27:43 2013 -0400
Fixes for procattr calls to handle cache properly.
We were asked not to link to libpthread but to use gcc internals.
We were not handling properly the fact that a cache was UNSET, and this
patch fixes this.
Can the two of you work out a proper fix that works for you both?
> ---
> libselinux/src/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libselinux/src/Makefile b/libselinux/src/Makefile
> index 02dd829..2a79b27 100644
> --- a/libselinux/src/Makefile
> +++ b/libselinux/src/Makefile
> @@ -114,7 +114,7 @@ $(LIBA): $(OBJS)
> $(RANLIB) $@
>
> $(LIBSO): $(LOBJS)
> - $(CC) $(CFLAGS) -shared -o $@ $^ -lpcre -ldl $(LDFLAGS) -L$(LIBDIR) -Wl,-soname,$(LIBSO),-z,defs,-z,relro
> + $(CC) $(CFLAGS) -shared -o $@ $^ -lpcre -lpthread -ldl $(LDFLAGS) -L$(LIBDIR) -Wl,-soname,$(LIBSO),-z,defs,-z,relro
> ln -sf $@ $(TARGET)
>
> $(LIBPC): $(LIBPC).in ../VERSION
>
--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.
next prev parent reply other threads:[~2013-11-04 13:09 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-02 23:13 [PATCH 1/2] Explicitly link libselinux against -lpthread Laurent Bigonville
2013-11-02 23:13 ` [PATCH 2/2] src/libselinux.pc.in: Move -lpthread to Libs.private Laurent Bigonville
2013-11-04 13:09 ` Stephen Smalley [this message]
2013-11-04 13:18 ` [PATCH 1/2] Explicitly link libselinux against -lpthread Stephen Smalley
2013-11-04 13:59 ` Daniel J Walsh
2013-11-04 14:16 ` Laurent Bigonville
2013-11-06 15:40 ` Sven Vermeulen
2013-11-06 17:09 ` Stephen Smalley
2013-11-06 17:19 ` Laurent Bigonville
2013-11-06 17:35 ` Stephen Smalley
2013-11-06 20:00 ` Stephen Smalley
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52779C9A.4050306@tycho.nsa.gov \
--to=sds@tycho.nsa.gov \
--cc=bigon@debian.org \
--cc=dwalsh@redhat.com \
--cc=selinux@tycho.nsa.gov \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.