security.c: Curl_sec_fflush_fd tweaks
authorJulien Chaffraix <julien.chaffraix@gmail.com>
Sun, 12 Sep 2010 23:32:41 +0000 (16:32 -0700)
committerDaniel Stenberg <daniel@haxx.se>
Wed, 22 Sep 2010 21:34:36 +0000 (23:34 +0200)
- Use an early return as it makes the code more readable.
- Added a FIXME about a conversion.

lib/security.c

index 699be47..d0eccdf 100644 (file)
@@ -336,12 +336,14 @@ static ssize_t sec_write(struct connectdata *conn, int fd,
   return tx;
 }
 
-int
-Curl_sec_fflush_fd(struct connectdata *conn, int fd)
+/* FIXME: fd should be a curl_socket_t */
+int Curl_sec_fflush_fd(struct connectdata *conn, int fd)
 {
-  if(conn->data_prot != prot_clear) {
-    do_sec_send(conn, fd, NULL, 0);
-  }
+  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;
 }