security.c: Remove Curl_sec_fflush_fd.
authorJulien Chaffraix <julien.chaffraix@gmail.com>
Mon, 27 Sep 2010 05:35:00 +0000 (22:35 -0700)
committerJulien Chaffraix <julien.chaffraix@gmail.com>
Wed, 29 Sep 2010 05:05:24 +0000 (22:05 -0700)
The current implementation would make us send wrong data on a closed
socket. We don't buffer our data so the method can be safely removed.

lib/ftp.c
lib/krb4.h
lib/security.c

index e3f9387..f1376d7 100644 (file)
--- a/lib/ftp.c
+++ b/lib/ftp.c
@@ -3076,10 +3076,6 @@ static CURLcode ftp_done(struct connectdata *conn, CURLcode status,
   /* free the dir tree and file parts */
   freedirs(ftpc);
 
-#if defined(HAVE_KRB4) || defined(HAVE_GSSAPI)
-  Curl_sec_fflush_fd(conn, conn->sock[SECONDARYSOCKET]);
-#endif
-
   /* shut down the socket to inform the server we're done */
 
 #ifdef _WIN32_WCE
index 5dc3971..5da1dc6 100644 (file)
@@ -47,7 +47,6 @@ extern struct Curl_sec_client_mech Curl_krb5_client_mech;
 #endif
 
 CURLcode Curl_krb_kauth(struct connectdata *conn);
-int Curl_sec_fflush_fd(struct connectdata *conn, int fd);
 int Curl_sec_fprintf (struct connectdata *, FILE *, const char *, ...);
 int Curl_sec_getc (struct connectdata *conn, FILE *);
 int Curl_sec_putc (struct connectdata *conn, int, FILE *);
index 5becb0c..303a1be 100644 (file)
@@ -345,17 +345,6 @@ static ssize_t sec_write(struct connectdata *conn, curl_socket_t fd,
   return tx;
 }
 
-/* FIXME: fd should be a curl_socket_t */
-int Curl_sec_fflush_fd(struct connectdata *conn, int fd)
-{
-  if(conn->data_prot == prot_clear)
-    return 0;
-
-  /* Force a flush by trying to send no data */
-  do_sec_send(conn, fd, NULL, 0);
-  return 0;
-}
-
 /* Matches Curl_send signature */
 static ssize_t sec_send(struct connectdata *conn, int sockindex,
                         const void *buffer, size_t len, CURLcode *err)