On Wed, Feb 23, 2011 at 10:38:36PM +0100, Niels Baggesen wrote:
> I would like to solicit some input on the changes that I made to get
> it to compile and quell warnings, so would some of you who has a
> Kerberos setup please look at the attached dif? The important point
> is the first ten lines or so that tweaks the includes.

OK, next step waqs looking at the configure parts for ksm ...

The attached patch first fixes the problem that if you specify
ksm as a security module it MUST be the first given!

And then there a strange difference between indentation and paranthesis
in the detection logic, so I have a feeling that it never really
worked for other than Heimdal Kerberos

Opinions?

/Niels

-- 
Niels Baggesen - @home - Ã…rhus - Denmark - n...@users.sourceforge.net
The purpose of computing is insight, not numbers   ---   R W Hamming
Index: configure.d/config_modules_lib
===================================================================
--- configure.d/config_modules_lib      (revision 19988)
+++ configure.d/config_modules_lib      (working copy)
@@ -131,16 +131,18 @@
 # Handle the special case of KSM to see which crypto API we support
 #
 
-ac_cv_NETSNMP_USE_KERBEROS_MIT=no
 rebuilt_new_with_security=
 already_checked_krb5=no
 
 for sec in $new_with_security
 do
-  if test "x$sec" = "xksm" -a "xyes" = "x$already_checked_krb5"; then
-    continue
+  if test "x$sec" = "xksm"; then
+    if "xyes" = "x$already_checked_krb5"; then
+      continue
+    else
+      already_checked_krb5=yes
+    fi
   fi
-  already_checked_krb5=yes
 
   if test x$sec != xksm; then
     rebuilt_new_with_security="$rebuilt_new_with_security $sec"
@@ -199,10 +201,9 @@
     fi
 
     AC_MSG_CHECKING([to see which crypto API we need to use])
-    AC_CHECK_FUNC(krb5_c_encrypt,[
     AC_CACHE_VAL(ac_cv_NETSNMP_USE_KERBEROS_MIT,
       AC_CHECK_LIB(k5crypto, krb5_init_context,
-       ac_cv_NETSNMP_USE_KERBEROS_MIT=yes, ac_cv_MIT_NEW_CRYPTo=no
+       ac_cv_NETSNMP_USE_KERBEROS_MIT=yes, ac_cv_NETSNMP_USE_KERBEROS_MIT=no
       )
     )
 
@@ -214,7 +215,7 @@
        LIBS="$LIBS -lk5crypto"
       fi
     else
-      AC_MSG_RESULT(old MIT crypto API)])
+      AC_MSG_RESULT(old MIT crypto API)
     fi
   fi
 done
------------------------------------------------------------------------------
Free Software Download: Index, Search & Analyze Logs and other IT data in 
Real-Time with Splunk. Collect, index and harness all the fast moving IT data 
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business 
insights. http://p.sf.net/sfu/splunk-dev2dev 
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to