* rc_io.c (getdir): Subtract one from dirlen, since sizeof

	includes the terminating NUL character.  Avoid redundant path
	separators.
	(krb5_rc_io_move): When renaming OLD to NEW, don't copy the
	filename.  This was causing temporary files to get leaked.
	(krb5_rc_io_close): Don't FREE if d->fn is NULL.

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

diff --git a/src/lib/krb5/rcache/ChangeLog b/src/lib/krb5/rcache/ChangeLog
index 933a574652185fca4f6d169422ce04e9ea040bbc..622d4fe1c2f40e616536b8f67259c4e3ddcfdfb6 100644 (file)
--- a/src/lib/krb5/rcache/ChangeLog
+++ b/src/lib/krb5/rcache/ChangeLog
@@ -1,3 +1,12 @@
+2001-01-23  Tom Yu  <tlyu@mit.edu>
+
+       * rc_io.c (getdir): Subtract one from dirlen, since sizeof
+       includes the terminating NUL character.  Avoid redundant path
+       separators.
+       (krb5_rc_io_move): When renaming OLD to NEW, don't copy the
+       filename.  This was causing temporary files to get leaked.
+       (krb5_rc_io_close): Don't FREE if d->fn is NULL.
+
 2001-01-16  Ken Raeburn  <raeburn@mit.edu>
 
        * rcfns.c: New file, with function versions of former dispatch

diff --git a/src/lib/krb5/rcache/rc_io.c b/src/lib/krb5/rcache/rc_io.c
index 549788f5276fea6625ac5c6a0c06365dada76a56..8c7853eeb8d4bed49a04df62f63795f29b52e0dd 100644 (file)
--- a/src/lib/krb5/rcache/rc_io.c
+++ b/src/lib/krb5/rcache/rc_io.c
@@ -61,7 +61,7 @@ static void getdir()
 #if defined(_MSDOS) || defined(_WIN32)
      if (!(dir = getenv("TEMP")))
         if (!(dir = getenv("TMP")))
-            dir = "C:\\";
+            dir = "C:";
 #else
      if (!(dir = getenv("TMPDIR")))
 #ifdef RCTMPDIR
@@ -70,7 +70,7 @@ static void getdir()
        dir = "/tmp";
 #endif
 #endif
-   dirlen = strlen(dir) + sizeof(PATH_SEPARATOR);
+   dirlen = strlen(dir) + sizeof(PATH_SEPARATOR) - 1;
 }
 
 krb5_error_code krb5_rc_io_creat (context, d, fn)
@@ -242,33 +242,32 @@ krb5_error_code krb5_rc_io_move (context, new, old)
     krb5_rc_iostuff *new;
     krb5_rc_iostuff *old;
 {
+    char *fn = NULL;
+
 #if defined(_MSDOS) || defined(_WIN32)
     /*
      * Work around provided by Tom Sanfilippo to work around poor
      * Windows emulation of POSIX functions.  Rename and dup has
      * different semantics!
      */
-    char *fn = NULL;
     GETDIR;
     close(new->fd);
     unlink(new->fn);
     close(old->fd);
     if (rename(old->fn,new->fn) == -1) /* MUST be atomic! */
        return KRB5_RC_IO_UNKNOWN;
-    if (!(fn = malloc(strlen(new->fn) - dirlen + 1)))
-       return KRB5_RC_IO_MALLOC;
-    strcpy(fn, new->fn + dirlen);
+    fn = new->fn;
+    new->fn = NULL;            /* avoid clobbering */
     krb5_rc_io_close(context, new);
     krb5_rc_io_open(context, new, fn);
     free(fn);
 #else
     if (rename(old->fn,new->fn) == -1) /* MUST be atomic! */
        return KRB5_RC_IO_UNKNOWN;
+    fn = new->fn;
+    new->fn = NULL;            /* avoid clobbering */
     (void) krb5_rc_io_close(context, new);
-    new->fn = malloc(strlen(old->fn)+1);
-    if (new->fn == 0)
-       return ENOMEM;
-    strcpy(new->fn, old->fn);
+    new->fn = fn;
 #ifdef macintosh
     new->fd = fcntl(old->fd, F_DUPFD);
 #else
@@ -339,7 +338,8 @@ krb5_error_code krb5_rc_io_close (context, d)
     krb5_context context;
     krb5_rc_iostuff *d;
 {
- FREE(d->fn);
+ if (d->fn != NULL)
+   FREE(d->fn);
  d->fn = NULL;
  if (close(d->fd) == -1) /* can't happen */
    return KRB5_RC_IO_UNKNOWN;