Fixes so that the Unix changes no longer breaks on the PC
authorKeith Vetter <keithv@fusion.com>
Fri, 28 Apr 1995 01:18:29 +0000 (01:18 +0000)
committerKeith Vetter <keithv@fusion.com>
Fri, 28 Apr 1995 01:18:29 +0000 (01:18 +0000)
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@5587 dc483132-0cff-0310-8789-dd5450dbe970

14 files changed:
src/include/ChangeLog
src/include/krb5.h
src/include/krb5/ChangeLog
src/include/krb5/adm_proto.h
src/include/krb5/k5-config.h
src/lib/krb5/krb/ChangeLog
src/lib/krb5/krb/adm_rw.c
src/lib/krb5/krb/mk_cred.c
src/lib/krb5/krb/rd_cred.c
src/lib/krb5/os/ChangeLog
src/lib/krb5/os/adm_conn.c
src/lib/krb5/os/genaddrs.c
src/lib/krb5/os/mk_faddr.c
src/lib/krb5/os/os-proto.h

index bb920ba3a17829092b5aa1c02a65debdc3bd1a4f..094bc56dfbfbd307f2c7646b07fc3d7bca4fa0c9 100644 (file)
@@ -1,3 +1,6 @@
+Thu Apr 27 17:57:36 1995 Keith Vetter (keithv@fusion.com)
+
+       * krb5.h: krb5_mk_req somehow got INTERFACE added to it--removed it.
 
 Tue Apr 25 21:58:23 1995  Chris Provenzano  (proven@mit.edu)
 
index 3a2d456aa974ee986aaf5883f1691a6eab536747..65b6e26865336d8396a54b943060f11e5e4c6404 100644 (file)
@@ -1266,7 +1266,7 @@ krb5_error_code INTERFACE krb5_get_cred_via_tkt
                    krb5_address * const *,
                    krb5_creds *,
                    krb5_creds **));
