From 5c912c70e9333298ff48f7ea15424f72ca977b99 Mon Sep 17 00:00:00 2001 From: Fu Juntang <StrongTiger_001@163.com> Date: 星期五, 17 九月 2021 09:43:55 +0800 Subject: [PATCH] Add the heartbeat logic feature. --- src/bh_api.cpp | 1099 +++++++++++++++++++++++++++++---------------------------- 1 files changed, 551 insertions(+), 548 deletions(-) diff --git a/src/bh_api.cpp b/src/bh_api.cpp index e36daee..635fe95 100644 --- a/src/bh_api.cpp +++ b/src/bh_api.cpp @@ -13,6 +13,8 @@ #include "../proto/source/bhome_msg.pb.h" #include "../proto/source/bhome_msg_api.pb.h" +#define TIME_WAIT 3 + static Logger *logger = LoggerFactory::getLogger(); static int gRun_stat = 0; @@ -20,7 +22,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 +67,7 @@ int count = 0; void *buf = NULL; int min = 0; + char *errString = NULL; ProcInfo pData; #if defined(PRO_DE_SERIALIZE) @@ -41,13 +80,11 @@ }_input; ::bhome_msg::ProcInfo input; - if(!input.ParseFromArray(proc_info, proc_info_len)) { + 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)); - - goto exit_entry; + bus_errorset(rv); + + return false; } _input.proc_id = input.proc_id().c_str(); @@ -56,13 +93,11 @@ _input.private_info = input.private_info().c_str(); #else - if ((proc_info == NULL) || (proc_info_len == 0)) { + 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)); - - goto exit_entry; + bus_errorset(rv); + + return false; } #endif @@ -74,10 +109,9 @@ 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); - goto exit_entry; + return false; } rv = pthread_mutex_trylock(&mutex); @@ -136,49 +170,65 @@ } #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(); key = hashtable_alloc_key(hashtable); + count = hashtable_alloc_key(hashtable); + rv = hashtable_alloc_key(hashtable); + net_mod_socket_int_set(gNetmod_socket, count); + net_mod_socket_svr_set(gNetmod_socket, rv); + sprintf(pData.int_info, "%d", count); + sprintf(pData.svr_info, "%d", rv); net_mod_socket_bind(gNetmod_socket, key); 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; } -exit_entry: + errString = bus_strerror(0, 1); + #if defined(PRO_DE_SERIALIZE) - ::bhome_msg::MsgCommonReply mcr; - mcr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); - mcr.mutable_errmsg()->set_errstring(errString); - *reply_len = mcr.ByteSizeLong(); - *reply = malloc(*reply_len); - mcr.SerializePartialToArray(*reply, *reply_len); + ::bhome_msg::MsgCommonReply mcr; + mcr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); + mcr.mutable_errmsg()->set_errstring(errString); + *reply_len = mcr.ByteSizeLong(); + *reply = malloc(*reply_len); + mcr.SerializePartialToArray(*reply, *reply_len); #else - min = strlen(errString) + 1; - buf = malloc(min) ; - memcpy(buf, errString, strlen(errString)); - *((char *)buf + min - 1) = '\0'; - - *reply = buf; - *reply_len = min; + min = strlen(errString) + 1; + buf = malloc(min) ; + memcpy(buf, errString, strlen(errString)); + *((char *)buf + min - 1) = '\0'; + + *reply = buf; + *reply_len = min; #endif - if (rv == 0) - return true; + pthread_create(&gTids, NULL, client_run_check, NULL); + + return true; - return false; } int BHUnregister(const void *proc_info, const int proc_info_len, void **reply, int *reply_len, const int timeout_ms) @@ -186,6 +236,7 @@ int rv; int min; void *buf = NULL; + char *errString = NULL; #if defined(PRO_DE_SERIALIZE) struct _ProcInfo_proto @@ -198,20 +249,25 @@ ::bhome_msg::ProcInfo input; - if(!input.ParseFromArray(proc_info, proc_info_len)) { + if(!input.ParseFromArray(proc_info, proc_info_len) || (reply == NULL) || (reply_len == NULL)) { rv = EBUS_INVALID_PARA; + bus_errorset(rv); - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - - goto exit_entry; + return false; } _input.proc_id = input.proc_id().c_str(); _input.name = input.name().c_str(); _input.public_info = input.public_info().c_str(); _input.private_info = input.private_info().c_str(); +#else + if ((reply == NULL) || (reply_len == NULL)) { + rv = EBUS_INVALID_PARA; + bus_errorset(rv); + + return false; + } #endif if (gRun_stat == 0) { @@ -219,10 +275,9 @@ 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); - goto exit_entry; + return false; } rv = pthread_mutex_trylock(&mutex); @@ -237,20 +292,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; } -exit_entry: + errString = bus_strerror(0, 1); #if defined(PRO_DE_SERIALIZE) ::bhome_msg::MsgCommonReply mcr; mcr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -268,10 +322,8 @@ *reply_len = min; #endif - if (rv == 0) - return true; - - return false; + return true; + } int BHRegisterTopics(const void *topics, const int topics_len, void **reply, int *reply_len, const int timeout_ms) @@ -281,6 +333,7 @@ void *buf = NULL; int total = 0; int count = 0; + char *errString = NULL; char *topics_buf = NULL; #if defined(PRO_DE_SERIALIZE) @@ -291,14 +344,12 @@ }_input; ::bhome_msg::MsgTopicList input; - if(!input.ParseFromArray(topics, topics_len)) { + 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); - goto exit_entry; + return false; } _input.amount = input.topic_list_size(); @@ -312,14 +363,12 @@ total += strlen(_input.topics[i]) + 1; } #else - if ((topics == NULL) || (topics_len == 0)) { + 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); - goto exit_entry; + return false; } total = topics_len; @@ -329,65 +378,50 @@ 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); - goto exit_entry; + return false; } - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { + topics_buf = (char *)malloc(total); + if (topics_buf == NULL) { + + rv = EBUS_NO_MEM; + bus_errorset(rv); + + logger->error("in BHRegisterTopics: Out of memory!\n"); - topics_buf = (char *)malloc(total); - if (topics_buf == NULL) { - - rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - - logger->error("in BHRegisterTopics: Out of memory!\n"); - - pthread_mutex_unlock(&mutex); - - goto exit_entry; - } - memset(topics_buf, 0x00, total); + return false; + } + memset(topics_buf, 0x00, total); #if defined(PRO_DE_SERIALIZE) - for (i = 0; i < _input.amount; i++) { - - len = strlen(_input.topics[i]); - strncpy(topics_buf + count, _input.topics[i], len); - - count += len; + for (i = 0; i < _input.amount; i++) { + + len = strlen(_input.topics[i]); + strncpy(topics_buf + count, _input.topics[i], len); + + count += len; - if ((_input.amount > 1) && (i < (_input.amount - 1))) { - strncpy(topics_buf + count, STR_MAGIC, strlen(STR_MAGIC)); - - count++; - } + if ((_input.amount > 1) && (i < (_input.amount - 1))) { + strncpy(topics_buf + count, STR_MAGIC, strlen(STR_MAGIC)); + + count++; } - -#else - memcpy(topics_buf, topics, topics_len); - count = topics_len; -#endif - - 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); - pthread_mutex_unlock(&mutex); - - } else { - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); } -exit_entry: +#else + memcpy(topics_buf, topics, topics_len); + count = topics_len; +#endif + + rv = net_mod_socket_reg(gNetmod_socket, topics_buf, count, NULL, 0, timeout_ms, PROC_REG_TCS); + + free(topics_buf); + + bus_errorset(rv); + errString = bus_strerror(0, 1); + #if defined(PRO_DE_SERIALIZE) ::bhome_msg::MsgCommonReply mcr; mcr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -405,10 +439,8 @@ *reply_len = len; #endif - if (rv == 0) - return true; + return true; - return false; } int BHQueryTopicAddress(const void *remote, const int remote_len, const void *topic, const int topic_len, void **reply, int *reply_len, const int timeout_ms) @@ -417,6 +449,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; @@ -434,13 +467,11 @@ ::bhome_msg::BHAddress input0; ::bhome_msg::MsgQueryTopic input1; - if (!input0.ParseFromArray(remote, remote_len) || !input1.ParseFromArray(topic, topic_len)) { + 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); - goto exit_entry; + return false; } _input0.mq_id = input0.mq_id(); @@ -450,13 +481,11 @@ _input1 = input1.topic().c_str(); #else - if ((topic == NULL) || (topic_len == 0)) { + 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); - goto exit_entry; + return false; } #endif @@ -464,38 +493,22 @@ 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); - goto exit_entry; + return false; } - - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { #if defined(PRO_DE_SERIALIZE) - min = (strlen(_input1) > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : strlen(_input1)); - strncpy(topics_buf, _input1, min); + min = (strlen(_input1) > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : strlen(_input1)); + strncpy(topics_buf, _input1, min); #else - min = (topic_len > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : topic_len); - buf = const_cast<void *>(topic); - strncpy(topics_buf, (const char *)buf, min); + min = (topic_len > (MAX_STR_LEN - 1) ? (MAX_STR_LEN - 1) : topic_len); + buf = const_cast<void *>(topic); + strncpy(topics_buf, (const char *)buf, min); #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)); - - pthread_mutex_unlock(&mutex); - - } else { - - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - } - -exit_entry: + rv = net_mod_socket_reg(gNetmod_socket, topics_buf, min, &buf, &size, timeout_ms, PROC_QUE_TCS); + + bus_errorset(rv); #if defined(PRO_DE_SERIALIZE) struct _MsgQueryTopicReply @@ -526,6 +539,8 @@ mtr_list[i].port = 5000; } } + + errString = bus_strerror(0, 1); ::bhome_msg::MsgQueryTopicReply mtr; mtr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -560,10 +575,8 @@ #endif - if (rv == 0) - return true; + return true; - return false; } int BHQueryProcs(const void *remote, const int remote_len, const void *query, const int query_len, void **reply, int *reply_len, const int timeout_ms) @@ -572,6 +585,7 @@ void *buf = NULL; int size; int min; + char *errString = NULL; ProcInfo_sum *Proc_ptr = NULL; char data_buf[MAX_STR_LEN] = { 0x00 }; @@ -588,13 +602,12 @@ ::bhome_msg::BHAddress input0; ::bhome_msg::MsgQueryProc input1; - if (!input0.ParseFromArray(remote, remote_len) || !input1.ParseFromArray(query, query_len)) { + 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); - goto exit_entry; + return false; } _input0.mq_id = input0.mq_id(); @@ -602,40 +615,31 @@ _input0.ip = input0.ip().c_str(); _input0.port = input0.port(); _input1 = input1.proc_id().c_str(); +#else + if ((reply == NULL) || (reply_len == NULL)) { + rv = EBUS_INVALID_PARA; + bus_errorset(rv); + + return false; + } #endif if (gRun_stat == 0) { 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)); - - goto exit_entry; + bus_errorset(rv); + + return false; } - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { - - if (query != NULL) { - strncpy(data_buf, (char *)query, (sizeof(data_buf) - 1) > query_len ? query_len : (sizeof(data_buf) - 1)); - } - - 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)); - - pthread_mutex_unlock(&mutex); - - } else { - - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + if (query != NULL) { + strncpy(data_buf, (char *)query, (sizeof(data_buf) - 1) > query_len ? query_len : (sizeof(data_buf) - 1)); } -exit_entry: + rv = net_mod_socket_reg(gNetmod_socket, data_buf, strlen(data_buf), &buf, &size, timeout_ms, PROC_QUE_ATCS); + bus_errorset(rv); + #if defined(PRO_DE_SERIALIZE) struct _MsgQueryProcReply { @@ -680,6 +684,8 @@ } } } + + errString = bus_strerror(0, 1); ::bhome_msg::MsgQueryProcReply mpr; mpr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -718,10 +724,7 @@ } #endif - if (rv == 0) - return true; - - return false; + return true; } @@ -733,6 +736,7 @@ int count = 0; int len, i; void *buf = NULL; + char *errString = NULL; char *topics_buf = NULL; #if defined(PRO_DE_SERIALIZE) @@ -743,13 +747,12 @@ }_input; ::bhome_msg::MsgTopicList input; - if(!input.ParseFromArray(topics, topics_len)) { + 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); - goto exit_entry; + return false; } _input.amount = input.topic_list_size(); @@ -765,12 +768,11 @@ } #else - if ((topics == NULL) || (topics_len == 0)) { + 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); - goto exit_entry; + return false; } #endif @@ -778,79 +780,62 @@ 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); - goto exit_entry; + return false; } - - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { - - topics_buf = (char *)malloc(total); - if (topics_buf == NULL) { - - rv = EBUS_NO_MEM; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - - logger->error("in BHSubscribeTopics: Out of memory!\n"); - pthread_mutex_unlock(&mutex); + topics_buf = (char *)malloc(total); + if (topics_buf == NULL) { + + rv = EBUS_NO_MEM; + bus_errorset(rv); + + logger->error("in BHSubscribeTopics: Out of memory!\n"); - goto exit_entry; - } - memset(topics_buf, 0x00, total); - + return false; + } + memset(topics_buf, 0x00, total); + #if defined(PRO_DE_SERIALIZE) - for (i = 0; i < _input.amount; i++) { - len = strlen(_input.topics[i]); - strncpy(topics_buf + count, _input.topics[i], len); - - count += len; + for (i = 0; i < _input.amount; i++) { + len = strlen(_input.topics[i]); + strncpy(topics_buf + count, _input.topics[i], len); + + count += len; - if ((_input.amount > 1) && (i < (_input.amount - 1))) { - strncpy(topics_buf + count, STR_MAGIC, strlen(STR_MAGIC)); - - count++; - } + if ((_input.amount > 1) && (i < (_input.amount - 1))) { + strncpy(topics_buf + count, STR_MAGIC, strlen(STR_MAGIC)); + + count++; } + } #else - memcpy(topics_buf, topics, topics_len); - count = topics_len; + memcpy(topics_buf, topics, topics_len); + count = topics_len; #endif - if (timeout_ms > 0) { - - sec = timeout_ms / 1000; - nsec = (timeout_ms - sec * 1000) * 1000 * 1000; - rv = net_mod_socket_sub_timeout(gNetmod_socket, topics_buf, count, sec, nsec); + if (timeout_ms > 0) { + + sec = timeout_ms / 1000; + nsec = (timeout_ms - sec * 1000) * 1000 * 1000; + rv = net_mod_socket_sub_timeout(gNetmod_socket, topics_buf, count, sec, nsec); - } else if (timeout_ms == 0) { - - rv = net_mod_socket_sub_nowait(gNetmod_socket, topics_buf, count); - - } else { - - rv = net_mod_socket_sub(gNetmod_socket, topics_buf, count); - - } - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - - free(topics_buf); - pthread_mutex_unlock(&mutex); - + } else if (timeout_ms == 0) { + + rv = net_mod_socket_sub_nowait(gNetmod_socket, topics_buf, count); + } else { - - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + + rv = net_mod_socket_sub(gNetmod_socket, topics_buf, count); + } + + errString = bus_strerror(0, 1); + + free(topics_buf); -exit_entry: #if defined(PRO_DE_SERIALIZE) ::bhome_msg::MsgCommonReply mcr; mcr.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); @@ -868,10 +853,7 @@ *reply_len = len; #endif - if (rv == 0) - return true; - - return false; + return true; } @@ -891,6 +873,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 @@ -905,9 +888,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; } @@ -918,14 +899,14 @@ _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)); mcr.mutable_errmsg()->set_errstring(errString); - *reply_len=mcr.ByteSizeLong(); - *reply=malloc(*reply_len); + *reply_len = mcr.ByteSizeLong(); + *reply = malloc(*reply_len); mcr.SerializePartialToArray(*reply,*reply_len); #endif @@ -955,8 +936,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; } @@ -967,8 +947,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; } @@ -978,51 +957,38 @@ 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; } - - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { + #if defined(PRO_DE_SERIALIZE) - if (timeout_ms > 0) { - rv = net_mod_socket_pub_timeout(gNetmod_socket, &node_arr, node_arr_len, _input.topic, strlen(_input.topic), _input.data, strlen(_input.data), timeout_ms); - } else if (timeout_ms == 0) { - rv = net_mod_socket_pub_nowait(gNetmod_socket, &node_arr, node_arr_len, _input.topic, strlen(_input.topic), _input.data, strlen(_input.data)); - - } else { - - rv = net_mod_socket_pub(gNetmod_socket, &node_arr, node_arr_len, _input.topic, strlen(_input.topic), _input.data, strlen(_input.data)); - } -#else - if (timeout_ms > 0) { - rv = net_mod_socket_pub_timeout(gNetmod_socket, &node_arr, node_arr_len, topic, strlen(topic), content, strlen(content), timeout_ms); - - } else if (timeout_ms == 0) { - rv = net_mod_socket_pub_nowait(gNetmod_socket, &node_arr, node_arr_len, topic, strlen(topic), content, strlen(content)); + if (timeout_ms > 0) { + rv = net_mod_socket_pub_timeout(gNetmod_socket, &node_arr, node_arr_len, _input.topic, strlen(_input.topic), _input.data, strlen(_input.data), timeout_ms); + } else if (timeout_ms == 0) { + rv = net_mod_socket_pub_nowait(gNetmod_socket, &node_arr, node_arr_len, _input.topic, strlen(_input.topic), _input.data, strlen(_input.data)); - } else { - rv = net_mod_socket_pub(gNetmod_socket, &node_arr, node_arr_len, topic, strlen(topic), content, strlen(content)); - } + } else { + + rv = net_mod_socket_pub(gNetmod_socket, &node_arr, node_arr_len, _input.topic, strlen(_input.topic), _input.data, strlen(_input.data)); + } +#else + if (timeout_ms > 0) { + rv = net_mod_socket_pub_timeout(gNetmod_socket, &node_arr, node_arr_len, topic, strlen(topic), content, strlen(content), timeout_ms); + + } else if (timeout_ms == 0) { + rv = net_mod_socket_pub_nowait(gNetmod_socket, &node_arr, node_arr_len, topic, strlen(topic), content, strlen(content)); + + } else { + rv = net_mod_socket_pub(gNetmod_socket, &node_arr, node_arr_len, topic, strlen(topic), content, strlen(content)); + } #endif - pthread_mutex_unlock(&mutex); - - if (rv > 0) - return true; - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - - } else { - - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - } - + if (rv > 0) + return true; + + bus_errorset(rv); + return false; } @@ -1034,8 +1000,9 @@ int key; int size; int sec, nsec; - char topics_buf[MAX_STR_LEN] = { 0x00 }; - char data_buf[MAX_STR_LEN * 3] = { 0x00 }; + char buf_temp[100] = { 0x00 }; + char *topics_buf = NULL; + char *data_buf = NULL; struct _ReadSubReply { @@ -1048,10 +1015,16 @@ 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; + bus_errorset(rv); + + return false; } if (timeout_ms > 0) { @@ -1075,11 +1048,38 @@ if (len > size) { len = size; } - strncpy(topics_buf, (char *)buf, len > (sizeof(topics_buf) - 1) ? (sizeof(topics_buf) - 1) : len); + + topics_buf = (char *)malloc(len + 10); + if (topics_buf == NULL) { + + rv = EBUS_NO_MEM; + bus_errorset(rv); + + logger->error("in BHRequest: Out of memory!\n"); + + return false; + } + memset(topics_buf, 0x00, len + 10); + + strncpy(topics_buf, (char *)buf, len); if (len < size) { - len = strlen(topics_buf) + 1; + + data_buf = (char *)malloc(size - len + 10); + if (data_buf == NULL) { + + rv = EBUS_NO_MEM; + bus_errorset(rv); + + logger->error("in BHRequest: Out of memory!\n"); + + free(topics_buf); + return false; + } + memset(data_buf, 0x00, size - len + 10); + + len = strlen(topics_buf) + 1; strncpy(data_buf, (char *)buf + len, size - len); } @@ -1087,15 +1087,23 @@ #if defined(PRO_DE_SERIALIZE) rsr.topic = topics_buf; - rsr.data = data_buf; + if (data_buf != NULL) { + rsr.data = data_buf; + } else { + rsr.data = topics_buf; + } - memset(topics_buf, 0x00, sizeof(topics_buf)); - sprintf(topics_buf, "%d", key); + sprintf(buf_temp, "%d", key); - rsr.proc_id = topics_buf; - *proc_id_len = rsr.proc_id.size(); - *proc_id = malloc(*proc_id_len); - memcpy(*proc_id, rsr.proc_id.data(), *proc_id_len); + if ((proc_id != NULL) && (proc_id_len != NULL)) { + rsr.proc_id = buf_temp; + *proc_id_len = rsr.proc_id.size(); + *proc_id = malloc(*proc_id_len); + memcpy(*proc_id, rsr.proc_id.data(), *proc_id_len); + } + + free(topics_buf); + free(data_buf); ::bhome_msg::MsgPublish Mp; Mp.set_topic(rsr.topic); @@ -1115,20 +1123,25 @@ } *msgpub = ptr; *msgpub_len = len; - - memset(topics_buf, 0x00, sizeof(topics_buf)); - sprintf(topics_buf, "%d", key); - - *proc_id_len = strlen(topics_buf); - *proc_id = malloc(*proc_id_len); - memcpy(*proc_id, topics_buf, *proc_id_len); + + free(topics_buf); + free(data_buf); + + if ((proc_id != NULL) && (proc_id_len != NULL)) { + memset(buf_temp, 0x00, sizeof(buf_temp)); + sprintf(buf_temp, "%d", key); + + *proc_id_len = strlen(buf_temp); + *proc_id = malloc(*proc_id_len); + memcpy(*proc_id, buf_temp, *proc_id_len); + } #endif } else { - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); + } if (rv == 0) @@ -1145,6 +1158,7 @@ int val; int len; int min; + int data; int sec, nsec; std::string MsgID; int timeout_ms = 3000; @@ -1171,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; } @@ -1188,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; } @@ -1199,101 +1211,87 @@ 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; } - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { #if defined(PRO_DE_SERIALIZE) - strncpy(buf_temp, _input1.topic, (sizeof(buf_temp) - 1) > strlen(_input1.topic) ? strlen(_input1.topic) : (sizeof(buf_temp) - 1)); + strncpy(buf_temp, _input1.topic, (sizeof(buf_temp) - 1) > strlen(_input1.topic) ? strlen(_input1.topic) : (sizeof(buf_temp) - 1)); #else - strncpy(buf_temp, (char *)request, (sizeof(buf_temp) - 1) > strlen((char *)request) ? strlen((char *)request) : (sizeof(buf_temp) - 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, buf_temp, strlen(buf_temp), &buf, &size, timeout_ms, PROC_QUE_STCS); - if (rv == 0) { + + 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); + val = atoi((char *)buf); - free(buf); + free(buf); - if (val > 0) { + if (val > 0) { - len = strlen(buf_temp) + 1; + len = strlen(buf_temp) + 1; #if defined(PRO_DE_SERIALIZE) - len += strlen(_input1.data); + 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); + topics_buf = (char *)malloc(len); + if (topics_buf == NULL) { - strncpy(topics_buf, buf_temp, strlen(buf_temp) + 1); + rv = EBUS_NO_MEM; + bus_errorset(rv); + + logger->error("in BHRequest: Out of memory!\n"); + + 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)); + strncpy(topics_buf + strlen(buf_temp) + 1, _input1.data, strlen(_input1.data)); #endif - if (timeout_ms > 0) { + data = net_mod_socket_svr_get(gNetmod_socket); + if (timeout_ms > 0) { - sec = timeout_ms / 1000; - nsec = (timeout_ms - sec * 1000) * 1000 * 1000; - - rv = net_mod_socket_sendto_timeout(gNetmod_socket, topics_buf, len, val, sec, nsec); - - } else if (timeout_ms == 0) { - - rv = net_mod_socket_sendto_nowait(gNetmod_socket, topics_buf, len, val); - - } else { - - rv = net_mod_socket_sendto(gNetmod_socket, topics_buf, len, val); - } - - free(topics_buf); + sec = timeout_ms / 1000; + nsec = (timeout_ms - sec * 1000) * 1000 * 1000; + rv = net_mod_socket_sendto_timeout(gNetmod_socket, topics_buf, len, val, sec, nsec, SVR_STR, data); + + } else if (timeout_ms == 0) { + + rv = net_mod_socket_sendto_nowait(gNetmod_socket, topics_buf, len, val, SVR_STR, data); + } else { - rv = EBUS_RES_UNSUPPORT; - - } - } + rv = net_mod_socket_sendto(gNetmod_socket, topics_buf, len, val, SVR_STR, data); + } - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - - pthread_mutex_unlock(&mutex); - - if((msg_id == NULL) || (msg_id_len == NULL)) { - if (rv == 0) - return true; + free(topics_buf); - return false; - } - } else { + } else { - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + rv = EBUS_RES_UNSUPPORT; + + } + } + + bus_errorset(rv); + if((msg_id == NULL) || (msg_id_len == NULL)) { + if (rv == 0) + return true; + + return false; } if (rv == 0) { - memset(topics_buf, 0x00, sizeof(topics_buf)); - sprintf(topics_buf, "%d", val); - MsgID = topics_buf; + + memset(buf_temp, 0x00, sizeof(buf_temp)); + sprintf(buf_temp, "%d", val); + MsgID = buf_temp; *msg_id_len = MsgID.size(); *msg_id = malloc(*msg_id_len); @@ -1314,12 +1312,14 @@ int size; int val; int min, len; + int data; net_node_t node; int node_size; int recv_arr_size; 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; @@ -1346,28 +1346,26 @@ ::bhome_msg::BHAddress input0; ::bhome_msg::MsgRequestTopic input1; - if (!input0.ParseFromArray(remote, remote_len) || !input1.ParseFromArray(request, request_len)) { + 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)) { + 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; } @@ -1377,139 +1375,146 @@ 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; } - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { #if defined(PRO_DE_SERIALIZE) - strncpy(buf_temp, _input1.topic, (sizeof(buf_temp) - 1) > strlen(_input1.topic) ? strlen(_input1.topic) : (sizeof(buf_temp) - 1)); + strncpy(buf_temp, _input1.topic, (sizeof(buf_temp) - 1) > strlen(_input1.topic) ? strlen(_input1.topic) : (sizeof(buf_temp) - 1)); #else - strncpy(buf_temp, (char *)request, (sizeof(buf_temp) - 1) > request_len ? request_len : (sizeof(buf_temp) - 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, buf_temp, strlen(buf_temp), &buf, &size, timeout_ms, PROC_QUE_STCS); + if (rv == 0) { - 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); + + if (val > 0) { + memset(&node, 0x00, sizeof(node)); - val = atoi((char *)buf); - - free(buf); - - if (val > 0) { - memset(&node, 0x00, sizeof(node)); - - len = strlen(buf_temp) + 1; + len = strlen(buf_temp) + 1; #if defined(PRO_DE_SERIALIZE) - len += strlen(_input1.data); + 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); + data = net_mod_socket_svr_get(gNetmod_socket); + topics_buf = (char *)malloc(len); + if (topics_buf == NULL) { - 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 = EBUS_NO_MEM; + bus_errorset(rv); - 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); + logger->error("in BHRequest: Out of memory!\n"); - } 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; - - 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; - *reply_len = size; -#endif - } - - net_mod_socket_free_recv_msg_arr(recv_arr, recv_arr_size); - - if(errarr_size > 0) { - free(errarr); - } - - rv = 0; - - } else { - rv = EBUS_TIMEOUT; - } - - } else { - rv = EBUS_RES_UNSUPPORT; + 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 - free(topics_buf); + node.key = val; + + 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; + + size = recv_arr[0].content_length; + buf = (char *)malloc(size); + if (buf == NULL) { + printf("Out of memory\n"); + + exit(0); + } + memset((char *)buf, 0x00, size); + + strncpy((char *)buf, (char *)recv_arr[0].content, size); + + } + + net_mod_socket_free_recv_msg_arr(recv_arr, recv_arr_size); + + if(errarr_size > 0) { + free(errarr); + } + + rv = 0; + + } else { + rv = EBUS_TIMEOUT; + } + + } else { + rv = EBUS_RES_UNSUPPORT; } - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + free(topics_buf); + } - if (rv == 0) { + bus_errorset(rv); + if (rv == 0) { + if ((proc_id != NULL) && (proc_id_len != NULL)) { memset(buf_temp, 0x00, sizeof(buf_temp)); sprintf(buf_temp, "%d", node.key); 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(buf_temp, 0x00, sizeof(buf_temp)); - memcpy(buf_temp, buf, size); - rr.data = buf_temp; + memcpy(*proc_id, rr.proc_id.c_str(), *proc_id_len); } - - pthread_mutex_unlock(&mutex); - } else { - - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + topics_buf = (char *)malloc(size + 10); + if (topics_buf == NULL) { + + rv = EBUS_NO_MEM; + bus_errorset(rv); + + logger->error("in BHRequest: Out of memory!\n"); + + return false; + } + memset(topics_buf, 0x00, size + 10); + + memcpy(topics_buf, buf, size); + rr.data = topics_buf; + + free(buf); + free(topics_buf); } -#if defined(PRO_DE_SERIALIZE) - ::bhome_msg::MsgRequestTopicReply mrt; - mrt.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); - mrt.mutable_errmsg()->set_errstring(errString); - mrt.set_data(rr.data.data()); - *reply_len = mrt.ByteSizeLong(); - *reply = malloc(*reply_len); - mrt.SerializePartialToArray(*reply, *reply_len); + errString = bus_strerror(0, 1); + +#if defined(PRO_DE_SERIALIZE) + if (rv == 0) { + ::bhome_msg::MsgRequestTopicReply mrt; + mrt.mutable_errmsg()->set_errcode(::bhome_msg::ErrorCode(rv)); + mrt.mutable_errmsg()->set_errstring(errString); + mrt.set_data(rr.data.data()); + *reply_len = mrt.ByteSizeLong(); + *reply = malloc(*reply_len); + mrt.SerializePartialToArray(*reply, *reply_len); + } #else - if (rv > 0) { + if (rv == 0) { min = strlen(errString) + 1; buf = malloc(min); memcpy(buf, errString, strlen(errString)); @@ -1533,6 +1538,7 @@ int key; int size; int len; + int data; int sec, nsec; char buf_temp[MAX_STR_LEN] = { 0x00 }; char *topics_buf = NULL; @@ -1541,26 +1547,33 @@ 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; + bus_errorset(rv); + + return false; + } + data = net_mod_socket_svr_get(gNetmod_socket); if (timeout_ms > 0) { sec = timeout_ms / 1000; nsec = (timeout_ms - sec * 1000) * 1000 * 1000; - rv = net_mod_socket_recvfrom_timeout(gNetmod_socket, &buf, &size, &key, sec, nsec); + rv = net_mod_socket_recvfrom_timeout(gNetmod_socket, &buf, &size, &key, sec, nsec, SVR_STR, data); } else if (timeout_ms == 0) { - rv = net_mod_socket_recvfrom_nowait(gNetmod_socket, &buf, &size, &key); + rv = net_mod_socket_recvfrom_nowait(gNetmod_socket, &buf, &size, &key, SVR_STR, data); } else { - rv = net_mod_socket_recvfrom(gNetmod_socket, &buf, &size, &key); + rv = net_mod_socket_recvfrom(gNetmod_socket, &buf, &size, &key, SVR_STR, data); } if (rv == 0) { @@ -1572,19 +1585,20 @@ void *src; } rrr; - sprintf(buf_temp, "%d", key); - rrr.proc_id = buf_temp; + if ((proc_id != NULL) && (proc_id_len != NULL)) { + 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); + *proc_id_len = rrr.proc_id.size(); + *proc_id = malloc(*proc_id_len); + memcpy(*proc_id, rrr.proc_id.data(), *proc_id_len); + } 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)); + bus_errorset(rv); logger->error("in BHReadRequest: Out of memory!\n"); @@ -1626,8 +1640,7 @@ *src = buf; } - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); + bus_errorset(rv); if (rv == 0) return true; @@ -1638,15 +1651,17 @@ int BHSendReply(void *src, const void *reply, const int reply_len) { int rv; + int data; + int sec = 3; + int nsec = 0; const char *_input; #if defined(PRO_DE_SERIALIZE) ::bhome_msg::MsgRequestTopicReply input; - if (!input.ParseFromArray(reply, reply_len)) { +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; } @@ -1657,8 +1672,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; } @@ -1671,27 +1685,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; } - rv = pthread_mutex_trylock(&mutex); - if (rv == 0) { + data = net_mod_socket_svr_get(gNetmod_socket); + rv = net_mod_socket_sendto_timeout(gNetmod_socket, _input, strlen(_input), *(int *)src, sec, nsec, SVR_STR, data); - rv = net_mod_socket_sendto(gNetmod_socket, _input, strlen(_input), *(int *)src); - - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - - pthread_mutex_unlock(&mutex); - } else { - - rv = EBUS_RES_BUSY; - memset(errString, 0x00, sizeof(errString)); - strncpy(errString, bus_strerror(rv), sizeof(errString)); - } + bus_errorset(rv); if (rv == 0) return true; @@ -1711,6 +1713,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