Don't fclose(NULL). Don't declare string functions that are macros
authorKen Raeburn <raeburn@mit.edu>
Tue, 21 Sep 1999 22:56:49 +0000 (22:56 +0000)
committerKen Raeburn <raeburn@mit.edu>
Tue, 21 Sep 1999 22:56:49 +0000 (22:56 +0000)
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@11836 dc483132-0cff-0310-8789-dd5450dbe970

src/appl/gssftp/ftp/ChangeLog
src/appl/gssftp/ftp/ftp.c
src/appl/gssftp/ftpd/ChangeLog
src/appl/gssftp/ftpd/ftpd.c

index 893c318d57e5d5af564fde43d1c8c3daa9e1a13d..64a6890d582c1307ee794f22686468f7bd1452f1 100644 (file)
@@ -1,3 +1,8 @@
+1999-09-21  Ken Raeburn  <raeburn@mit.edu>
+
+       * ftp.c (getreply): Don't declare strpbrk or strstr functions if
+       they're defined as macros.
+
 1999-08-27  Tom Yu  <tlyu@mit.edu>
 
        * ftp.c: Diable krb5-mech2 for now.
index bccd2facc25b8ff1d624076a08d7388a01a81067..24049eabe6b6874b92bc94e16bbfa1f487ac3868 100644 (file)
@@ -541,7 +541,12 @@ getreply(expecteof)
        sigtype cmdabort();
        char ibuf[FTP_BUFSIZ], obuf[FTP_BUFSIZ];
        int safe = 0;
-       extern char *strpbrk(), *strstr();
+#ifndef strpbrk
+       extern char *strpbrk();
+#endif
+#ifndef strstr
+       extern char *strstr();
+#endif
 
        ibuf[0] = '\0';
        if (reply_parse) reply_ptr = reply_buf;
index c84e770288aeb6adda5d10f953739b5a07e002c6..3d5aa408ed2802a41c4fd44f582d8c1c07eb188f 100644 (file)
@@ -1,3 +1,10 @@
+1999-09-21  Ken Raeburn  <raeburn@mit.edu>
+
+       * ftpd.c (checkuser): Only call fclose on file handle if it's not
+       NULL.
+       (send_file_list): Don't declare strpbrk function if it's defined
+       as a macro.
+
 Thu Mar 25 23:13:56 1999  Tom Yu  <tlyu@mit.edu>
 
        * ftpd.c (login): Check that the luid is not the uid we want to
index d4987affcf160e96144e553cf17d79eaf850c06b..52be05fb6b87777f6c2d2bcfdac6fe9645ad84dc 100644 (file)
@@ -840,8 +840,8 @@ checkuser(name)
                        }
                  }
             }
+            (void) fclose(fd);
        }
-       (void) fclose(fd);
 
        return (0);
 }
@@ -2018,7 +2018,9 @@ void
 myoob()
 {
        char *cp, *cs;
+#ifndef strpbrk
        extern char *strpbrk();
+#endif
 
        /* only process if transfer occurring */
        if (!transflag)
@@ -2486,7 +2488,9 @@ send_file_list(whichfiles)
        FILE *dout = NULL;
        register char **dirlist, *dirname;
        int simple = 0;
+#ifndef strpbrk
        char *strpbrk();
+#endif
        int ret = 0;
 
        if (strpbrk(whichfiles, "~{[*?") != NULL) {