Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 16 Jan 2013 18:11:16 -0600
From:      Brooks Davis <brooks@freebsd.org>
To:        Kimmo Paasiala <kpaasial@gmail.com>
Cc:        freebsd-stable@freebsd.org, Dimitry Andric <dim@freebsd.org>
Subject:   Re: CLANG 3.2 breaks security/pam_ssh_agent_auth on stable/9
Message-ID:  <20130117001116.GD29437@lor.one-eyed-alien.net>
In-Reply-To: <CA%2B7WWSfE1bAr7GriRGAKBVpkiNE%2Btn-%2Bd0cO3vpNPY-SWxfghg@mail.gmail.com>
References:  <CA%2B7WWSeOpuAv9PL2_G6Z1Q_SLv5xcTjcujiw6Rs8tdBsrLO7wQ@mail.gmail.com> <50F6D20A.6070306@FreeBSD.org> <CA%2B7WWSfE1bAr7GriRGAKBVpkiNE%2Btn-%2Bd0cO3vpNPY-SWxfghg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help

--Qrgsu6vtpU/OV/zm
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jan 16, 2013 at 08:01:00PM +0200, Kimmo Paasiala wrote:
> On Wed, Jan 16, 2013 at 6:15 PM, Dimitry Andric <dim@freebsd.org> wrote:
> > On 2013-01-16 13:05, Kimmo Paasiala wrote:
> >>
> >> I just updated my stable/9 system after clang3.2 was added. My system
> >> is amd64, both world and kernel are compiled with clang3.2 and the
> >> default compiler is clang. I'm tracking the sources with GIT and the
> >> version I have corresponds to SVN revision r245451.
> >>
> >> Everything else seems to work but the pam authentication module
> >> security/pam_ssh_agent_auth segfaults immediately.
> >
> > ...
> >
> >> #0  0x0000000800ef2070 in strsvis () from /lib/libc.so.7
> >> #1  0x0000000800ef2584 in strvis () from /lib/libc.so.7
> >> #2  0x0000000800ef25e5 in strnvis () from /lib/libc.so.7
> >> #3  0x0000000801c0e2e7 in do_log () from
> >> /usr/local/lib/pam_ssh_agent_auth.so
> >> #4  0x0000000801c0e4ff in logit () from
> >> /usr/local/lib/pam_ssh_agent_auth.so
> >
> > ...
> >
> >> The str*vis() calls suggest that it's something in the libc maybe?
> >
> >
> > Brooks merged the new strvis implementations in r245439, so you may have
> > run into a bug with them.  I don't think this is caused specifically by
> > clang, at least not without more proof. :-)
> >
> > Can you try reverting to the revision just before r245439, rebuilding
> > and reinstalling at least libc, and see if the pam_ssh_agent_auth crash
> > goes away?
>=20
> I'm rebuilding world now. Took me some time to figure out how to
> revert the commits in git. I'll report back once finished.

NetBSD and OpenBSD use different signatures for strnvis(). :(
pam_ssh_agent_auth assumes that if the system has one it is the OpenBSD
one but ours is the NetBSD one.  The port will need to be patched to use
the openbsd version like it was doing or to swap the second and third
arguments when build on newer versions of FreeBSD.

-- Brooks

--Qrgsu6vtpU/OV/zm
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (FreeBSD)

iD4DBQFQ90GkXY6L6fI4GtQRAn2cAJd3pImKCFw5WQwGfrbq0h5O6u+bAKDNT1h0
qkkIS4ub6iaR3Si14aqVZw==
=sjN8
-----END PGP SIGNATURE-----

--Qrgsu6vtpU/OV/zm--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130117001116.GD29437>