Verify the seriliazed parameters parsing logic.

Fu Juntang

2021-09-03 4e122537c7c45290272c35721e38c95d9c8ae218

Verify the seriliazed parameters parsing logic.

 CMakeLists.txt

@@ -21,7 +21,7 @@


list(APPEND EXTRA_INCLUDES "${PROJECT_SOURCE_DIR}/include/usgcommon")
list(APPEND EXTRA_LIBS ${PROJECT_SOURCE_DIR}/lib/libusgcommon.a  pthread rt protobuf-lite.a)
list(APPEND EXTRA_LIBS ${PROJECT_SOURCE_DIR}/lib/libusgcommon.a  pthread rt protobuf protobuf-lite.a)

# build api doc
if (BUILD_DOC)

 src/bh_api.cc

File was deleted

 src/bh_api.cpp

@@ -7,6 +7,9 @@
#include "bh_api.h"
#include <pthread.h>
#include <getopt.h>
#include "bhome_msg_api.pb.h"
#include "bhome_msg.pb.h"
#include "error_msg.pb.h"
#include "../proto/source/bhome_msg.pb.h"
#include "../proto/source/bhome_msg_api.pb.h"

@@ -338,7 +341,7 @@
    topics_buf = (char *)malloc(total);
    if (topics_buf == NULL) {
      
      rv = EBUS_RES_NO;
      rv = EBUS_NO_MEM;
      memset(errString, 0x00, sizeof(errString));
      strncpy(errString, bus_strerror(rv), sizeof(errString));
      
@@ -417,7 +420,7 @@
  char topics_buf[MAX_STR_LEN] = { 0x00 };
  ProcInfo_query *ptr = NULL;
  ProcInfo *Proc_ptr = NULL;
  

#if defined(PRO_DE_SERIALIZE)
  struct _BHAddress
    {
@@ -517,9 +520,9 @@
    
    for(int i = 0; i < mtr_list_num; i++) {
      mtr_list[i].proc_id = ptr->procData.proc_id;
      mtr_list[i].mq_id = 0x00;
      mtr_list[i].abs_addr = 0x00;
      mtr_list[i].ip = "192.168.1.1";
      mtr_list[i].mq_id = ID_RSV;
      mtr_list[i].abs_addr = ABS_ID_RSV;
      mtr_list[i].ip = "127.0.0.1";
      mtr_list[i].port = 5000;
    }
  }
@@ -698,7 +701,7 @@
    
    *reply_len = mpr.ByteSizeLong();
    *reply = malloc(*reply_len);
    mpr.SerializePartialToArray(*reply,*reply_len);
    mpr.SerializePartialToArray(*reply, *reply_len);
  }
