Diff for /libaitrpc/src/cli.c between versions 1.24 and 1.24.2.4

version 1.24, 2015/01/21 21:17:05 version 1.24.2.4, 2015/05/18 15:00:54
Line 161  rpc_cli_openClient(u_char InstID, int netBuf, const ch Line 161  rpc_cli_openClient(u_char InstID, int netBuf, const ch
         rpc_cli_t *cli = NULL;          rpc_cli_t *cli = NULL;
         sockaddr_t sa = E_SOCKADDR_INIT;          sockaddr_t sa = E_SOCKADDR_INIT;
   
        if (!e_gethostbyname(csHost, Port, &sa))        if (proto < 0 || proto > SOCK_RAW) {
                 rpc_SetErr(EINVAL, "Invalid parameters can`t open RPC client");
                 return NULL;                  return NULL;
        if (!Port)        }
 
         srandom(time(NULL) ^ getpid());
 
         if (!Port && proto < SOCK_RAW)
                 Port = RPC_DEFPORT;                  Port = RPC_DEFPORT;
           if (!e_gethostbyname(csHost, Port, &sa))
                   return NULL;
         if (!proto)          if (!proto)
                 proto = SOCK_STREAM;                  proto = SOCK_STREAM;
         if (netBuf < RPC_MIN_BUFSIZ)          if (netBuf < RPC_MIN_BUFSIZ)
Line 195  rpc_cli_openClient(u_char InstID, int netBuf, const ch Line 202  rpc_cli_openClient(u_char InstID, int netBuf, const ch
         AIT_SET_BUFSIZ(&cli->cli_buf, 0, netBuf);          AIT_SET_BUFSIZ(&cli->cli_buf, 0, netBuf);
   
         /* connect to RPC server */          /* connect to RPC server */
        cli->cli_sock = socket(cli->cli_sa.sa.sa_family, cli->cli_id, 0);        cli->cli_sock = socket(cli->cli_sa.sa.sa_family, cli->cli_id, 
                         cli->cli_id == SOCK_RAW ? IPPROTO_ERPC : 0);
         if (cli->cli_sock == -1) {          if (cli->cli_sock == -1) {
                 LOGERR;                  LOGERR;
                 goto err;                  goto err;
Line 245  rpc_cli_reconnectClient(rpc_cli_t * __restrict cli) Line 253  rpc_cli_reconnectClient(rpc_cli_t * __restrict cli)
   
         close(cli->cli_sock);          close(cli->cli_sock);
   
           srandom(time(NULL) ^ getpid());
   
         cli->cli_sock = socket(cli->cli_sa.sa.sa_family, cli->cli_id, 0);          cli->cli_sock = socket(cli->cli_sa.sa.sa_family, cli->cli_id, 0);
         if (cli->cli_sock == -1) {          if (cli->cli_sock == -1) {
                 LOGERR;                  LOGERR;
Line 311  rpc_cli_closeClient(rpc_cli_t ** __restrict cli) Line 321  rpc_cli_closeClient(rpc_cli_t ** __restrict cli)
 int  int
 rpc_pkt_Send(int sock, int type, sockaddr_t * __restrict sa, ait_val_t * __restrict pkt, int len)  rpc_pkt_Send(int sock, int type, sockaddr_t * __restrict sa, ait_val_t * __restrict pkt, int len)
 {  {
         u_char *buf;  
   
         if (!pkt) {          if (!pkt) {
                 rpc_SetErr(EINVAL, "Invalid argument(s)!");                  rpc_SetErr(EINVAL, "Invalid argument(s)!");
                 return -1;                  return -1;
        } else        }
                buf = AIT_GET_BUF(pkt); 
   
        return rpc_Write(sock, type, MSG_NOSIGNAL, sa, buf, len);        return rpc_Write(sock, type, MSG_NOSIGNAL, sa, pkt, len);
 }  }
   
 /*  /*
Line 329  rpc_pkt_Send(int sock, int type, sockaddr_t * __restri Line 336  rpc_pkt_Send(int sock, int type, sockaddr_t * __restri
  * @type = Type of socket   * @type = Type of socket
  * @sa = Server address   * @sa = Server address
  * @pkt = RPC packet   * @pkt = RPC packet
    * @seq = Signed packet with seq.no
  * return: -1 error, 0 EOF or >0 received bytes   * return: -1 error, 0 EOF or >0 received bytes
  */   */
 int  int
rpc_pkt_Receive(int sock, int type, sockaddr_t * __restrict sa, ait_val_t * __restrict pkt)rpc_pkt_Receive(int sock, int type, sockaddr_t * __restrict sa, ait_val_t * __restrict pkt, int seq)
 {  {
        int ret, estlen = 0, blen = sizeof(struct tagRPCCall);        int ret;
        u_char *buf; 
         struct tagRPCCall *rpc;          struct tagRPCCall *rpc;
   
         if (!pkt) {          if (!pkt) {
                 rpc_SetErr(EINVAL, "Invalid argument(s)!");                  rpc_SetErr(EINVAL, "Invalid argument(s)!");
                 return -1;                  return -1;
         } else {  
                 buf = AIT_GET_BUF(pkt);  
                 rpc = (struct tagRPCCall*) buf;  
         }          }
   
         /* reply from RPC server */          /* reply from RPC server */
         do {          do {
                if (type == SOCK_STREAM)                if (type == SOCK_STREAM || type == SOCK_EXT)
                        ret = rpc_Read(sock, type, !estlen ? MSG_PEEK : 0, NULL, buf, blen);                        ret = rpc_Read(sock, type, 0, NULL, pkt);
                else if (type == SOCK_EXT) {                else
                        ret = rpc_Read(sock, type, 0, NULL, buf, blen);                        ret = rpc_Read(sock, type, 0, sa, pkt);
                        if (estlen)     /* hack for skip sanity checks */ 
                                ret += sizeof(struct tagRPCCall); 
                } else if (type == SOCK_BPF) { 
                        ret = rpc_Read(sock, type, 0, sa, AIT_GET_BUF(pkt), AIT_LEN(pkt)); 
                        if (ret > 0) 
                                estlen = ret; 
                 } else 
                        ret = rpc_Read(sock, type, !estlen ? MSG_PEEK : 0, sa, buf, blen); 
                if (ret < 1) 
                        return ret; 
   
                /* 1st read for RPC header */                if (ret > 0) {
                if (ret < sizeof(struct tagRPCCall)) {                        rpc = (struct tagRPCCall*) AIT_GET_BUF(pkt);
                        rpc_SetErr(ERPCMISMATCH, "Short RPC packet %d bytes", ret); 
                        return -1; 
                } 
   
                /* check for loop request */                        /* check for loop request */
                if (!estlen && !(rpc->call_io & RPC_ACK))                        if (!(rpc->call_io & RPC_ACK))
                        continue;                                continue;
                        /* check for signed request */
                /* check for response from known address */                        if (seq != ntohl(rpc->call_seq))
                if (!estlen) {                                continue;
                        /* calc estimated length */ 
                        estlen = ntohl(rpc->call_len); 
                        if (estlen > AIT_LEN(pkt)) 
                                AIT_RE_BUF(pkt, estlen); 
                        blen = estlen; 
                        buf = AIT_GET_BUF(pkt); 
 
                        if (type == SOCK_EXT) { 
                                blen -= sizeof(struct tagRPCCall); 
                                buf += sizeof(struct tagRPCCall); 
                        } 
 
                        rpc = (struct tagRPCCall*) buf; 
                        continue; 
                 }                  }
           } while (ret < 1);
   
                 /* compiler optimize loop if while(0) and stop working 'continue' on some platforms */  
                 break;  
         } while (42);  
   
         if (estlen != ret) {  
                 rpc_SetErr(ERPCMISMATCH, "RPC packet mismatch estimate %d bytes, but received %d\n",   
                                 estlen, ret);  
                 return -1;  
         }  
   
         return ret;          return ret;
 }  }
   
Line 413  rpc_pkt_Receive(int sock, int type, sockaddr_t * __res Line 381  rpc_pkt_Receive(int sock, int type, sockaddr_t * __res
  * @vars = Function argument array of values, may be NULL   * @vars = Function argument array of values, may be NULL
  * @noreply = We not want RPC reply   * @noreply = We not want RPC reply
  * @nocrc = Without CRC calculation   * @nocrc = Without CRC calculation
    * @seq = Sign packet with seq.no
  * return: -1 error or != -1 prepared bytes into packet   * return: -1 error or != -1 prepared bytes into packet
  */   */
 int  int
 rpc_pkt_Request(ait_val_t * __restrict pkt, rpc_sess_t * __restrict sess, u_short tag,   rpc_pkt_Request(ait_val_t * __restrict pkt, rpc_sess_t * __restrict sess, u_short tag, 
                array_t * __restrict vars, int noreply, int nocrc)                array_t * __restrict vars, int noreply, int nocrc, int seq)
 {  {
         struct tagRPCCall *rpc;          struct tagRPCCall *rpc;
         int ret = 0, estlen, len = sizeof(struct tagRPCCall);          int ret = 0, estlen, len = sizeof(struct tagRPCCall);
Line 460  rpc_pkt_Request(ait_val_t * __restrict pkt, rpc_sess_t Line 429  rpc_pkt_Request(ait_val_t * __restrict pkt, rpc_sess_t
         rpc->call_len = htonl(len);          rpc->call_len = htonl(len);
         rpc->call_io = RPC_REQ;          rpc->call_io = RPC_REQ;
   
           /* sign packet */
           rpc->call_seq = htonl(seq);
   
         if (!nocrc) {          if (!nocrc) {
                 /* calculate CRC */                  /* calculate CRC */
                 rpc->call_crc ^= rpc->call_crc;                  rpc->call_crc ^= rpc->call_crc;
Line 474  rpc_pkt_Request(ait_val_t * __restrict pkt, rpc_sess_t Line 446  rpc_pkt_Request(ait_val_t * __restrict pkt, rpc_sess_t
  *   *
  * @pkt = Packet buffer   * @pkt = Packet buffer
  * @sess = RPC session info, if =NULL don't check session   * @sess = RPC session info, if =NULL don't check session
 * @tag = Function tag * @tag = Function tag, if =CALL_TAG_MAX don't check tag
  * @vars = Function argument array of values, may be NULL   * @vars = Function argument array of values, may be NULL
  * @nocrc = Without CRC calculation   * @nocrc = Without CRC calculation
  * return: -1 error or != -1 return value from function   * return: -1 error or != -1 return value from function
Line 510  rpc_pkt_Replay(ait_val_t * __restrict pkt, rpc_sess_t  Line 482  rpc_pkt_Replay(ait_val_t * __restrict pkt, rpc_sess_t 
                 rpc_SetErr(ERPCMISMATCH, "Get invalid RPC session");                  rpc_SetErr(ERPCMISMATCH, "Get invalid RPC session");
                 return -1;                  return -1;
         }          }
        if (ntohs(rpc->call_tag) != tag) {        if (tag != CALL_TAG_MAX && ntohs(rpc->call_tag) != tag) {
                 rpc_SetErr(ERPCMISMATCH, "Get wrong RPC reply");                  rpc_SetErr(ERPCMISMATCH, "Get wrong RPC reply");
                 return -1;                  return -1;
         }          }
Line 564  rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short  Line 536  rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
 {  {
         int type = 0, wlen;          int type = 0, wlen;
         u_char *buf;          u_char *buf;
           u_int seq = 0;
   
         if (!cli) {          if (!cli) {
                 rpc_SetErr(EINVAL, "Can`t execute call because parameter is null or invalid!");                  rpc_SetErr(EINVAL, "Can`t execute call because parameter is null or invalid!");
Line 571  rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short  Line 544  rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
         } else {          } else {
                 if (cli->cli_id == SOCK_STREAM || cli->cli_id == SOCK_EXT)                  if (cli->cli_id == SOCK_STREAM || cli->cli_id == SOCK_EXT)
                         type = cli->cli_id;                          type = cli->cli_id;
                   else
                           seq = random(); /* sign package */
                 buf = AIT_GET_BUF(&cli->cli_buf);                  buf = AIT_GET_BUF(&cli->cli_buf);
         }          }
         if (out_vars)          if (out_vars)
                 *out_vars = NULL;                  *out_vars = NULL;
   
        if ((wlen = rpc_pkt_Request(&cli->cli_buf, cli->cli_parent, tag, in_vars, noreply, type)) == -1)        if ((wlen = rpc_pkt_Request(&cli->cli_buf, cli->cli_parent, tag, in_vars, noreply, type, seq)) == -1)
                 return -1;                  return -1;
   
         if ((wlen = rpc_pkt_Send(cli->cli_sock, cli->cli_id, &cli->cli_sa, &cli->cli_buf, wlen)) == -1)          if ((wlen = rpc_pkt_Send(cli->cli_sock, cli->cli_id, &cli->cli_sa, &cli->cli_buf, wlen)) == -1)
Line 587  rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short  Line 562  rpc_cli_execCall(rpc_cli_t *cli, int noreply, u_short 
         if (noreply)    /* we not want reply */          if (noreply)    /* we not want reply */
                 return 0;                  return 0;
   
        if ((wlen = rpc_pkt_Receive(cli->cli_sock, cli->cli_id, &cli->cli_sa, &cli->cli_buf)) == -1)        if ((wlen = rpc_pkt_Receive(cli->cli_sock, cli->cli_id, &cli->cli_sa, &cli->cli_buf, seq)) == -1)
                 return -1;                  return -1;
         if (!wlen)      /* closed rpc connection */          if (!wlen)      /* closed rpc connection */
                 return 1;                  return 1;

Removed from v.1.24  
changed lines
  Added in v.1.24.2.4


FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>