]> granicus.if.org Git - curl/commitdiff
smtp: Fixed const'ness of nread parameter in Curl_smtp_escape_eob()
authorSteve Holme <steve_holme@hotmail.com>
Wed, 26 Nov 2014 22:58:39 +0000 (22:58 +0000)
committerSteve Holme <steve_holme@hotmail.com>
Wed, 26 Nov 2014 23:31:57 +0000 (23:31 +0000)
...and some comment typos!

lib/smtp.c
lib/smtp.h

index 2dbd2ea30a357c9ffa99fd60da68aa30120a9cf2..d57e6272316b73ac379387eea465745541cead69 100644 (file)
@@ -2308,7 +2308,7 @@ static CURLcode smtp_calc_sasl_details(struct connectdata *conn,
   return result;
 }
 
-CURLcode Curl_smtp_escape_eob(struct connectdata *conn, ssize_t nread)
+CURLcode Curl_smtp_escape_eob(struct connectdata *conn, const ssize_t nread)
 {
   /* When sending a SMTP payload we must detect CRLF. sequences making sure
      they are sent as CRLF.. instead, as a . on the beginning of a line will
@@ -2323,7 +2323,7 @@ CURLcode Curl_smtp_escape_eob(struct connectdata *conn, ssize_t nread)
   char *scratch = data->state.scratch;
   char *oldscratch = NULL;
 
-  /* Do we need to allocate the scatch buffer? */
+  /* Do we need to allocate a scratch buffer? */
   if(!scratch || data->set.crlf) {
     oldscratch = scratch;
 
@@ -2380,10 +2380,8 @@ CURLcode Curl_smtp_escape_eob(struct connectdata *conn, ssize_t nread)
     smtp->eob = 0;
   }
 
+  /* Only use the new buffer if we replaced something */
   if(si != nread) {
-    /* Only use the new buffer if we replaced something */
-    nread = si;
-
     /* Upload from the new (replaced) buffer instead */
     data->req.upload_fromhere = scratch;
 
@@ -2394,7 +2392,7 @@ CURLcode Curl_smtp_escape_eob(struct connectdata *conn, ssize_t nread)
     Curl_safefree(oldscratch);
 
     /* Set the new amount too */
-    data->req.upload_present = nread;
+    data->req.upload_present = si;
   }
   else
     Curl_safefree(scratch);
index db1b1e672890b4173f4975061a7788b95cf49758..7a95efeafd70c43d2ba85fd90ab0a5503c395c40 100644 (file)
@@ -101,6 +101,6 @@ extern const struct Curl_handler Curl_handler_smtps;
 #define SMTP_EOB_REPL "\x0d\x0a\x2e\x2e"
 #define SMTP_EOB_REPL_LEN 4
 
-CURLcode Curl_smtp_escape_eob(struct connectdata *conn, ssize_t nread);
+CURLcode Curl_smtp_escape_eob(struct connectdata *conn, const ssize_t nread);
 
 #endif /* HEADER_CURL_SMTP_H */