From c331c73ec6d16aa2c40eb7e9e0f2426c7170c47d Mon Sep 17 00:00:00 2001 From: Michal Marek Date: Thu, 13 Nov 2008 08:20:23 +0000 Subject: [PATCH] - Fixed a potential data loss in Curl_client_write() when the transfer is paused. --- CHANGES | 4 ++++ lib/sendf.c | 8 ++++---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/CHANGES b/CHANGES index dc66f9e29..e30f89c02 100644 --- a/CHANGES +++ b/CHANGES @@ -6,6 +6,10 @@ Changelog +Michal Marek (13 Nov 2008) +- Fixed a potential data loss in Curl_client_write() when the transfer is + paused. + Daniel Stenberg (11 Nov 2008) - Rainer Canavan filed bug #2255627 (http://curl.haxx.se/bug/view.cgi?id=2255627) which pointed out that a diff --git a/lib/sendf.c b/lib/sendf.c index 4fe0b4806..566e26aa6 100644 --- a/lib/sendf.c +++ b/lib/sendf.c @@ -425,7 +425,7 @@ static CURLcode pausewrite(struct SessionHandle *data, } -/* client_write() sends data to the write callback(s) +/* Curl_client_write() sends data to the write callback(s) The bit pattern defines to what "streams" to write to. Body and/or header. The defines are in sendf.h of course. @@ -442,6 +442,9 @@ CURLcode Curl_client_write(struct connectdata *conn, struct SessionHandle *data = conn->data; size_t wrote; + if(0 == len) + len = strlen(ptr); + /* If reading is actually paused, we're forced to append this chunk of data to the already held data, but only if it is the same type as otherwise it can't work and it'll return error instead. */ @@ -469,9 +472,6 @@ CURLcode Curl_client_write(struct connectdata *conn, return CURLE_OK; } - if(0 == len) - len = strlen(ptr); - if(type & CLIENTWRITE_BODY) { if((conn->protocol&PROT_FTP) && conn->proto.ftpc.transfertype == 'A') { #ifdef CURL_DOES_CONVERSIONS