--- libaitrpc/src/aitrpc.c 2013/11/14 22:29:31 1.12.4.1 +++ libaitrpc/src/aitrpc.c 2013/11/15 09:07:42 1.12.4.5 @@ -3,7 +3,7 @@ * by Michael Pounov * * $Author: misho $ -* $Id: aitrpc.c,v 1.12.4.1 2013/11/14 22:29:31 misho Exp $ +* $Id: aitrpc.c,v 1.12.4.5 2013/11/15 09:07:42 misho Exp $ * ************************************************************************** The ELWIX and AITNET software is distributed under the following @@ -166,39 +166,31 @@ rpc_Read(int sock, int type, int flags, sockaddr_t * _ memset(&sa2, 0, sizeof sa2); salen = sa2.ss.ss_len = sizeof(sockaddr_t); - if (type == SOCK_STREAM) { - for (len = blen; len > 0; len -= ret, buf += ret) { - if ((ret = poll(&pfd, 1, DEF_RPC_TIMEOUT * 1000)) < 1 || - pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) { + for (len = blen; len > 0; len -= ret, buf += ret) { + if ((ret = poll(&pfd, 1, DEF_RPC_TIMEOUT * 1000)) < 1 || + pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) { + if (ret) LOGERR; - return -1; - } else - ret = recv(sock, buf, len, flags); - if (ret == -1) { - LOGERR; - return -1; - } + else + rpc_SetErr(ETIMEDOUT, "Timeout reached! Server not respond"); + return -1; } - if (ret > -1) - ret = blen - len; - } else { - do { - if ((ret = poll(&pfd, 1, DEF_RPC_TIMEOUT * 1000)) < 1 || - pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) { - LOGERR; + if (type == SOCK_STREAM) + ret = recv(sock, buf, len, flags); + else { + ret = recvfrom(sock, buf, len, flags, &sa2.sa, &salen); + if (ret > -1 && sa && e_addrcmp(sa, &sa2, 42)) { + rpc_SetErr(ERPCMISMATCH, + "Received RPC response from unknown address"); return -1; - } else - ret = recvfrom(sock, buf, blen, flags, &sa2.sa, &salen); - if (ret < 0) { - LOGERR; - return -1; } - if (e_addrcmp(sa, &sa2, 42)) - rpc_SetErr(ERPCMISMATCH, "Received RPC response from unknown address"); - else - break; - } while (42); + } + if (ret == -1) { + LOGERR; + return -1; + } } + ret = blen - len; return ret; }