From 52ce9893a71311c46d4e9ba14ef2a98b3b715e36 Mon Sep 17 00:00:00 2001 From: fujuntang <fujuntang@smartai.com> Date: 星期日, 26 九月 2021 21:57:24 +0800 Subject: [PATCH] trival version: need confirm whether protobuf can run weil yet. --- src/bh_api.cpp | 323 +++++++++++++++++++++++++++-------------------------- 1 files changed, 164 insertions(+), 159 deletions(-) diff --git a/src/bh_api.cpp b/src/bh_api.cpp index 73c7772..f4215d3 100644 --- a/src/bh_api.cpp +++ b/src/bh_api.cpp @@ -7,11 +7,11 @@ #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/error_msg.pb.h" #include "../proto/source/bhome_msg.pb.h" #include "../proto/source/bhome_msg_api.pb.h" + +#define TIME_WAIT 3 static Logger *logger = LoggerFactory::getLogger(); @@ -20,7 +20,43 @@ static pthread_mutex_t mutex; -static char errString[100] = { 0x00 }; +static pthread_t gTids; + +static void *client_run_check(void *skptr) { + + pthread_detach(pthread_self()); + + int data; + int sec, nsec; + int rv; + int key; + char buf[MAX_STR_LEN] = { 0x00 }; + void *buf_temp = NULL; + int size; + + sec = TIME_WAIT; + nsec = 0; + sprintf(buf, "%s", "Success"); + data = net_mod_socket_int_get(gNetmod_socket); + while(true) { + + rv = net_mod_socket_recvfrom(gNetmod_socket, &buf_temp, &size, &key, SVR_STR, data); + if (rv == 0) { + + BHFree(buf_temp, size); + + rv = net_mod_socket_sendto_timeout(gNetmod_socket, buf, strlen(buf), key, sec, nsec, SVR_STR, data); + if (rv != 0) { + logger->error("the process check response failed with error: %s!\n", bus_strerror(rv)); + } + + } else { + + logger->error("the process check failed with error: %s!\n", bus_strerror(rv)); + + } + } +} int BHRegister(const void *proc_info, const int proc_info_len, void **reply, int *reply_len, const int timeout_ms) { @@ -29,6 +65,7 @@ int count = 0; void *buf = NULL; int min = 0; + char *errString = NULL; ProcInfo pData; #if defined(PRO_DE_SERIALIZE) @@ -43,10 +80,8 @@ ::bhome_msg::ProcInfo input; if ((!input.ParseFromArray(proc_info, proc_info_len)) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - + bus_errorset(rv); + return false; } @@ -58,10 +93,8 @@ #else if ((proc_info == NULL) || (proc_info_len == 0) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x90, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - + bus_errorset(rv); + return false; } #endif @@ -74,8 +107,7 @@ logger->error("the process has already registered!\n"); rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -89,52 +121,62 @@ #if defined(PRO_DE_SERIALIZE) if (_input.proc_id != NULL) { count = strlen(_input.proc_id) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : count; strncpy(pData.proc_id, _input.proc_id, min); } if (_input.name != NULL) { count = strlen(_input.name) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1)? (MAX_STR_LEN -1) : count; strncpy(pData.name, _input.name, min); } if (_input.public_info != NULL) { count = strlen(_input.public_info) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1)? (MAX_STR_LEN - 1) : count; strncpy(pData.public_info, _input.public_info, min); } if (_input.private_info != NULL) { count = strlen(_input.private_info) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1)? (MAX_STR_LEN - 1): count; strncpy(pData.private_info, _input.private_info, min); } #else if (strlen((const char *)(((ProcInfo *)proc_info)->proc_id)) > 0) { count = strlen((const char *)(((ProcInfo *)proc_info)->proc_id)) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1): count; strncpy(pData.proc_id, ((ProcInfo *)proc_info)->proc_id, min); } if (strlen((const char *)(((ProcInfo *)proc_info)->name)) > 0) { count = strlen((const char *)(((ProcInfo *)proc_info)->name)) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : count; strncpy(pData.name, ((ProcInfo *)proc_info)->name, min); } if (strlen((const char *)(((ProcInfo *)proc_info)->public_info)) > 0) { count = strlen((const char *)(((ProcInfo *)proc_info)->public_info)) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : count; strncpy(pData.public_info, ((ProcInfo *)proc_info)->public_info, min); } if (strlen((const char *)(((ProcInfo *)proc_info)->private_info)) > 0) { count = strlen((const char *)(((ProcInfo *)proc_info)->private_info)) + 1; - min = count > MAX_STR_LEN ? MAX_STR_LEN : count; + min = count > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : count; strncpy(pData.private_info, ((ProcInfo *)proc_info)->private_info, min); } #endif + + if (pData.proc_id == NULL) { + rv = EBUS_INVALID_PARA; + + bus_errorset(rv); + + pthread_mutex_unlock(&mutex); + + return false; + } gNetmod_socket = net_mod_socket_open(); hashtable_t *hashtable = mm_get_hashtable(); @@ -149,19 +191,19 @@ rv = net_mod_socket_reg(gNetmod_socket, &pData, sizeof(ProcInfo), NULL, 0, timeout_ms, PROC_REG); - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); pthread_mutex_unlock(&mutex); } else { rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } + + errString = bus_strerror(0, 1); #if defined(PRO_DE_SERIALIZE) ::bhome_msg::MsgCommonReply mcr; @@ -180,7 +222,9 @@ *reply_len = min; #endif - + + pthread_create(&gTids, NULL, client_run_check, NULL); + return true; } @@ -190,6 +234,7 @@ int rv; int min; void *buf = NULL; + char *errString = NULL; #if defined(PRO_DE_SERIALIZE) struct _ProcInfo_proto @@ -205,9 +250,7 @@ if(!input.ParseFromArray(proc_info, proc_info_len) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -219,9 +262,7 @@ #else if ((reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -232,8 +273,7 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -250,21 +290,19 @@ gRun_stat = 0; } - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); pthread_mutex_unlock(&mutex); } else { rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } + errString = bus_strerror(0, 1); #if defined(PRO_DE_SERIALIZE) ::bhome_msg::MsgCommonReply mcr; mcr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -293,6 +331,7 @@ void *buf = NULL; int total = 0; int count = 0; + char *errString = NULL; char *topics_buf = NULL; #if defined(PRO_DE_SERIALIZE) @@ -306,11 +345,9 @@ if(!input.ParseFromArray(topics, topics_len) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); - return false; + return false; } _input.amount = input.topic_list_size(); @@ -327,9 +364,7 @@ if ((topics == NULL) || (topics_len == 0) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -341,8 +376,7 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -351,8 +385,7 @@ if (topics_buf == NULL) { rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); logger->error("in BHRegisterTopics: Out of memory!\n"); @@ -382,10 +415,10 @@ rv = net_mod_socket_reg(gNetmod_socket, topics_buf, count, NULL, 0, timeout_ms, PROC_REG_TCS); - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - free(topics_buf); + + bus_errorset(rv); + errString = bus_strerror(0, 1); #if defined(PRO_DE_SERIALIZE) ::bhome_msg::MsgCommonReply mcr; @@ -414,6 +447,7 @@ int min; void *buf = NULL; int size; + char *errString = NULL; char topics_buf[MAX_STR_LEN] = { 0x00 }; ProcInfo_query *ptr = NULL; ProcInfo *Proc_ptr = NULL; @@ -433,9 +467,7 @@ ::bhome_msg::MsgQueryTopic input1; if (!input0.ParseFromArray(remote, remote_len) || !input1.ParseFromArray(topic, topic_len) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -449,9 +481,7 @@ #else if ((topic == NULL) || (topic_len == 0) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -461,8 +491,7 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -477,9 +506,7 @@ #endif rv = net_mod_socket_reg(gNetmod_socket, topics_buf, min, &buf, &size, timeout_ms, PROC_QUE_TCS); - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - + bus_errorset(rv); #if defined(PRO_DE_SERIALIZE) struct _MsgQueryTopicReply @@ -501,15 +528,18 @@ if (mtr_list_num > sizeof(mtr_list) / sizeof(mtr_list[0])) { mtr_list_num = sizeof(mtr_list) / sizeof(mtr_list[0]); } - + + Proc_ptr = &(ptr->procData); for(int i = 0; i < mtr_list_num; i++) { - mtr_list[i].proc_id = ptr->procData.proc_id; + mtr_list[i].proc_id = (Proc_ptr + i)->proc_id; 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; } } + + errString = bus_strerror(0, 1); ::bhome_msg::MsgQueryTopicReply mtr; mtr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -554,6 +584,7 @@ void *buf = NULL; int size; int min; + char *errString = NULL; ProcInfo_sum *Proc_ptr = NULL; char data_buf[MAX_STR_LEN] = { 0x00 }; @@ -573,8 +604,7 @@ if (!input0.ParseFromArray(remote, remote_len) || !input1.ParseFromArray(query, query_len) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -587,8 +617,7 @@ #else if ((reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -598,9 +627,8 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - + bus_errorset(rv); + return false; } @@ -609,9 +637,7 @@ } rv = net_mod_socket_reg(gNetmod_socket, data_buf, strlen(data_buf), &buf, &size, timeout_ms, PROC_QUE_ATCS); - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); #if defined(PRO_DE_SERIALIZE) struct _MsgQueryProcReply @@ -657,6 +683,8 @@ } } } + + errString = bus_strerror(0, 1); ::bhome_msg::MsgQueryProcReply mpr; mpr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -707,6 +735,7 @@ int count = 0; int len, i; void *buf = NULL; + char *errString = NULL; char *topics_buf = NULL; #if defined(PRO_DE_SERIALIZE) @@ -720,8 +749,7 @@ if(!input.ParseFromArray(topics, topics_len) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -741,8 +769,7 @@ #else if ((topics == NULL) || (topics_len == 0) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -752,8 +779,7 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -762,8 +788,7 @@ if (topics_buf == NULL) { rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); logger->error("in BHSubscribeTopics: Out of memory!\n"); @@ -806,8 +831,7 @@ } - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + errString = bus_strerror(0, 1); free(topics_buf); @@ -848,6 +872,7 @@ int BHHeartbeat(const void *proc_info, const int proc_info_len, void **reply, int *reply_len, const int timeout_ms) { int rv; + char *errString = NULL; #if defined(PRO_DE_SERIALIZE) struct _ProcInfo_proto @@ -862,9 +887,7 @@ if(!input.ParseFromArray(proc_info,proc_info_len)) { rv = EBUS_INVALID_PARA; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -875,8 +898,8 @@ _input.private_info = input.private_info().c_str(); rv = 0; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); + errString = bus_strerror(0, 1); ::bhome_msg::MsgCommonReply mcr; mcr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -912,8 +935,7 @@ if(!input.ParseFromArray(msgpub, msgpub_len)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -924,8 +946,7 @@ if ((topic == NULL) || (content == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -935,8 +956,7 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -966,8 +986,7 @@ if (rv > 0) return true; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -995,16 +1014,14 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } if ((msgpub == NULL) || (msgpub_len == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1035,8 +1052,7 @@ if (topics_buf == NULL) { rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); logger->error("in BHRequest: Out of memory!\n"); @@ -1052,8 +1068,7 @@ if (data_buf == NULL) { rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); logger->error("in BHRequest: Out of memory!\n"); @@ -1124,8 +1139,8 @@ } else { - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); + } if (rv == 0) @@ -1146,6 +1161,7 @@ int sec, nsec; std::string MsgID; int timeout_ms = 3000; + char data_buf[MAX_STR_LEN] = { 0x00 }; char buf_temp[MAX_STR_LEN] = { 0x00 }; char *topics_buf = NULL; @@ -1169,8 +1185,7 @@ if (!input0.ParseFromArray(remote, remote_len) || !input1.ParseFromArray(request, request_len)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1186,8 +1201,7 @@ if ((request == NULL) || (request_len == 0)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1197,8 +1211,7 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1212,7 +1225,9 @@ 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); + len = size > (sizeof(data_buf) - 1) ? (sizeof(data_buf) - 1) : size; + memcpy(data_buf, (char *)buf, len); + val = atoi((char *)data_buf); free(buf); @@ -1227,8 +1242,7 @@ if (topics_buf == NULL) { rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); logger->error("in BHRequest: Out of memory!\n"); @@ -1267,9 +1281,7 @@ } } - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - + bus_errorset(rv); if((msg_id == NULL) || (msg_id_len == NULL)) { if (rv == 0) return true; @@ -1306,9 +1318,11 @@ net_node_t node; int node_size; int recv_arr_size; + char data_buf[MAX_STR_LEN] = { 0x00 }; net_mod_recv_msg_t *recv_arr; net_mod_err_t *errarr; int errarr_size = 0; + char *errString = NULL; char buf_temp[MAX_STR_LEN] = { 0x00 }; char *topics_buf = NULL; @@ -1338,25 +1352,23 @@ if (!input0.ParseFromArray(remote, remote_len) || !input1.ParseFromArray(request, request_len) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } _input0.mq_id = input0.mq_id(); - _input0.abs_addr = input0.abs_addr(); - _input0.ip = input0.ip().c_str(); - _input0.port = input0.port(); - _input1.topic = input1.topic().c_str(); - _input1.data = input1.data().c_str(); + _input0.abs_addr = input0.abs_addr(); + _input0.ip = input0.ip().c_str(); + _input0.port = input0.port(); + _input1.topic = input1.topic().c_str(); + _input1.data = input1.data().c_str(); #else if ((request == NULL) || (request_len == 0) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1366,8 +1378,7 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1381,7 +1392,9 @@ 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); + len = size > (sizeof(data_buf) - 1) ? (sizeof(data_buf) - 1) : size; + memcpy(data_buf, (char *)buf, len); + val = atoi((char *)data_buf); free(buf); @@ -1393,13 +1406,11 @@ len += strlen(_input1.data); #endif - data = net_mod_socket_svr_get(gNetmod_socket); 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)); + bus_errorset(rv); logger->error("in BHRequest: Out of memory!\n"); @@ -1463,9 +1474,7 @@ free(topics_buf); } - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - + bus_errorset(rv); if (rv == 0) { if ((proc_id != NULL) && (proc_id_len != NULL)) { memset(buf_temp, 0x00, sizeof(buf_temp)); @@ -1481,8 +1490,7 @@ if (topics_buf == NULL) { rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); logger->error("in BHRequest: Out of memory!\n"); @@ -1497,6 +1505,8 @@ free(topics_buf); } + errString = bus_strerror(0, 1); + #if defined(PRO_DE_SERIALIZE) if (rv == 0) { ::bhome_msg::MsgRequestTopicReply mrt; @@ -1541,16 +1551,14 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } if ((request == NULL) || (request_len == 0) || (src == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1594,8 +1602,7 @@ if (topics_buf == NULL) { rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); logger->error("in BHReadRequest: Out of memory!\n"); @@ -1637,8 +1644,7 @@ *src = buf; } - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); if (rv == 0) return true; @@ -1650,6 +1656,8 @@ { int rv; int data; + int sec = 3; + int nsec = 0; const char *_input; #if defined(PRO_DE_SERIALIZE) @@ -1657,8 +1665,7 @@ if (!input.ParseFromArray(reply, reply_len) || (src == NULL)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1669,8 +1676,7 @@ if ((src == NULL) || (reply == NULL) || (reply_len == 0)) { rv = EBUS_INVALID_PARA; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } @@ -1683,17 +1689,15 @@ logger->error("the process has not been registered yet!\n"); rv = EBUS_RES_NO; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); return false; } data = net_mod_socket_svr_get(gNetmod_socket); - rv = net_mod_socket_sendto(gNetmod_socket, _input, strlen(_input), *(int *)src, SVR_STR, data); + rv = net_mod_socket_sendto_timeout(gNetmod_socket, _input, strlen(_input), *(int *)src, sec, nsec, SVR_STR, data); - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); if (rv == 0) return true; @@ -1713,6 +1717,7 @@ int BHGetLastError(void **msg, int *msg_len) { void *buf = NULL; + char *errString = bus_strerror(0, 1); buf = malloc(strlen(errString) + 1); -- Gitblit v1.8.0