From 83956b12d863924936a98c9dfbece37feb0cce9c Mon Sep 17 00:00:00 2001 From: wangzhengquan <wangzhengquan85@126.com> Date: 星期四, 04 二月 2021 14:48:04 +0800 Subject: [PATCH] update --- src/socket/shm_socket.cpp | 63 ++++++++++++++++--------------- 1 files changed, 32 insertions(+), 31 deletions(-) diff --git a/src/socket/shm_socket.cpp b/src/socket/shm_socket.cpp index bddcec0..39c79b5 100644 --- a/src/socket/shm_socket.cpp +++ b/src/socket/shm_socket.cpp @@ -153,11 +153,14 @@ int rv; - shm_packet_t sendpak; + shm_packet_t sendpak = {0}; sendpak.key = sockt->key; sendpak.size = size; - sendpak.buf = mm_malloc(size); - memcpy(sendpak.buf, buf, size); + if(buf != NULL) { + sendpak.buf = mm_malloc(size); + memcpy(sendpak.buf, buf, size); + } + rv = shm_sendpakto(sockt, &sendpak, key, timeout, flag); return rv; } @@ -176,8 +179,10 @@ sendpak.key = sockt->key; sendpak.size = send_size; - sendpak.buf = mm_malloc(send_size); - memcpy(sendpak.buf, send_buf, send_size); + if(send_buf != NULL) { + sendpak.buf = mm_malloc(send_size); + memcpy(sendpak.buf, send_buf, send_size); + } memcpy(sendpak.uuid, uuid.c_str(), uuid.length() + 1); // uuid.copy(sendpak.uuid, sizeof sendpak.uuid); rv = shm_sendpakto(sockt, &sendpak, key, timeout, flags); @@ -242,55 +247,50 @@ } /** - * @callback void (*recv_callback_fn)(void **sendbuf, int *sendsize) + * @callback void (*recvandsend_callback_fn)(void *recvbuf, int recvsize, int key, void **sendbuf, int *sendsize) * sendbuf 鍜� sendsize鏄痗allbak_fn鍥炶皟鍑芥暟鐨勮繑鍥炲��, 鍒嗗埆琛ㄧず鍙戦�佹暟鎹紝鍜屽彂閫佹暟鎹殑澶у皬銆� * */ -int shm_recvandsend(shm_socket_t *sockt, void **recvbuf, int *recvsize, int *key, recv_callback_fn callback, - const struct timespec *timeout, int flag) { +int shm_recvandsend(shm_socket_t *sockt, recvandsend_callback_fn callback, + const struct timespec *timeout, int flag, void *user_data) { int rv; - void *sendbuf = NULL; + void *sendbuf, *recvbuf = NULL; int sendsize = 0; - shm_packet_t recvpak; + shm_packet_t recvpak = {0}; rv = shm_recvpakfrom(sockt , &recvpak, timeout, flag); - - if (rv != 0) { - if(rv == ETIMEDOUT) + if(rv == ETIMEDOUT){ + logger->debug("%d shm_recvandsend failed %s", shm_socket_get_key(sockt), bus_strerror(EBUS_TIMEOUT)); return EBUS_TIMEOUT; - else { - logger->debug("%d shm_recvfrom failed %s", shm_socket_get_key(sockt), bus_strerror(rv)); - return rv; } + + logger->error("%d shm_recvandsend failed %s", shm_socket_get_key(sockt), bus_strerror(rv)); + return rv; } - if(recvbuf != NULL) { - void *_buf = malloc(recvpak.size); - memcpy(_buf, recvpak.buf, recvpak.size); - *recvbuf = _buf; + if(recvpak.buf != NULL) { + recvbuf = malloc(recvpak.size); + memcpy(recvbuf, recvpak.buf, recvpak.size); + mm_free(recvpak.buf); } - if(recvsize != NULL) - *recvsize = recvpak.size; + callback(recvbuf, recvpak.size, recvpak.key, &sendbuf, &sendsize, user_data); - if(key != NULL) - *key = recvpak.key; - - mm_free(recvpak.buf); - - callback(&sendbuf, &sendsize); - - shm_packet_t sendpak; + shm_packet_t sendpak = {0}; sendpak.key = sockt->key; sendpak.size = sendsize; memcpy(sendpak.uuid, recvpak.uuid, sizeof sendpak.uuid); if(sendbuf !=NULL && sendsize > 0) { sendpak.buf = mm_malloc(sendsize); memcpy(sendpak.buf, sendbuf, sendsize); + } else { + + logger->warn("%d shm_recvandsend : sendbuf is null", shm_socket_get_key(sockt)); + // return -1; } rv = shm_sendpakto(sockt, &sendpak, recvpak.key, timeout, flag); @@ -318,7 +318,7 @@ } - if(buf != NULL) { + if(buf != NULL && recvpak.buf != NULL) { void *_buf = malloc(recvpak.size); memcpy(_buf, recvpak.buf, recvpak.size); *buf = _buf; @@ -555,6 +555,7 @@ LABEL_POP: + printf("%p start recv.....\n", sockt); rv = sockt->queue->pop(*recvpak, timeout, flag); -- Gitblit v1.8.0