-krb5_error_code INTERFACE krb5_mk_req
+krb5_error_code krb5_mk_req
        PROTOTYPE((krb5_context,
                   krb5_auth_context **,
                   const krb5_flags,
index 26edc36cb4561d719033ae9e7a1a53e9cd5a2a66..1905f2afc56bfb222dd80d0deebc57ef1b517289 100644 (file)
@@ -1,3 +1,8 @@
+Thu Apr 27 17:39:04 1995 Keith Vetter (keithv@fusion.com)
+
+       * k5-config.h: Added MAXPATHLEN constant.
+       * adm_prot.h: updated prototype.
+
 Thu Apr 27 11:24:24 1995 Keith Vetter (keithv@fusion.com)
 
        * k5-config.h: Changed over to the new single config file on the PC.
index c515fdf8e620230d17becf827b62babcbcdfc7bc..11bc1eb6c3eb6af7243979a244242936faaa8c90 100644 (file)
@@ -46,7 +46,7 @@ void INTERFACE krb5_adm_disconnect
 /* adm_proto.c */
 void INTERFACE krb5_free_adm_data
        PROTOTYPE((krb5_context,
-                  int,
+                  krb5_int32,
                   krb5_data *));
 
 krb5_error_code INTERFACE krb5_send_adm_cmd
index a8134ca0c1037b016dccf4b64c2b4ec225420775..a4d24c3d6615bc6e286f956c77ac39fec8eb7b0c 100644 (file)
@@ -49,6 +49,7 @@
 #define SIZEOF_SHORT    2
 #define SIZEOF_LONG     4
 #define MAXHOSTNAMELEN  512
+#define MAXPATHLEN     128
 
 #define KRB5_USE_INET
 #define MSDOS_FILESYSTEM
index cd4cf7beb66556d6941f61dcc45f7e34cfc8ebf6..eb60c3cecd4dae71487d17115c699eafae8169cd 100644 (file)
@@ -1,3 +1,8 @@
+Thu Apr 27 17:40:20 1995 Keith Vetter (keithv@fusion.com)
+
+       * adm_rw.c, mk_cred.c, rd_cred.c:
+          malloc on the PC must be size SIZE_T not int32.
+       * adm_rw.c: krb5_free_adm_data second argument now a krb5_int32.
 
 Thu Apr 27 16:33:17 EDT 1995   Paul Park       (pjpark@mit.edu)
 
index 6155f8f732abf78ed9c2498fd52ac06487410ff2..afd466d2ad801e426c7ad96c3a04f5d61343cd6b 100644 (file)
@@ -25,6 +25,7 @@
 /*
  * Routines to engage in the administrative (password changing) protocol.
  */
+#define NEED_SOCKETS
 #include "k5-int.h"
 #include "auth_con.h"
 #include "adm_proto.h"
@@ -79,7 +80,7 @@ kadm_copyout_int32(outint, cp)
 void INTERFACE
 krb5_free_adm_data(kcontext, ncomp, datap)
     krb5_context       kcontext;
-    int                        ncomp;
+    krb5_int32         ncomp;
     krb5_data          *datap;
 {
     int i;
@@ -329,12 +330,12 @@ krb5_read_adm_cmd(kcontext, sock, ctx, nargs, arglist)
 
                    /* Get the memory for the list */
                    if (*arglist = (krb5_data *)
-                       malloc((*nargs) * sizeof(krb5_data))) {
+                       malloc((size_t) (*nargs) * sizeof(krb5_data))) {
                        krb5_data *xarglist;
 
                        xarglist = *arglist;
-                       memset((char *) (xarglist), 0, 
-                              (*nargs) * sizeof(krb5_data));
+                       memset((char *) (xarglist), 0,
+                               (size_t) (*nargs) * sizeof(krb5_data));
 
                        replyok = 1;
                        /* Copy out each list entry */
@@ -454,12 +455,12 @@ krb5_read_adm_reply(kcontext, sock, ctx, cmd_stat, ncomps, complist)
 
                    /* Get the memory for the list */
                    if (*complist = (krb5_data *)
-                       malloc((*ncomps) * sizeof(krb5_data))) {
+                       malloc((size_t) ((*ncomps) * sizeof(krb5_data)))) {
                        krb5_data *xcomplist;
 
                        xcomplist = *complist;
                        memset((char *) (xcomplist), 0, 
-                              (*ncomps) * sizeof(krb5_data));
+                              (size_t) ((*ncomps) * sizeof(krb5_data)));
 
                        replyok = 1;
                        /* Copy out each list entry */
index 6b95301a3b2652c76401a788c0eb7b238c2816bf..5fbd63f39a59320c1023f2b2e70052df8f6f0377 100644 (file)
@@ -8,6 +8,9 @@
  *
  * MODIFIED
  * $Log$
+ * Revision 5.9  1995/04/28 01:18:18  keithv
+ * Fixes so that the Unix changes no longer breaks on the PC.
+ *
  * Revision 5.8  1995/04/26 03:03:11  proven
  *     * Makefile.in : Added gc_via_tkt.c and removed get_fcreds.c
  *     * auth_con.c (krb5_auth_con_setaddrs()) : Fixed so it allocates
@@ -146,14 +149,14 @@ krb5_mk_ncred_basic(context, ppcreds, nppcreds, keyblock,
 
     /* Get memory for creds and initialize it */
     if ((credenc.ticket_info = (krb5_cred_info **)
-               malloc(sizeof(krb5_cred_info *) * (nppcreds + 1))) == NULL) {
+               malloc((size_t) (sizeof(krb5_cred_info *) * (nppcreds + 1)))) == NULL) {
         return ENOMEM;
     }
-    if ((tmp = (char *)malloc(sizeof(krb5_cred_info) * nppcreds)) == NULL) {
+    if ((tmp = (char *)malloc((size_t) (sizeof(krb5_cred_info) * nppcreds))) == NULL) {
        retval = ENOMEM;
        goto cleanup_info;
     }
-    memset(tmp, 0, sizeof(krb5_cred_info) * nppcreds);
+    memset(tmp, 0, (size_t) (sizeof(krb5_cred_info) * nppcreds));
 
     /*
      * For each credential in the list, initialize a cred info
index 179a922b70d971698cdb6eb3397b74a182803c00..f221d9a2e2b795a7baca8535e362b6308447c640 100644 (file)
@@ -126,8 +126,9 @@ krb5_rd_cred_basic(context, pcreddata, pkeyblock, local_addr, remote_addr,
     * krb5_free_tgt_creds can be used to free the list.
     */
     for (ncreds = 0; pcred->tickets[ncreds]; ncreds++);
+       
     if ((*pppcreds = 
-        (krb5_creds **)malloc(sizeof(krb5_creds *) * ncreds + 1)) == NULL) {
+        (krb5_creds **)malloc((size_t)(sizeof(krb5_creds *) * ncreds + 1))) == NULL) {
         retval = ENOMEM;
         goto cleanup_cred;
     }
index 67c27e8214930f9a35dd21b010d08ca5865ba89b..513489d10fd281d26d998686565cb94c5fcd04c4 100644 (file)
@@ -1,3 +1,11 @@
+Thu Apr 27 17:50:14 1995 Keith Vetter (keithv@fusion.com)
+
+       * adm_conn.c: added constant to pull in needed PC code, and
+          removed two unused variables.
+       * genaddrs.c: needed casts on getsockname and getpeername calls.
+       * mk_faddr.c: the int->byte code needed explicit casts.
+       * os-proto.h: added missing prototype for krb5_make_fulladdr.
+
 Wed Apr 26 16:00:52 EDT 1995   Paul Park       (pjpark@mit.edu)
 
        * Add adm_conn.c - module to provide a connection to an administrative
index c036fddf7f21a39d96716e3af312819030025c44..7eb7bcb62f25aa9e2467d1ef50dc72a6df12fff1 100644 (file)
@@ -25,6 +25,7 @@
  * Routines to contact an administrative protocol server.
  */
 #define        NEED_SOCKETS
+#define        NEED_LOWLEVEL_IO
 #include "k5-int.h"
 #include "adm.h"
 #include "adm_proto.h"
@@ -208,8 +209,6 @@ kadm_contact_server(kcontext, realmp, sockp, local, remote)
     struct hostent     *remote_host;
     struct servent     *service;
     char               **hostlist;
-    int                        host_count;
-    int                        namelen;
     int                        i, count;
 
     krb5_error_code    kret;
index 1ca9dee012bca4b2ce8b5f4194aaf5456c959ac6..f13ea09cf06fa4cd3f923240483e000ea1de8c5c 100644 (file)
@@ -51,7 +51,7 @@ krb5_auth_con_genaddrs(context, auth_context, fd, flags)
     ssize = sizeof(struct sockaddr);
     if ((flags & KRB5_AUTH_CONTEXT_GENERATE_LOCAL_FULL_ADDR) ||
        (flags & KRB5_AUTH_CONTEXT_GENERATE_LOCAL_ADDR)) {
-       if (retval = getsockname(fd, &saddr, &ssize)) 
+       if (retval = getsockname(fd, (struct sockaddr *) &saddr, &ssize)) 
            return retval;
 
        if (flags & KRB5_AUTH_CONTEXT_GENERATE_LOCAL_FULL_ADDR) {
@@ -69,7 +69,7 @@ krb5_auth_con_genaddrs(context, auth_context, fd, flags)
 
     if ((flags & KRB5_AUTH_CONTEXT_GENERATE_REMOTE_FULL_ADDR) ||
        (flags & KRB5_AUTH_CONTEXT_GENERATE_REMOTE_ADDR)) {
-        if (retval = getpeername(fd, &saddr, &ssize))
+        if (retval = getpeername(fd, (struct sockaddr *) &saddr, &ssize))
            return retval;
 
        if (flags & KRB5_AUTH_CONTEXT_GENERATE_REMOTE_FULL_ADDR) {
index 3d090edd8368c9f459a83ee000e383a1e3c89cfa..e2256418dd71fbab2991bcac48fa319c3592142f 100644 (file)
 #include "k5-int.h"
 
 #ifdef KRB5_USE_INET
-
+   
 #include "os-proto.h"
 #ifndef _WINSOCKAPI_
 #include <netinet/in.h>
 #endif
 
-krb5_error_code INTERFACE
+krb5_error_code
 krb5_make_fulladdr(context, saddr, kaddr)
     krb5_context         context;
     struct sockaddr_in         * saddr;
@@ -58,14 +58,14 @@ krb5_make_fulladdr(context, saddr, kaddr)
     tmp16 = ADDRTYPE_INET;
     *marshal++ = 0x00;
     *marshal++ = 0x00;
-    *marshal++ = tmp16 & 0xff;
-    *marshal++ = (tmp16 >> 8) & 0xff;
+    *marshal++ = (krb5_octet) (tmp16 & 0xff);
+    *marshal++ = (krb5_octet) ((tmp16 >> 8) & 0xff);
 
     tmp32 = sizeof(smushaddr);
-    *marshal++ = tmp32 & 0xff;
-    *marshal++ = (tmp32 >> 8) & 0xff;
-    *marshal++ = (tmp32 >> 16) & 0xff;
-    *marshal++ = (tmp32 >> 24) & 0xff;
+    *marshal++ = (krb5_octet) (tmp32 & 0xff);
+    *marshal++ = (krb5_octet) ((tmp32 >> 8) & 0xff);
+    *marshal++ = (krb5_octet) ((tmp32 >> 16) & 0xff);
+    *marshal++ = (krb5_octet) ((tmp32 >> 24) & 0xff);
 
     (void) memcpy((char *)marshal, (char *)&smushaddr, sizeof(smushaddr));
     marshal += sizeof(smushaddr);
@@ -73,14 +73,14 @@ krb5_make_fulladdr(context, saddr, kaddr)
     tmp16 = ADDRTYPE_IPPORT;
     *marshal++ = 0x00;
     *marshal++ = 0x00;
-    *marshal++ = tmp16 & 0xff;
-    *marshal++ = (tmp16 >> 8) & 0xff;
+    *marshal++ = (krb5_octet) (tmp16 & 0xff);
+    *marshal++ = (krb5_octet) ((tmp16 >> 8) & 0xff);
 
     tmp32 = sizeof(smushport);
-    *marshal++ = tmp32 & 0xff;
-    *marshal++ = (tmp32 >> 8) & 0xff;
-    *marshal++ = (tmp32 >> 16) & 0xff;
-    *marshal++ = (tmp32 >> 24) & 0xff;
+    *marshal++ = (krb5_octet) (tmp32 & 0xff);
+    *marshal++ = (krb5_octet) ((tmp32 >> 8) & 0xff);
+    *marshal++ = (krb5_octet) ((tmp32 >> 16) & 0xff);
+    *marshal++ = (krb5_octet) ((tmp32 >> 24) & 0xff);
 
     (void) memcpy((char *)marshal, (char *)&smushport, sizeof(smushport));
     marshal += sizeof(smushport);
index e73610d5f2435e43a96d95e27a3d0ecbea2ac156..895923c38d5cb94b1e967ab4b8eb71260a18a292 100644 (file)
@@ -48,6 +48,12 @@ krb5_error_code krb5_make_full_ipaddr
               int,                     /* unsigned short promotes to signed
                                           int */
               krb5_address **));
+
+krb5_error_code krb5_make_fulladdr
+    PROTOTYPE((krb5_context,
+              struct sockaddr_in *,
+              krb5_address *));
+
 #endif /* KRB5_USE_INET */
 
 #endif /* KRB5_LIBOS_INT_PROTO__ */