[GNUnet-SVN] [gnunet] branch master updated: benchmark: fix request size collection and add invariant check

[gnunet]

This is an automated email from the git hooks/post-receive script.

dold pushed a commit to branch master
in repository gnunet.

The following commit(s) were added to refs/heads/master by this push:
     new afe1bb767 benchmark: fix request size collection and add invariant 
check
afe1bb767 is described below

commit afe1bb7673a350d86228e2d916e346e27e4fe7a6
Author: Florian Dold <address@hidden>
AuthorDate: Fri May 3 01:01:39 2019 +0200

    benchmark: fix request size collection and add invariant check
---
 src/curl/curl.c | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/curl/curl.c b/src/curl/curl.c
index 4bd2a55d0..1c352f195 100644
--- a/src/curl/curl.c
+++ b/src/curl/curl.c
@@ -526,11 +526,20 @@ GNUNET_CURL_perform2 (struct GNUNET_CURL_Context *ctx,
                                      &size_long));
     bytes_sent += size_long;
 
+    /* We obtain this value to check an invariant, but never use it otherwise. 
*/
     GNUNET_break (CURLE_OK ==
                   curl_easy_getinfo (cmsg->easy_handle,
                                      CURLINFO_SIZE_UPLOAD_T,
                                      &size_curl));
-    bytes_sent += size_curl;
+
+    /* CURLINFO_SIZE_UPLOAD_T <= CURLINFO_REQUEST_SIZE should
+       be an invariant.
+       As verified with
+         curl -w "foo%{size_request} -XPOST --data "ABC" $URL
+      the CURLINFO_REQUEST_SIZE should be the whole size of the request
+      including headers and body.
+     */
+    GNUNET_break (size_curl <= size_long);
 
     urd = get_url_benchmark_data (url, (unsigned int) response_code);
     urd->count++;

-- 
To stop receiving notification emails like this one, please contact
address@hidden