Browse Source

test 1541: verify getinfo values on first header callback

Reported-by: chensong1211 on github
Ref: #13125
Closes #13128
Stefan Eissing 1 month ago
parent
commit
6ea75877fa
4 changed files with 172 additions and 104 deletions
  1. 1 1
      tests/data/Makefile.inc
  2. 73 0
      tests/data/test1541
  3. 4 1
      tests/libtest/Makefile.inc
  4. 94 102
      tests/libtest/lib1541.c

+ 1 - 1
tests/data/Makefile.inc

@@ -195,7 +195,7 @@ test1508 test1509 test1510 test1511 test1512 test1513 test1514 test1515 \
 test1516 test1517 test1518 test1519 test1520 test1521 test1522 test1523 \
 test1524 test1525 test1526 test1527 test1528 test1529 test1530 test1531 \
 test1532 test1533 test1534 test1535 test1536 test1537 test1538 test1539 \
-test1540          test1542 test1543 test1544 test1545 \
+test1540 test1541 test1542 test1543 test1544 test1545 \
 \
 test1550 test1551 test1552 test1553 test1554 test1555 test1556 test1557 \
 test1558 test1559 test1560 test1561 test1562 test1563 test1564 test1565 \

+ 73 - 0
tests/data/test1541

@@ -0,0 +1,73 @@
+<testcase>
+<info>
+<keywords>
+HTTP
+HTTP GET
+chunked Transfer-Encoding
+Trailer:
+DELAY
+</keywords>
+</info>
+
+# Server-side
+<reply>
+<data>
+HTTP/1.1 200 OK swsclose
+Transfer-Encoding: chunked
+
+4
+data
+5
+d474
+
+0
+
+</data>
+<datacheck>
+CURLINFO_CONNECT_TIME_T on 1st header is OK
+CURLINFO_PRETRANSFER_TIME_T on 1st header is OK
+CURLINFO_STARTTRANSFER_TIME_T on 1st header is OK
+CURLINFO_TOTAL_TIME_T on 1st header is OK
+CURLINFO_APPCONNECT_TIME_T on 1st header is OK
+CURLINFO_SPEED_DOWNLOAD_T on 1st header is OK
+HTTP/1.1 200 OK swsclose
+Transfer-Encoding: chunked
+
+datad474
+CURLINFO_CONNECT_TIME_T on done is OK
+CURLINFO_PRETRANSFER_TIME_T on done is OK
+CURLINFO_STARTTRANSFER_TIME_T on done is OK
+CURLINFO_APPCONNECT_TIME_T on done is OK
+CURLINFO_SPEED_DOWNLOAD_T on done is OK
+CURLINFO_TOTAL_TIME_T on done is OK
+</datacheck>
+<servercmd>
+writedelay: 10
+</servercmd>
+</reply>
+# Client-side
+<client>
+<server>
+http
+</server>
+<tool>
+lib%TESTNUMBER
+</tool>
+<name>
+chunked with trailers and pausing the receive
+</name>
+<command>
+http://%HOSTIP:%HTTPPORT/%TESTNUMBER
+</command>
+</client>
+
+# Verify data after the test has been "shot"
+<verify>
+<protocol>
+GET /%TESTNUMBER HTTP/1.1
+Host: %HOSTIP:%HTTPPORT
+Accept: */*
+
+</protocol>
+</verify>
+</testcase>

+ 4 - 1
tests/libtest/Makefile.inc

@@ -59,7 +59,7 @@ noinst_PROGRAMS = chkhostname libauthretry libntlmconnect libprereq      \
  lib1518         lib1520 lib1521 lib1522 lib1523 \
  lib1525 lib1526 lib1527 lib1528 lib1529 lib1530 lib1531 lib1532 lib1533 \
  lib1534 lib1535 lib1536 lib1537 lib1538 lib1539 \
- lib1540         lib1542 lib1543         lib1545 \
+ lib1540 lib1541 lib1542 lib1543         lib1545 \
  lib1550 lib1551 lib1552 lib1553 lib1554 lib1555 lib1556 lib1557 \
  lib1558 lib1559 lib1560 lib1564 lib1565 lib1567 lib1568 lib1569 \
  lib1591 lib1592 lib1593 lib1594 lib1596 lib1597 lib1598 \
@@ -464,6 +464,9 @@ lib1539_CPPFLAGS = $(AM_CPPFLAGS) -DLIB1539
 lib1540_SOURCES = lib1540.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS)
 lib1540_LDADD = $(TESTUTIL_LIBS)
 
+lib1541_SOURCES = lib1541.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS)
+lib1541_LDADD = $(TESTUTIL_LIBS)
+
 lib1542_SOURCES = lib1542.c $(SUPPORTFILES) $(TESTUTIL) $(TSTTRACE) $(WARNLESS)
 lib1542_LDADD = $(TESTUTIL_LIBS)
 

+ 94 - 102
tests/libtest/lib1541.c

@@ -21,140 +21,132 @@
  * SPDX-License-Identifier: curl
  *
  ***************************************************************************/
