aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYang Tse <yangsita@gmail.com>2012-12-05 17:28:32 +0100
committerYang Tse <yangsita@gmail.com>2012-12-05 17:28:32 +0100
commit60edbf65b6d1bfd264e10a72cd35fb81bfad7b8d (patch)
treeaa0b83f21e9d90bca434bcbf79456a9704d746f0
parentfe2b2a3b9dbe18883cf7c9789cd958915978dc3a (diff)
libtest: fix some compiler warnings
-rw-r--r--tests/libtest/Makefile.inc2
-rw-r--r--tests/libtest/lib1502.c9
-rw-r--r--tests/libtest/lib540.c11
-rw-r--r--tests/libtest/lib591.c11
-rw-r--r--tests/libtest/lib597.c11
-rw-r--r--tests/libtest/libntlmconnect.c18
6 files changed, 44 insertions, 18 deletions
diff --git a/tests/libtest/Makefile.inc b/tests/libtest/Makefile.inc
index 881622a90..c773d2b73 100644
--- a/tests/libtest/Makefile.inc
+++ b/tests/libtest/Makefile.inc
@@ -220,5 +220,5 @@ lib1502_SOURCES = lib1502.c $(SUPPORTFILES)
libauthretry_SOURCES = libauthretry.c $(SUPPORTFILES)
-libntlmconnect_SOURCES = libntlmconnect.c $(SUPPORTFILES) $(TESTUTIL)
+libntlmconnect_SOURCES = libntlmconnect.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS)
libntlmconnect_LDADD = $(TESTUTIL_LIBS)
diff --git a/tests/libtest/lib1502.c b/tests/libtest/lib1502.c
index ec1058826..bb008f17c 100644
--- a/tests/libtest/lib1502.c
+++ b/tests/libtest/lib1502.c
@@ -26,6 +26,10 @@
#include "test.h"
+#ifdef HAVE_LIMITS_H
+#include <limits.h>
+#endif
+
#include <stdio.h>
#include <string.h>
@@ -92,11 +96,12 @@ int test(char *URL)
curl_multi_timeout(multi_handle, &curl_timeo);
if(curl_timeo >= 0) {
- timeout.tv_sec = curl_timeo / 1000;
+ int itimeout = (curl_timeo > (long)INT_MAX) ? INT_MAX : (int)curl_timeo;
+ timeout.tv_sec = itimeout / 1000;
if(timeout.tv_sec > 1)
timeout.tv_sec = 1;
else
- timeout.tv_usec = (curl_timeo % 1000) * 1000;
+ timeout.tv_usec = (itimeout % 1000) * 1000;
}
/* get file descriptors from the transfers */
diff --git a/tests/libtest/lib540.c b/tests/libtest/lib540.c
index a47f1dee6..ac0ebe60b 100644
--- a/tests/libtest/lib540.c
+++ b/tests/libtest/lib540.c
@@ -5,7 +5,7 @@
* | (__| |_| | _ <| |___
* \___|\___/|_| \_\_____|
*
- * Copyright (C) 1998 - 2011, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2012, Daniel Stenberg, <daniel@haxx.se>, et al.
*
* This software is licensed as described in the file COPYING, which
* you should have received as part of this distribution. The terms
@@ -30,6 +30,10 @@
#include "test.h"
+#ifdef HAVE_LIMITS_H
+#include <limits.h>
+#endif
+
#include "testutil.h"
#include "warnless.h"
#include "memdebug.h"
@@ -139,8 +143,9 @@ static int loop(int num, CURLM *cm, const char* url, const char* userpwd,
/* At this point, L is guaranteed to be greater or equal than -1. */
if(L != -1) {
- T.tv_sec = L/1000;
- T.tv_usec = (L%1000)*1000;
+ int itimeout = (L > (long)INT_MAX) ? INT_MAX : (int)L;
+ T.tv_sec = itimeout/1000;
+ T.tv_usec = (itimeout%1000)*1000;
}
else {
T.tv_sec = 5;
diff --git a/tests/libtest/lib591.c b/tests/libtest/lib591.c
index 8a55e2cd1..5cd4644e3 100644
--- a/tests/libtest/lib591.c
+++ b/tests/libtest/lib591.c
@@ -5,7 +5,7 @@
* | (__| |_| | _ <| |___
* \___|\___/|_| \_\_____|
*
- * Copyright (C) 1998 - 2011, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2012, Daniel Stenberg, <daniel@haxx.se>, et al.
*
* This software is licensed as described in the file COPYING, which
* you should have received as part of this distribution. The terms
@@ -23,6 +23,10 @@
/* lib591 is used for test cases 591, 592, 593 and 594 */
+#ifdef HAVE_LIMITS_H
+#include <limits.h>
+#endif
+
#include <fcntl.h>
#include "testutil.h"
@@ -112,8 +116,9 @@ int test(char *URL)
/* At this point, timeout is guaranteed to be greater or equal than -1. */
if(timeout != -1L) {
- interval.tv_sec = timeout/1000;
- interval.tv_usec = (timeout%1000)*1000;
+ int itimeout = (timeout > (long)INT_MAX) ? INT_MAX : (int)timeout;
+ interval.tv_sec = itimeout/1000;
+ interval.tv_usec = (itimeout%1000)*1000;
}
else {
interval.tv_sec = 0;
diff --git a/tests/libtest/lib597.c b/tests/libtest/lib597.c
index 6a5ee4f8f..a27cefd5b 100644
--- a/tests/libtest/lib597.c
+++ b/tests/libtest/lib597.c
@@ -5,7 +5,7 @@
* | (__| |_| | _ <| |___
* \___|\___/|_| \_\_____|
*
- * Copyright (C) 1998 - 2011, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2012, Daniel Stenberg, <daniel@haxx.se>, et al.
*
* This software is licensed as described in the file COPYING, which
* you should have received as part of this distribution. The terms
@@ -21,6 +21,10 @@
***************************************************************************/
#include "test.h"
+#ifdef HAVE_LIMITS_H
+#include <limits.h>
+#endif
+
#include "testutil.h"
#include "warnless.h"
#include "memdebug.h"
@@ -113,8 +117,9 @@ int test(char *URL)
/* At this point, timeout is guaranteed to be greater or equal than -1. */
if(timeout != -1L) {
- interval.tv_sec = timeout/1000;
- interval.tv_usec = (timeout%1000)*1000;
+ int itimeout = (timeout > (long)INT_MAX) ? INT_MAX : (int)timeout;
+ interval.tv_sec = itimeout/1000;
+ interval.tv_usec = (itimeout%1000)*1000;
}
else {
interval.tv_sec = TEST_HANG_TIMEOUT/1000+1;
diff --git a/tests/libtest/libntlmconnect.c b/tests/libtest/libntlmconnect.c
index fd64e5f84..66d09e936 100644
--- a/tests/libtest/libntlmconnect.c
+++ b/tests/libtest/libntlmconnect.c
@@ -21,8 +21,13 @@
***************************************************************************/
#include "test.h"
+#ifdef HAVE_LIMITS_H
+#include <limits.h>
+#endif
#include <assert.h>
+
#include "testutil.h"
+#include "warnless.h"
#include "memdebug.h"
#define TEST_HANG_TIMEOUT 5 * 1000
@@ -40,7 +45,7 @@ int res = 0;
static size_t callback(char* ptr, size_t size, size_t nmemb, void* data)
{
- int idx = ((CURL **) data) - easy;
+ ssize_t idx = ((CURL **) data) - easy;
curl_socket_t sock;
long lastsock;
@@ -69,8 +74,8 @@ static size_t callback(char* ptr, size_t size, size_t nmemb, void* data)
sockets[idx] = sock;
}
else if (sock != sockets[idx]) {
- fprintf(stderr, "Handle %d started on socket %d and moved to %d\n", idx,
- sockets[idx], sock);
+ fprintf(stderr, "Handle %d started on socket %d and moved to %d\n",
+ curlx_sztosi(idx), (int)sockets[idx], (int)sock);
res = TEST_ERR_MAJOR_BAD;
return 0;
}
@@ -195,7 +200,7 @@ int test(char *url)
matched socket_exists should be true and we would never get here */
assert(curfd != sockets[num_handles-1]);
fprintf(stderr, "Handle %d wrote to socket %d then detected on %d\n",
- num_handles-1, sockets[num_handles-1], curfd);
+ num_handles-1, (int)sockets[num_handles-1], (int)curfd);
res = TEST_ERR_MAJOR_BAD;
goto test_cleanup;
}
@@ -224,8 +229,9 @@ int test(char *url)
__FILE__, __LINE__, num_handles, timeout);
if(timeout != -1L) {
- interval.tv_sec = timeout/1000;
- interval.tv_usec = (timeout%1000)*1000;
+ int itimeout = (timeout > (long)INT_MAX) ? INT_MAX : (int)timeout;
+ interval.tv_sec = itimeout/1000;
+ interval.tv_usec = (itimeout%1000)*1000;
}
else {
interval.tv_sec = TEST_HANG_TIMEOUT/1000+1;