aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYang Tse <yangsita@gmail.com>2006-07-12 05:19:00 +0000
committerYang Tse <yangsita@gmail.com>2006-07-12 05:19:00 +0000
commit58176d14848f315ea74772bd9ce7edd847b2a17d (patch)
tree42685c540585d5c2d1205176734da6f2a54222e4
parent10489879f731465f3d2105870e7533c2b355af5d (diff)
Use platform's native types for recv() and send() arguments.
-rw-r--r--lib/sendf.c2
-rw-r--r--lib/setup.h71
-rw-r--r--lib/telnet.c10
-rw-r--r--tests/server/sockfilt.c3
-rw-r--r--tests/server/sws.c4
5 files changed, 70 insertions, 20 deletions
diff --git a/lib/sendf.c b/lib/sendf.c
index d248c189a..d598197bd 100644
--- a/lib/sendf.c
+++ b/lib/sendf.c
@@ -332,7 +332,7 @@ CURLcode Curl_write(struct connectdata *conn,
/* only TRUE if krb4 enabled */
bytes_written = Curl_sec_write(conn, sockfd, mem, len);
else
- bytes_written = (ssize_t)swrite(sockfd, mem, len);
+ bytes_written = swrite(sockfd, mem, len);
if(-1 == bytes_written) {
int err = Curl_sockerrno();
diff --git a/lib/setup.h b/lib/setup.h
index 57ac87d5b..cf7216ba0 100644
--- a/lib/setup.h
+++ b/lib/setup.h
@@ -214,6 +214,65 @@ typedef unsigned char bool;
#define struct_stat struct stat
#endif /* Win32 with large file support */
+/*
+ * The definitions for the return type and arguments types
+ * of functions recv() and send() belong and come from the
+ * configuration file. Do not define them in any other place.
+ *
+ * HAVE_RECV is defined if you have a function named recv()
+ * which is used to read incoming data from sockets. If your
+ * function has another name then don't define HAVE_RECV.
+ *
+ * If HAVE_RECV is defined then RECV_TYPE_ARG1, RECV_TYPE_ARG2,
+ * RECV_TYPE_ARG3, RECV_TYPE_ARG4 and RECV_TYPE_RETV must also
+ * be defined.
+ *
+ * HAVE_SEND is defined if you have a function named send()
+ * which is used to write outgoing data on a connected socket.
+ * If yours has another name then don't define HAVE_SEND.
+ *
+ * If HAVE_SEND is defined then SEND_TYPE_ARG1, SEND_QUAL_ARG2,
+ * SEND_TYPE_ARG2, SEND_TYPE_ARG3, SEND_TYPE_ARG4 and
+ * SEND_TYPE_RETV must also be defined.
+ */
+
+#ifdef HAVE_RECV
+#if !defined(RECV_TYPE_ARG1) || \
+ !defined(RECV_TYPE_ARG2) || \
+ !defined(RECV_TYPE_ARG3) || \
+ !defined(RECV_TYPE_ARG4) || \
+ !defined(RECV_TYPE_RETV)
+ /* */
+ Error: Missing definition of return and arguments types of recv().
+ /* */
+#else
+#define sread(x,y,z) (ssize_t)recv((RECV_TYPE_ARG1)x, (RECV_TYPE_ARG2)y, (RECV_TYPE_ARG3)z, (RECV_TYPE_ARG4)SEND_4TH_ARG)
+#endif
+#else /* HAVE_RECV */
+#ifdef DJGPP
+#define sread(x,y,z) (ssize_t)read_s((int)x, (char *)y, (int)z)
+#endif
+#endif /* HAVE_RECV */
+
+#ifdef HAVE_SEND
+#if !defined(SEND_TYPE_ARG1) || \
+ !defined(SEND_QUAL_ARG2) || \
+ !defined(SEND_TYPE_ARG2) || \
+ !defined(SEND_TYPE_ARG3) || \
+ !defined(SEND_TYPE_ARG4) || \
+ !defined(SEND_TYPE_RETV)
+ /* */
+ Error: Missing definition of return and arguments types of send().
+ /* */
+#else
+#define swrite(x,y,z) (ssize_t)send((SEND_TYPE_ARG1)x, (SEND_TYPE_ARG2)y, (SEND_TYPE_ARG3)z, (SEND_TYPE_ARG4)SEND_4TH_ARG)
+#endif
+#else /* HAVE_SEND */
+#ifdef DJGPP
+#define swrite(x,y,z) (ssize_t)write_s((int)x, (char *)y, (int)z)
+#endif
+#endif /* HAVE_SEND */
+
/* Below we define four functions. They should
1. close a socket
@@ -229,16 +288,10 @@ typedef unsigned char bool;
#if !defined(__GNUC__) || defined(__MINGW32__)
#define sclose(x) closesocket(x)
-/* Since Windows doesn't have/use the POSIX prototype for send() and recv(),
- we typecast the third argument in the macros to avoid compiler warnings. */
-#define sread(x,y,z) recv(x,y,(int)(z), SEND_4TH_ARG)
-#define swrite(x,y,z) (size_t)send(x,y, (int)(z), SEND_4TH_ARG)
#undef HAVE_ALARM
#else
/* gcc-for-win is still good :) */
#define sclose(x) close(x)
-#define sread(x,y,z) recv(x,y,z, SEND_4TH_ARG)
-#define swrite(x,y,z) send(x,y,z, SEND_4TH_ARG)
#define HAVE_ALARM
#endif /* !GNU or mingw */
@@ -250,8 +303,6 @@ typedef unsigned char bool;
#ifdef DJGPP
#include <sys/ioctl.h>
#define sclose(x) close_s(x)
-#define sread(x,y,z) read_s(x,y,z)
-#define swrite(x,y,z) write_s(x,y,z)
#define select(n,r,w,x,t) select_s(n,r,w,x,t)
#define ioctl(x,y,z) ioctlsocket(x,y,(char *)(z))
#define IOCTL_3_ARGS
@@ -264,12 +315,8 @@ typedef unsigned char bool;
#ifdef __BEOS__
#define sclose(x) closesocket(x)
-#define sread(x,y,z) (ssize_t)recv(x,y,z, SEND_4TH_ARG)
-#define swrite(x,y,z) (ssize_t)send(x,y,z, SEND_4TH_ARG)
#else /* __BEOS__ */
#define sclose(x) close(x)
-#define sread(x,y,z) recv(x,y,z, SEND_4TH_ARG)
-#define swrite(x,y,z) send(x,y,z, SEND_4TH_ARG)
#endif /* __BEOS__ */
#define HAVE_ALARM
diff --git a/lib/telnet.c b/lib/telnet.c
index a609cd092..0b57f7706 100644
--- a/lib/telnet.c
+++ b/lib/telnet.c
@@ -289,12 +289,13 @@ static void printoption(struct SessionHandle *data,
static void send_negotiation(struct connectdata *conn, int cmd, int option)
{
unsigned char buf[3];
+ ssize_t bytes_written;
buf[0] = CURL_IAC;
buf[1] = cmd;
buf[2] = option;
- (void)swrite(conn->sock[FIRSTSOCKET], (char *)buf, 3);
+ bytes_written = swrite(conn->sock[FIRSTSOCKET], buf, 3);
printoption(conn->data, "SENT", cmd, option);
}
@@ -843,6 +844,7 @@ static void suboption(struct connectdata *conn)
{
struct curl_slist *v;
unsigned char temp[2048];
+ ssize_t bytes_written;
size_t len;
size_t tmplen;
char varname[128];
@@ -857,7 +859,7 @@ static void suboption(struct connectdata *conn)
snprintf((char *)temp, sizeof(temp),
"%c%c%c%c%s%c%c", CURL_IAC, CURL_SB, CURL_TELOPT_TTYPE,
CURL_TELQUAL_IS, tn->subopt_ttype, CURL_IAC, CURL_SE);
- (void)swrite(conn->sock[FIRSTSOCKET], (char *)temp, len);
+ bytes_written = swrite(conn->sock[FIRSTSOCKET], temp, len);
printsub(data, '>', &temp[2], len-2);
break;
case CURL_TELOPT_XDISPLOC:
@@ -865,7 +867,7 @@ static void suboption(struct connectdata *conn)
snprintf((char *)temp, sizeof(temp),
"%c%c%c%c%s%c%c", CURL_IAC, CURL_SB, CURL_TELOPT_XDISPLOC,
CURL_TELQUAL_IS, tn->subopt_xdisploc, CURL_IAC, CURL_SE);
- (void)swrite(conn->sock[FIRSTSOCKET], (char *)temp, len);
+ bytes_written = swrite(conn->sock[FIRSTSOCKET], temp, len);
printsub(data, '>', &temp[2], len-2);
break;
case CURL_TELOPT_NEW_ENVIRON:
@@ -888,7 +890,7 @@ static void suboption(struct connectdata *conn)
snprintf((char *)&temp[len], sizeof(temp) - len,
"%c%c", CURL_IAC, CURL_SE);
len += 2;
- (void)swrite(conn->sock[FIRSTSOCKET], (char *)temp, len);
+ bytes_written = swrite(conn->sock[FIRSTSOCKET], temp, len);
printsub(data, '>', &temp[2], len-2);
break;
}
diff --git a/tests/server/sockfilt.c b/tests/server/sockfilt.c
index eec0cd39f..636132615 100644
--- a/tests/server/sockfilt.c
+++ b/tests/server/sockfilt.c
@@ -173,6 +173,7 @@ static int juggle(curl_socket_t *sockfdp,
unsigned char buffer[256]; /* FIX: bigger buffer */
char data[256];
curl_socket_t sockfd;
+ ssize_t bytes_written;
timeout.tv_sec = 120;
timeout.tv_usec = 0;
@@ -301,7 +302,7 @@ static int juggle(curl_socket_t *sockfdp,
}
else
/* send away on the socket */
- swrite(sockfd, buffer, len);
+ bytes_written = swrite(sockfd, buffer, len);
}
else if(!memcmp("DISC", buffer, 4)) {
/* disconnect! */
diff --git a/tests/server/sws.c b/tests/server/sws.c
index 95429dfcd..3eff97d45 100644
--- a/tests/server/sws.c
+++ b/tests/server/sws.c
@@ -493,7 +493,7 @@ static int get_request(int sock, struct httprequest *req)
/* returns -1 on failure */
static int send_doc(int sock, struct httprequest *req)
{
- int written;
+ ssize_t written;
size_t count;
const char *buffer;
char *ptr;
@@ -519,7 +519,7 @@ static int send_doc(int sock, struct httprequest *req)
count = strlen(STREAMTHIS);
while(1) {
written = swrite(sock, STREAMTHIS, count);
- if(written != (int)count) {
+ if(written != count) {
logmsg("Stopped streaming");
break;
}