-/*
- * KNOW_BUGS "A shared connection cache is not thread-safe"
- *
- * This source code was used to verify shared connection cache but since this
- * is a known issue the test is no longer built or run. This code is here to
- * allow for testing once someone gets to work on fixing this.
- */
 #include "test.h"
 
 #include "testutil.h"
 #include "warnless.h"
 #include "memdebug.h"
 
-#ifdef HAVE_PTHREAD_H
-#include <pthread.h>
-#include <time.h>
-
-/* number of threads to fire up in parallel */
-#define NUM_THREADS 67
-
-/* for how many seconds each thread will loop */
-#define RUN_FOR_SECONDS 7
+struct transfer_status {
+  CURL *easy;
+  int hd_count;
+  int bd_count;
+  CURLcode result;
+};
 
-static pthread_mutex_t connlock;
+#define KN(a)   a, #a
 
-static size_t write_db(void *ptr, size_t size, size_t nmemb, void *data)
+static int geterr(const char *name, CURLcode val, int lineno)
 {
-  /* not interested in the downloaded bytes, return the size */
-  (void)ptr;  /* unused */
-  (void)data; /* unused */
-  return (size_t)(size * nmemb);
+  printf("CURLINFO_%s returned %d, \"%s\" on line %d\n",
+         name, val, curl_easy_strerror(val), lineno);
+  return (int)val;
 }
 
-static void lock_cb(CURL *handle, curl_lock_data data,
-                    curl_lock_access access, void *userptr)
+static void report_time(const char *key, const char *where, curl_off_t time,
+                        bool ok)
 {
-  (void)access; /* unused */
-  (void)userptr; /* unused */
-  (void)handle; /* unused */
-  (void)data; /* unused */
-  pthread_mutex_lock(&connlock);
+  if(ok)
+    printf("%s on %s is OK\n", key, where);
+  else
+    printf("%s on %s is WRONG: %" CURL_FORMAT_CURL_OFF_T "\n",
+           key, where, time);
 }
 
-static void unlock_cb(CURL *handle, curl_lock_data data,
-                      void *userptr)
+static void check_time(CURL *easy, int key, const char *name,
+                       const char *where)
 {
-  (void)userptr; /* unused */
-  (void)handle;  /* unused */
-  (void)data;    /* unused */
-  pthread_mutex_unlock(&connlock);
+  curl_off_t tval;
+  CURLcode res = curl_easy_getinfo(easy, (CURLINFO)key, &tval);
+  if(res) {
+    geterr(name, res, __LINE__);
+  }
+  else
+    report_time(name, where, tval, tval > 0);
 }
 
-static void init_locks(void)
+static void check_time0(CURL *easy, int key, const char *name,
+                        const char *where)
 {
-  pthread_mutex_init(&connlock, NULL);
+  curl_off_t tval;
+  CURLcode res = curl_easy_getinfo(easy, (CURLINFO)key, &tval);
+  if(res) {
+    geterr(name, res, __LINE__);
+  }
+  else
+    report_time(name, where, tval, !tval);
 }
 
-static void kill_locks(void)
+static size_t header_callback(void *ptr, size_t size, size_t nmemb,
+                              void *userp)
 {
-  pthread_mutex_destroy(&connlock);
+  struct transfer_status *st = (struct transfer_status *)userp;
+  size_t len = size * nmemb;
+
+  (void)ptr;
+  if(!st->hd_count++) {
+    /* first header, check some CURLINFO value to be reported. See #13125 */
+    check_time(st->easy, KN(CURLINFO_CONNECT_TIME_T), "1st header");
+    check_time(st->easy, KN(CURLINFO_PRETRANSFER_TIME_T), "1st header");
+    check_time(st->easy, KN(CURLINFO_STARTTRANSFER_TIME_T), "1st header");
+    /* continuously updated */
+    check_time(st->easy, KN(CURLINFO_TOTAL_TIME_T), "1st header");
+    /* no SSL, must be 0 */
+    check_time0(st->easy, KN(CURLINFO_APPCONNECT_TIME_T), "1st header");
+    /* download not really started */
+    check_time0(st->easy, KN(CURLINFO_SPEED_DOWNLOAD_T), "1st header");
+  }
+  (void)fwrite(ptr, size, nmemb, stdout);
+  return len;
 }
 
