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