From: Ken Raeburn <raeburn@mit.edu>
Date: Mon, 9 Nov 2009 06:13:34 +0000 (+0000)
Subject: quoting bug causes solaris pre-10 thread handling bugs
X-Git-Tag: krb5-1.8-alpha1~198
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=70e96202ed342a2241ef5f14f55ce32e24679adf;p=krb5.git

quoting bug causes solaris pre-10 thread handling bugs

Quoting problems in pattern matching on the OS name cause Solaris
versions up through 9 to not be properly recognized in the
thread-system configuration setup.  This causes our libraries to make
the erroneous assumption that valid thread support routines are
available on all Solaris systems, rather than just assuming it for
Solaris 10 and later.

The result is assertion failures like this one reported by Meraj
Mohammed and others:

  Assertion failed: k5int_i->did_run != 0, file krb5_libinit.c, line 63

Thanks to Tom Shaw for noticing the cause of the problem.

The bug may be present in the 1.6.x series as well.

ticket: 6579
target_version: 1.7.1
tags: pullup

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@23144 dc483132-0cff-0310-8789-dd5450dbe970
---

diff --git a/src/aclocal.m4 b/src/aclocal.m4
index 77ad1660f..ef4b210f9 100644
--- a/src/aclocal.m4
+++ b/src/aclocal.m4
@@ -205,7 +205,9 @@ if test "$enable_thread_support" = yes; then
       # don't exclude CFLAGS when linking.  *sigh*
       PTHREAD_CFLAGS="-D_REENTRANT -D_THREAD_SAFE -D_POSIX_C_SOURCE=199506L"
       ;;
+changequote(<<,>>)
     solaris2.[1-9])
+changequote([,])
       # On Solaris 10 with gcc 3.4.3, the autoconf archive macro doesn't
       # get the right result.   XXX What about Solaris 9 and earlier?
       if test "$GCC" = yes ; then