-struct initurl {
-  const char *url;
-  CURLSH *share;
-  int threadno;
-};
-
-static void *run_thread(void *ptr)
+static size_t write_callback(void *ptr, size_t size, size_t nmemb, void *userp)
 {
-  struct initurl *u = (struct initurl *)ptr;
-  int i;
-  time_t end = time(NULL) + RUN_FOR_SECONDS;
-
-  for(i = 0; time(NULL) < end; i++) {
-    CURL *curl = curl_easy_init();
-    curl_easy_setopt(curl, CURLOPT_URL, u->url);
-    curl_easy_setopt(curl, CURLOPT_VERBOSE, 0L);
-    curl_easy_setopt(curl, CURLOPT_SHARE, u->share);
-    curl_easy_setopt(curl, CURLOPT_WRITEFUNCTION, write_db);
-    curl_easy_perform(curl); /* ignores error */
-    curl_easy_cleanup(curl);
-    fprintf(stderr, "Thread %d transfer %d\n", u->threadno, i);
-  }
+  struct transfer_status *st = (struct transfer_status *)userp;
 
-  return NULL;
+  (void)ptr;
+  (void)st;
+  fwrite(ptr, size, nmemb, stdout);
+  return size * nmemb;
 }
 
 int test(char *URL)
 {
-  pthread_t tid[NUM_THREADS];
-  int i;
-  CURLSH *share;
-  struct initurl url[NUM_THREADS];
-
-  /* Must initialize libcurl before any threads are started */
-  curl_global_init(CURL_GLOBAL_ALL);
-
-  share = curl_share_init();
-  curl_share_setopt(share, CURLSHOPT_LOCKFUNC, lock_cb);
-  curl_share_setopt(share, CURLSHOPT_UNLOCKFUNC, unlock_cb);
-  curl_share_setopt(share, CURLSHOPT_SHARE, CURL_LOCK_DATA_CONNECT);
-
-  init_locks();
-
-  for(i = 0; i< NUM_THREADS; i++) {
-    int error;
-    url[i].url = URL;
-    url[i].share = share;
-    url[i].threadno = i;
-    error = pthread_create(&tid[i], NULL, run_thread, &url[i]);
-    if(0 != error)
-      fprintf(stderr, "Couldn't run thread number %d, errno %d\n", i, error);
-    else
-      fprintf(stderr, "Thread %d, gets %s\n", i, URL);
-  }
+  CURL *curls = NULL;
+  int res = 0;
+  struct transfer_status st;
 
-  /* now wait for all threads to terminate */
-  for(i = 0; i< NUM_THREADS; i++) {
-    pthread_join(tid[i], NULL);
-    fprintf(stderr, "Thread %d terminated\n", i);
-  }
+  start_test_timing();
+
+  memset(&st, 0, sizeof(st));
+
+  global_init(CURL_GLOBAL_ALL);
+
+  easy_init(curls);
+  st.easy = curls; /* to allow callbacks access */
+
+  easy_setopt(curls, CURLOPT_URL, URL);
+  easy_setopt(curls, CURLOPT_WRITEFUNCTION, write_callback);
+  easy_setopt(curls, CURLOPT_WRITEDATA, &st);
+  easy_setopt(curls, CURLOPT_HEADERFUNCTION, header_callback);
+  easy_setopt(curls, CURLOPT_HEADERDATA, &st);
+
+  easy_setopt(curls, CURLOPT_NOPROGRESS, 0L);
 
-  kill_locks();
+  res = curl_easy_perform(curls);
 
-  curl_share_cleanup(share);
+  check_time(curls, KN(CURLINFO_CONNECT_TIME_T), "done");
+  check_time(curls, KN(CURLINFO_PRETRANSFER_TIME_T), "done");
+  check_time(curls, KN(CURLINFO_STARTTRANSFER_TIME_T), "done");
+  /* no SSL, must be 0 */
+  check_time0(curls, KN(CURLINFO_APPCONNECT_TIME_T), "done");
+  check_time(curls, KN(CURLINFO_SPEED_DOWNLOAD_T), "done");
+  check_time(curls, KN(CURLINFO_TOTAL_TIME_T), "done");
+
+test_cleanup:
+
+  curl_easy_cleanup(curls);
   curl_global_cleanup();
-  return 0;
-}
 
-#else /* without pthread, this test doesn't work */
-int test(char *URL)
-{
-  (void)URL;
-  return 0;
+  return (int)res; /* return the final return code */
 }
-#endif