--- libaitrpc/src/cli.c	2012/05/19 00:29:51	1.10
+++ libaitrpc/src/cli.c	2012/11/19 21:50:26	1.13
@@ -3,7 +3,7 @@
 *  by Michael Pounov <misho@openbsd-bg.org>
 *
 * $Author: misho $
-* $Id: cli.c,v 1.10 2012/05/19 00:29:51 misho Exp $
+* $Id: cli.c,v 1.13 2012/11/19 21:50:26 misho Exp $
 *
 **************************************************************************
 The ELWIX and AITNET software is distributed under the following
@@ -57,13 +57,14 @@ rpc_cli_t *
 rpc_cli_openBLOBClient(rpc_cli_t * __restrict rpccli, u_short Port)
 {
 	rpc_cli_t *cli = NULL;
+	int n;
 
 	if (!rpccli) {
 		rpc_SetErr(EINVAL, "Invalid parameters can`t connect to BLOB server");
 		return NULL;
 	}
 
-	cli = malloc(sizeof(rpc_cli_t));
+	cli = io_malloc(sizeof(rpc_cli_t));
 	if (!cli) {
 		LOGERR;
 		return NULL;
@@ -89,18 +90,31 @@ rpc_cli_openBLOBClient(rpc_cli_t * __restrict rpccli, 
 	}
 
 	AIT_COPY_VAL(&cli->cli_buf, &rpccli->cli_buf);
+	n = AIT_LEN(&cli->cli_buf);
 
 	/* connect to BLOB server */
 	cli->cli_sock = socket(cli->cli_sa.sa.sa_family, SOCK_STREAM, 0);
 	if (cli->cli_sock == -1) {
 		LOGERR;
-		free(cli);
+		io_free(cli);
 		return NULL;
 	}
+	if (setsockopt(cli->cli_sock, SOL_SOCKET, SO_SNDBUF, &n, sizeof n) == -1) {
+		LOGERR;
+		close(cli->cli_sock);
+		io_free(cli);
+		return NULL;
+	}
+	if (setsockopt(cli->cli_sock, SOL_SOCKET, SO_RCVBUF, &n, sizeof n) == -1) {
+		LOGERR;
+		close(cli->cli_sock);
+		io_free(cli);
+		return NULL;
+	}
 	if (connect(cli->cli_sock, &cli->cli_sa.sa, cli->cli_sa.sa.sa_len) == -1) {
 		LOGERR;
 		close(cli->cli_sock);
-		free(cli);
+		io_free(cli);
 		return NULL;
 	} else
 		fcntl(cli->cli_sock, F_SETFL, fcntl(cli->cli_sock, F_GETFL) | O_NONBLOCK);
@@ -125,7 +139,7 @@ rpc_cli_closeBLOBClient(rpc_cli_t ** __restrict cli)
 
 	AIT_FREE_VAL(&(*cli)->cli_buf);
 
-	free(*cli);
+	io_free(*cli);
 	*cli = NULL;
 }
 
@@ -139,22 +153,26 @@ rpc_cli_closeBLOBClient(rpc_cli_t ** __restrict cli)
  * @netBuf = Network buffer length (min:512 bytes), if =0 == BUFSIZ (also meaning max RPC packet)
  * @csHost = Host name or IP address for bind server
  * @Port = Port for bind server, if Port == 0 default port is selected
+ * @proto = Protocol, if == 0 choose SOCK_STREAM
  * return: NULL == error or !=NULL connection to RPC server established
  */
 rpc_cli_t *
-rpc_cli_openClient(u_int ProgID, u_char ProcID, int netBuf, const char *csHost, u_short Port)
+rpc_cli_openClient(u_int ProgID, u_char ProcID, int netBuf, const char *csHost, 
+		u_short Port, int proto)
 {
 	rpc_cli_t *cli = NULL;
-	io_sockaddr_t sa;
+	io_sockaddr_t sa = IO_SOCKADDR_INIT;
 
 	if (!io_gethostbyname(csHost, Port, &sa))
 		return NULL;
 	if (!Port)
 		Port = RPC_DEFPORT;
+	if (!proto)
+		proto = SOCK_STREAM;
 	if (netBuf < RPC_MIN_BUFSIZ)
 		netBuf = BUFSIZ;
 	else
-		netBuf = io_align(netBuf, 1);	/* align netBuf length */
+		netBuf = io_align(netBuf, 2);	/* align netBuf length */
 
 #ifdef HAVE_SRANDOMDEV
 	srandomdev();
@@ -164,7 +182,7 @@ rpc_cli_openClient(u_int ProgID, u_char ProcID, int ne
 	srandom((time(&tim) ^ getpid()));
 #endif
 
-	cli = malloc(sizeof(rpc_cli_t));
+	cli = io_malloc(sizeof(rpc_cli_t));
 	if (!cli) {
 		LOGERR;
 		return NULL;
@@ -172,10 +190,10 @@ rpc_cli_openClient(u_int ProgID, u_char ProcID, int ne
 		memset(cli, 0, sizeof(rpc_cli_t));
 
 	/* build session */
-	cli->cli_parent = malloc(sizeof(rpc_sess_t));
+	cli->cli_parent = io_malloc(sizeof(rpc_sess_t));
 	if (!cli->cli_parent) {
 		LOGERR;
-		free(cli);
+		io_free(cli);
 		return NULL;
 	} else {
 		((rpc_sess_t*) cli->cli_parent)->sess_version = RPC_VERSION;
@@ -183,29 +201,41 @@ rpc_cli_openClient(u_int ProgID, u_char ProcID, int ne
 		((rpc_sess_t*) cli->cli_parent)->sess_process = ProcID;
 	}
 
+	cli->cli_id = proto;
 	memcpy(&cli->cli_sa, &sa, sizeof cli->cli_sa);
 	AIT_SET_BUF2(&cli->cli_buf, 0, netBuf);
 
 	/* connect to RPC server */
-	cli->cli_sock = socket(cli->cli_sa.sa.sa_family, SOCK_STREAM, 0);
+	cli->cli_sock = socket(cli->cli_sa.sa.sa_family, cli->cli_id, 0);
 	if (cli->cli_sock == -1) {
 		LOGERR;
-		AIT_FREE_VAL(&cli->cli_buf);
-		free(cli->cli_parent);
-		free(cli);
-		return NULL;
+		goto err;
 	}
-	if (connect(cli->cli_sock, &cli->cli_sa.sa, cli->cli_sa.sa.sa_len) == -1) {
+	if (setsockopt(cli->cli_sock, SOL_SOCKET, SO_SNDBUF, 
+				&netBuf, sizeof netBuf) == -1) {
 		LOGERR;
-		AIT_FREE_VAL(&cli->cli_buf);
-		close(cli->cli_sock);
-		free(cli->cli_parent);
-		free(cli);
-		return NULL;
-	} else
-		fcntl(cli->cli_sock, F_SETFL, fcntl(cli->cli_sock, F_GETFL) | O_NONBLOCK);
+		goto err;
+	}
+	if (setsockopt(cli->cli_sock, SOL_SOCKET, SO_RCVBUF, 
+				&netBuf, sizeof netBuf) == -1) {
+		LOGERR;
+		goto err;
+	}
+	if (cli->cli_id == SOCK_STREAM)
+		if (connect(cli->cli_sock, &cli->cli_sa.sa, cli->cli_sa.sa.sa_len) == -1) {
+			LOGERR;
+			goto err;
+		}
 
+	fcntl(cli->cli_sock, F_SETFL, fcntl(cli->cli_sock, F_GETFL) | O_NONBLOCK);
 	return cli;
+err:
+	AIT_FREE_VAL(&cli->cli_buf);
+	if (cli->cli_sock > 2)
+		close(cli->cli_sock);
+	io_free(cli->cli_parent);
+	io_free(cli);
+	return NULL;
 }
 
 /*
@@ -220,15 +250,16 @@ rpc_cli_closeClient(rpc_cli_t ** __restrict cli)
 	if (!cli || !*cli)
 		return;
 
-	shutdown((*cli)->cli_sock, SHUT_RDWR);
+	if ((*cli)->cli_id == SOCK_STREAM)
+		shutdown((*cli)->cli_sock, SHUT_RDWR);
 	close((*cli)->cli_sock);
 
 	AIT_FREE_VAL(&(*cli)->cli_buf);
 
 	if ((*cli)->cli_parent)
-		free((*cli)->cli_parent);
+		io_free((*cli)->cli_parent);
 
-	free(*cli);
+	io_free(*cli);
 	*cli = NULL;
 }
 
@@ -252,6 +283,8 @@ rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 	uint16_t crc;
 	u_char *buf;
 	struct pollfd pfd;
+	io_sockaddr_t sa;
+	socklen_t salen;
 
 	if (!cli) {
 		rpc_SetErr(EINVAL, "Can`t execute call because parameter is null or invalid!");
@@ -261,6 +294,8 @@ rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 	if (out_vars)
 		*out_vars = NULL;
 
+	memset(&sa, 0, sizeof sa);
+
 	/* prepare RPC call */
 	rpc = (struct tagRPCCall*) buf;
 	rpc_addPktSession(&rpc->call_session, cli->cli_parent);
@@ -299,7 +334,12 @@ rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 		return -1;
 	}
 	do {
-		if ((ret = send(cli->cli_sock, buf, wlen, MSG_NOSIGNAL)) == -1) {
+		if (cli->cli_id == SOCK_STREAM)
+			ret = send(cli->cli_sock, buf, wlen, MSG_NOSIGNAL);
+		else
+			ret = sendto(cli->cli_sock, buf, wlen, MSG_NOSIGNAL, 
+					&cli->cli_sa.sa, cli->cli_sa.sa.sa_len);
+		if (ret == -1) {
 			if (errno == EAGAIN)
 				continue;
 			LOGERR;
@@ -320,20 +360,25 @@ rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 	do {
 		if ((ret = poll(&pfd, 1, DEF_RPC_TIMEOUT * 1000)) < 1 || 
 				pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
-			if (ret)
+			if (ret) {
 				LOGERR;
-			else {
+			} else {
 				if (wlen++ < 7)
 					continue;
 				else
 					rpc_SetErr(ETIMEDOUT, "Timeout, no answer from RPC server");
 			}
+
 			return -1;
 		}
-	} while (0);
-	do {
+
 		memset(buf, 0, AIT_LEN(&cli->cli_buf));
-		if ((ret = recv(cli->cli_sock, buf, AIT_LEN(&cli->cli_buf), 0)) < 1) {
+		if (cli->cli_id == SOCK_STREAM)
+			ret = recv(cli->cli_sock, buf, AIT_LEN(&cli->cli_buf), 0);
+		else
+			ret = recvfrom(cli->cli_sock, buf, AIT_LEN(&cli->cli_buf), 0, 
+					&sa.sa, &salen);
+		if (ret < 1) {
 			if (ret) {
 				if (errno == EAGAIN)
 					continue;
@@ -342,6 +387,14 @@ rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 			}
 			return -1;
 		}
+
+		if (cli->cli_id == SOCK_DGRAM) {
+			/* check for response from known address */
+			if (io_addrcmp(&cli->cli_sa, &sa, 42)) {
+				rpc_SetErr(ERPCMISMATCH, "Received RPC response from unknown address");
+				continue;
+			}
+		}
 	} while (0);
 	if (ret < sizeof(struct tagRPCCall)) {
 		rpc_SetErr(ERPCMISMATCH, "Short RPC packet %d bytes", ret);
@@ -379,8 +432,13 @@ rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 
 	/* RPC is OK! Go de-marshaling variables ... */
 	if (out_vars && ntohs(rpc->call_argc)) {
+#ifdef CLI_RES_ZCOPY
 		*out_vars = io_buffer2vars(buf + wlen, AIT_LEN(&cli->cli_buf) - wlen, 
+				ntohs(rpc->call_argc), 42);
+#else
+		*out_vars = io_buffer2vars(buf + wlen, AIT_LEN(&cli->cli_buf) - wlen, 
 				ntohs(rpc->call_argc), 0);
+#endif
 		if (!*out_vars) {
 			rpc_SetErr(io_GetErrno(), "%s", io_GetError());
 			return -1;
@@ -392,6 +450,22 @@ rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 }
 
 /*
+ * rpc_cli_freeCall() - Free resouce allocated by RPC call
+ *
+ * @out_vars = Returned array with variables from RPC call
+ * return: none
+ */
+inline void
+rpc_cli_freeCall(array_t ** __restrict out_vars)
+{
+#ifdef CLI_RES_ZCOPY
+	io_arrayDestroy(out_vars);
+#else
+	io_freeVars(out_vars);
+#endif
+}
+
+/*
  * rpc_cli_ping() - Ping RPC server
  *
  * @cli = connected client
@@ -410,7 +484,7 @@ rpc_cli_ping(rpc_cli_t *cli)
 		return -1;
 	else
 		ret = AIT_GET_U16(io_array(arr, 0, ait_val_t*));
-	io_freeVars(&arr);
+	rpc_cli_freeCall(&arr);
 
 	return ret;
 }