struct krb_principal lossage

[John Hawkinson]

(Sorry for the delay, I had to fight with putting my local copy under
CVS -- is it really wise for include/.cvsignore to list config.h.in?).


| Is this enougth ?

Not quite.

Your patch was malformed (did you edit it by hand), this patch fixed it :-):

--- p0	Mon Mar  2 12:22:49 1998
+++ p1	Mon Mar  2 12:23:47 1998
@@ -42,7 +42,7 @@
 diff -u -w -r1.78 -r1.79
 --- configure.in        1998/03/02 13:46:14     1.78
 +++ configure.in        1998/03/02 14:57:10     1.79
-@@ -204,6 +204,23 @@
+@@ -204,3 +204,20 @@
  fi
  
  CFLAGS="-I../include $CFLAGS"
@@ -73,8 +73,8 @@
 +
 +/* We have krb_principal from kth-krb ? */
 +#undef HAVE_KRB_PRINCIPAL
---- stds.h   1998/03/02 13:14:52     1.1
-+++ stds.h   1998/03/02 14:58:06     1.2
+--- include/stds.h   1998/03/02 13:14:52     1.1
++++ include/stds.h   1998/03/02 14:58:06     1.2
 @@ -47,6 +47,15 @@
  
  #ifndef HAVE_U_INT32

Fun, that. Your stds.h patch didn't include definitions of ANAME_SZ, etc., and
nothing else was included to define them. So:

Index: include/stds.h
===================================================================
RCS file: /afs/sipb/project/arla/repository/arla/include/stds.h,v
retrieving revision 1.1.1.1
diff -c -r1.1.1.1 stds.h
*** stds.h	1998/03/02 17:42:22	1.1.1.1
--- stds.h	1998/03/02 18:14:09
***************
*** 47,52 ****
--- 47,66 ----
  
  #ifndef HAVE_U_INT32
  typedef u_int32_t u_int32;
+ #endif
+ 
+ /* for compat resons with other kerberos distributions */
+ #ifndef HAVE_KRB_PRINCIPAL 
+ 
+ #define         ANAME_SZ        40
+ #define         REALM_SZ        40
+ #define         INST_SZ         40
+ 
+ typedef struct krb_principal{
+     char name[ANAME_SZ];
+     char instance[INST_SZ];
+     char realm[REALM_SZ];
+ }krb_principal;
  #endif
  
  #endif /* __STDS_H */

Other than that, fine.

--jhawk