#else 
  if (rv == 0) {
@@ -787,7 +790,7 @@
    topics_buf = (char *)malloc(total);
    if (topics_buf == NULL) {
      
      rv = EBUS_RES_NO;
      rv = EBUS_NO_MEM;
      memset(errString, 0x00, sizeof(errString));
      strncpy(errString, bus_strerror(rv), sizeof(errString));
      
@@ -930,7 +933,7 @@
}

#if defined(PRO_DE_SERIALIZE)
int BHPublish(const char *msgpub, const int msgpub_len, const int timeout_ms)
int BHPublish(const void *msgpub, const int msgpub_len, const int timeout_ms)
#else
int BHPublish(const char *topic, const char *content, const int timeout_ms)
#endif
@@ -1145,7 +1148,8 @@
  int sec, nsec;
  std::string MsgID;
  int timeout_ms = 3000;
  char topics_buf[MAX_STR_LEN * MAX_TOPICS_NUN] = { 0x00 };
  char buf_temp[MAX_STR_LEN] = { 0x00 };
  char *topics_buf = NULL;
  
#if defined(PRO_DE_SERIALIZE)
  struct _BHAddress
@@ -1204,12 +1208,12 @@
  rv = pthread_mutex_trylock(&mutex);
  if (rv == 0) {
#if defined(PRO_DE_SERIALIZE)
    strncpy(topics_buf, _input1.topic, (sizeof(topics_buf) - 1) > strlen(_input1.topic) ? strlen(_input1.topic) : (sizeof(topics_buf) - 1));
    strncpy(buf_temp, _input1.topic, (sizeof(buf_temp) - 1) > strlen(_input1.topic) ? strlen(_input1.topic) : (sizeof(buf_temp) - 1));
#else 
    strncpy(topics_buf, (char *)request, (sizeof(topics_buf) - 1) > strlen((char *)request) ? strlen((char *)request) : (sizeof(topics_buf) - 1));
    strncpy(buf_temp, (char *)request, (sizeof(buf_temp) - 1) > strlen((char *)request) ? strlen((char *)request) : (sizeof(buf_temp) - 1));
#endif 
    
    rv = net_mod_socket_reg(gNetmod_socket, topics_buf, strlen(topics_buf), &buf, &size, timeout_ms, PROC_QUE_STCS);
    rv = net_mod_socket_reg(gNetmod_socket, buf_temp, strlen(buf_temp), &buf, &size, timeout_ms, PROC_QUE_STCS);
    if (rv == 0) {

      val = atoi((char *)buf);
@@ -1218,12 +1222,31 @@

      if (val > 0) {

        len = strlen(topics_buf);
        len = strlen(buf_temp) + 1;
#if defined(PRO_DE_SERIALIZE)
        min = (sizeof(topics_buf) - 1 - len ) > strlen(_input1.data) ? strlen(_input1.data) : (sizeof(topics_buf) - 1 - len );
        strncpy(topics_buf + len + 1, _input1.data, min);
        len += (min + 1);
        len += strlen(_input1.data);
#endif

        topics_buf = (char *)malloc(len);
        if (topics_buf == NULL) {
          
          rv = EBUS_NO_MEM;
          memset(errString, 0x00, sizeof(errString));
          strncpy(errString, bus_strerror(rv), sizeof(errString));
          
          logger->error("in BHRequest: Out of memory!\n");
          
          pthread_mutex_unlock(&mutex);

          return false;
        }
        memset(topics_buf, 0x00, len);
        
        strncpy(topics_buf, buf_temp, strlen(buf_temp) + 1);
#if defined(PRO_DE_SERIALIZE)
        strncpy(topics_buf + strlen(buf_temp) + 1, _input1.data, strlen(_input1.data));
#endif 

        if (timeout_ms > 0) {

          sec = timeout_ms / 1000;
@@ -1238,7 +1261,10 @@
        } else {

          rv = net_mod_socket_sendto(gNetmod_socket, topics_buf, len, val); 
        }            
        } 

        free(topics_buf);
        
      } else {

        rv = EBUS_RES_UNSUPPORT; 
@@ -1294,8 +1320,8 @@
  net_mod_recv_msg_t *recv_arr;
  net_mod_err_t *errarr;
  int errarr_size = 0;
  int sec, nsec;
  char topics_buf[MAX_STR_LEN] = { 0x00 };
  char buf_temp[MAX_STR_LEN] = { 0x00 };
  char *topics_buf = NULL;
  
  struct _RequestReply
  {
@@ -1360,13 +1386,14 @@
  rv = pthread_mutex_trylock(&mutex);
  if (rv == 0) {
#if defined(PRO_DE_SERIALIZE)
    strncpy(topics_buf, _input1.topic, (sizeof(topics_buf) - 1) > strlen(_input1.topic) ? strlen(_input1.topic)  : (sizeof(topics_buf) - 1));
    strncpy(buf_temp, _input1.topic, (sizeof(buf_temp) - 1) > strlen(_input1.topic) ? strlen(_input1.topic)  : (sizeof(buf_temp) - 1));
#else 
    strncpy(topics_buf, (char *)request, (sizeof(topics_buf) - 1) > request_len ? request_len : (sizeof(topics_buf) - 1));
    strncpy(buf_temp, (char *)request, (sizeof(buf_temp) - 1) > request_len ? request_len : (sizeof(buf_temp) - 1));
#endif 
    
    rv = net_mod_socket_reg(gNetmod_socket, topics_buf, strlen(topics_buf), &buf, &size, timeout_ms, PROC_QUE_STCS);
    rv = net_mod_socket_reg(gNetmod_socket, buf_temp, strlen(buf_temp), &buf, &size, timeout_ms, PROC_QUE_STCS);
    if (rv == 0) {
      
      val = atoi((char *)buf);

      free(buf);
@@ -1374,23 +1401,53 @@
      if (val > 0) {
        memset(&node, 0x00, sizeof(node));

        len = strlen(topics_buf);
        len = strlen(buf_temp) + 1;
#if defined(PRO_DE_SERIALIZE)
        min = (sizeof(topics_buf) - 1 - len ) > strlen(_input1.data) ? strlen(_input1.data) : (sizeof(topics_buf) - 1 - len );
        strncpy(topics_buf + len + 1, _input1.data, min);
        len += (min + 1);
        len += strlen(_input1.data);
#endif

        topics_buf = (char *)malloc(len);
        if (topics_buf == NULL) {
          
          rv = EBUS_NO_MEM;
          memset(errString, 0x00, sizeof(errString));
          strncpy(errString, bus_strerror(rv), sizeof(errString));
          
          logger->error("in BHRequest: Out of memory!\n");
          
          pthread_mutex_unlock(&mutex);

          return false;
        }
        memset(topics_buf, 0x00, len);
        
        strncpy(topics_buf, buf_temp, strlen(buf_temp) + 1);
#if defined(PRO_DE_SERIALIZE)
        strncpy(topics_buf + strlen(buf_temp) + 1, _input1.data, strlen(_input1.data));
#endif 

        node.key = val;
        rv = net_mod_socket_sendandrecv(gNetmod_socket, &node, 1, topics_buf, len, &recv_arr, &recv_arr_size, &errarr, &errarr_size);
        
        if (timeout_ms > 0) {
          
          rv = net_mod_socket_sendandrecv_timeout(gNetmod_socket, &node, 1, topics_buf, len, &recv_arr, &recv_arr_size, &errarr, &errarr_size, timeout_ms);

        } else if (timeout_ms == 0) {

          rv = net_mod_socket_sendandrecv_nowait(gNetmod_socket, &node, 1, topics_buf, len, &recv_arr, &recv_arr_size, &errarr, &errarr_size);
        
        } else {

          rv = net_mod_socket_sendandrecv(gNetmod_socket, &node, 1, topics_buf, len, &recv_arr, &recv_arr_size, &errarr, &errarr_size);
        }
        if (rv > 0) {
          if (recv_arr_size > 0) {

            node.key = recv_arr[0].key;
            
            memset(topics_buf, 0x00, sizeof(topics_buf));

            size = recv_arr[0].content_length;
            buf = (char *)malloc(size);
            memset(buf, 0x00, size);
            strncpy((char *)buf, (char *)recv_arr[0].content, size);
#if !defined(PRO_DE_SERIALIZE)
            *reply = buf;
@@ -1409,6 +1466,8 @@
        } else {
          rv = EBUS_TIMEOUT; 
        }
        
        free(topics_buf);

      } else {
        rv = EBUS_RES_UNSUPPORT; 
@@ -1419,17 +1478,17 @@
    strncpy(errString, bus_strerror(rv), sizeof(errString));

    if (rv == 0) {
      memset(topics_buf, 0x00, sizeof(topics_buf));
      sprintf(topics_buf, "%d", node.key);
      memset(buf_temp, 0x00, sizeof(buf_temp));
      sprintf(buf_temp, "%d", node.key);

      rr.proc_id = topics_buf;
      rr.proc_id = buf_temp;
      *proc_id_len = rr.proc_id.size();
      *proc_id = malloc(*proc_id_len);
      memcpy(*proc_id, rr.proc_id.data(), *proc_id_len);

      memset(topics_buf, 0x00, sizeof(topics_buf));
      memcpy(topics_buf, buf, size);
      rr.data = topics_buf;
      memset(buf_temp, 0x00, sizeof(buf_temp));
      memcpy(buf_temp, buf, size);
      rr.data = buf_temp;
    }
    
    pthread_mutex_unlock(&mutex);
@@ -1473,8 +1532,10 @@
  void *buf;
  int key;
  int size;
  int len;
  int sec, nsec;
  char topics_buf[MAX_STR_LEN] = { 0x00 };
  char buf_temp[MAX_STR_LEN] = { 0x00 };
  char *topics_buf = NULL;
  
  if (gRun_stat == 0) {
    logger->error("the process has not been registered yet!\n");
@@ -1513,17 +1574,42 @@
        void *src;
      } rrr;

      sprintf(topics_buf, "%d", key);
      rrr.proc_id = topics_buf;
      sprintf(buf_temp, "%d", key);
      rrr.proc_id = buf_temp;

      *proc_id_len = rrr.proc_id.size();
      *proc_id = malloc(*proc_id_len);
      memcpy(*proc_id, rrr.proc_id.data(), *proc_id_len);

      memset(topics_buf, 0x00, sizeof(topics_buf));
      memcpy(topics_buf, buf, size > sizeof(topics_buf) ? sizeof(topics_buf) : size);
      topics_buf = (char *)malloc(size + MIN_STR_LEN);
      if (topics_buf == NULL) {
        
        rv = EBUS_NO_MEM;
        memset(errString, 0x00, sizeof(errString));
        strncpy(errString, bus_strerror(rv), sizeof(errString));
        
        logger->error("in BHReadRequest: Out of memory!\n");
        
        pthread_mutex_unlock(&mutex);

        return false;
      }
      memset(topics_buf, 0x00, size + MIN_STR_LEN);
      
      len = strlen((char *)buf);
      if (len > size) {
        len = size;
      }
      strncpy(topics_buf, (char *)buf, len);
      rrr.topic = topics_buf;
      rrr.data = topics_buf;
      
      if (len < size) {
        strncpy(topics_buf + len + 1, (char *)buf + len + 1, size - len - 1);
      }

      rrr.data = topics_buf + len + 1;
      
      free(topics_buf);

#if defined(PRO_DE_SERIALIZE)
      ::bhome_msg::MsgRequestTopic mrt;
@@ -1536,6 +1622,8 @@
      *request = buf;
      *request_len = size;
#endif 

      free(buf);

      buf = malloc(sizeof(int));
      *(int *)buf = key;
@@ -1563,7 +1651,8 @@
int BHSendReply(void *src, const void *reply, const int reply_len)
{
  int rv;

  const char *_input;
  
#if defined(PRO_DE_SERIALIZE)
  ::bhome_msg::MsgRequestTopicReply input;
  if (!input.ParseFromArray(reply, reply_len)) {
@@ -1575,9 +1664,8 @@
    return false;
  }
  
  const char *_input;
  _input = input.data().data();
  

#else 
  if ((src == NULL) || (reply == NULL) || (reply_len == 0)) {

@@ -1587,6 +1675,9 @@
    
    return false;
  }
  
  _input = (char *)reply;
  
#endif 

  if (gRun_stat == 0) {
@@ -1602,7 +1693,7 @@
  rv = pthread_mutex_trylock(&mutex);
  if (rv == 0) {

    rv = net_mod_socket_sendto(gNetmod_socket, reply, reply_len, *(int *)src);
    rv = net_mod_socket_sendto(gNetmod_socket, _input, strlen(_input), *(int *)src);

    memset(errString, 0x00, sizeof(errString));
    strncpy(errString, bus_strerror(rv), sizeof(errString));

 src/bus_error.cpp

@@ -24,7 +24,8 @@
  "Service not supported",
  "Resource busy",
  "Resource not provide",
  "Invalid parameters"
  "Invalid parameters",
  "No enough memory"

};


 src/bus_error.h

@@ -17,6 +17,7 @@
#define EBUS_RES_BUSY  510
#define EBUS_RES_NO  511
#define EBUS_INVALID_PARA  512
#define EBUS_NO_MEM       513

extern int bus_errno;


 src/proc_def.h

@@ -8,7 +8,7 @@
#define MAX_STR_LEN     128 //keep the same with serializer in proc check
#define MIN_STR_LEN      10
#define MAX_PROC_NUM    128
#define MAX_TOPICS_NUN  60
#define MAX_TOPICS_NUN   60

#define PROC_REG        1
#define PROC_UNREG      2
@@ -16,6 +16,9 @@
#define PROC_QUE_TCS    4
#define PROC_QUE_STCS   5
#define PROC_QUE_ATCS   6

#define ID_RSV          16
#define ABS_ID_RSV      18

#define STR_MAGIC       ","

@@ -40,9 +43,9 @@
  ProcInfo procData;

  int stat;
  char reg_info[MAX_STR_LEN];
  char local_info[MAX_STR_LEN];
  char net_info[MAX_STR_LEN];
  char reg_info[MAX_STR_LEN * MAX_TOPICS_NUN];
  char local_info[MAX_STR_LEN * MAX_TOPICS_NUN];
  char net_info[MAX_STR_LEN * MAX_TOPICS_NUN];

  int list_num;


 src/socket/bus_server_socket.cpp

@@ -676,7 +676,6 @@
  int flag;
    char * action, *topic, *topics, *buf, *content;
    size_t head_len;
    char resp_buf[128];
    bus_head_t head;

  int rv;