fix memory leak

Author: cgzhang6

Makefile

@@ -18,7 +18,7 @@ endif
 
 wrapper:
 	echo $(PYLIB)
-	${CC}  -Wall -pedantic -Wextra -fPIC -shared  -std=c++1y  -fvisibility=default  -Wno-attributes  $(PYINC) -g -O0 -I. -I./include/spdlog/include/ -I./include/ -o libwrapper.so pyWrapper.cpp fswatch.cpp wrapper.cpp  -L. -L /opt/conda/envs/loader/lib $(PYLIB) -rdynamic
+	${CC}  -Wall -pedantic -Wextra -fPIC -shared  -std=c++1y  -fvisibility=default -Wno-unused-variable -Wno-deprecated-declarations -Wno-unused-parameter -Wno-attributes  $(PYINC) -g -O0 -I. -I./include/spdlog/include/ -I./include/ -o libwrapper.so pyWrapper.cpp fswatch.cpp wrapper.cpp  -L. -L /opt/conda/envs/loader/lib $(PYLIB) -rdynamic
 	mkdir -p wrapper_lib
 	cp libwrapper.so  ./wrapper_lib
 

fswatch.cpp

@@ -208,9 +208,9 @@ int main() {
     EventHandle e = test;
     funs.insert({"IN_MOVE_SELF", test});
     ino.InitWatchFile(s, NULL);
-    printf("pid,%d\n", _pid);
+    printf("pid,%lu\n", _pid);
     int ret = ino.StartWatchThread(funs, _pid);
-    printf("aftpid,%d\n", _pid);
+    printf("aftpid,%lu\n", _pid);
     ret = pthread_join(_pid, NULL);
     printf("%d\n", ret);
     return 0;

pyWrapper.cpp

@@ -94,7 +94,7 @@ py::bytes DataListNode::get_data() {
 }
 
 DataListNode *DataListCls::get(std::string key) {
-    for (int idx = 0; idx < list.size(); idx++) {
+    for (size_t idx = 0; idx < list.size(); idx++) {
         DataListNode *node = &list[idx];
         if (strcmp(node->key.c_str(), key.c_str()) == 0) {
             return node;
@@ -201,7 +201,7 @@ void PyWrapper::StartMonitorWrapperClass(std::string wrapperFileAbs) {
     FSInotify *ino = new FSInotify();
     pthread_t _pid;
     std::vector <std::string> s;
-    printf("starting monitoring %s, pid is: %d\n", wrapperFileAbs.c_str(), _pid);
+    printf("starting monitoring %s, pid is: %lu\n", wrapperFileAbs.c_str(), _pid);
     s.push_back(wrapperFileAbs);
     std::map <std::string, EventHandle> funs;
 
@@ -210,7 +210,7 @@ void PyWrapper::StartMonitorWrapperClass(std::string wrapperFileAbs) {
     ino->InitWatchFile(s, this);
     int ret = ino->StartWatchThread(funs, _pid);
     if (ret != 0) {
-        printf("Error starting monitoring %s, pid is: %d\n", wrapperFileAbs.c_str(), _pid);
+        printf("Error starting monitoring %s, pid is: %lu\n", wrapperFileAbs.c_str(), _pid);
     }
 }
 
@@ -297,7 +297,7 @@ int PyWrapper::wrapperOnceExec(const char *usrTag, std::map <std::string, std::s
                 return ret;
             }
             ptr = PyBytes_AsString(itemData.data.ptr());
-            Py_ssize_t size = PyBytes_GET_SIZE(itemData.data.ptr());
+            // Py_ssize_t size = PyBytes_GET_SIZE(itemData.data.ptr());
 //            printf("GetSIze, %d", size);
 //            printf("item data len: %d", itemData.len);
             memcpy(pr, ptr, itemData.len);
@@ -551,7 +551,7 @@ int PyWrapper::wrapperLoadRes(pDataList p, unsigned int resId) {
             item.data = py::bytes((char *) (p->data), len);
 
             item.len = p->len;
-            char t = static_cast<int>(p->type);
+            // char t = static_cast<int>(p->type);
             item.type = p->type;
             item.status = p->status;
             spdlog::debug("reqDatatype :{}，resID:{}", p->type, resId);
@@ -593,7 +593,7 @@ int PyWrapper::wrapperTest() {
         std::cout << e.what() << std::endl;
         return -1;
     }
-    for (int i = 0; i < l->list.size(); ++i) {
+    for (size_t i = 0; i < l->list.size(); ++i) {
         ResponseData d = l->list[i];
 //        std::cout << "Response key: " << d.key << std::endl;
         //      std::cout << "Response len" << d.len << std::endl;
@@ -616,7 +616,7 @@ int callbackMetric(const char *usrTag, const char *meterKey, int count) {
 }
 
 int callbackTrace(const char *usrTag, const char *key, const char *value) {
-    printf("callback Trace: %s, %s, %d\n", usrTag, key, value);
+    printf("callback Trace: %s, %s, %s\n", usrTag, key, value);
     return g_trace_cb(usrTag, key, value);
 }
 
@@ -629,8 +629,8 @@ int callBack(Response *resp, std::string sid) {
     }
     const char *usrTag = GetSidUsrTag(sid);
 
-    pDataList headPtr;
-    pDataList curPtr;
+    pDataList headPtr = nullptr;
+    pDataList curPtr = nullptr;
     // 先判断python有没有抛出错误. response中的 errorCode
     if (resp->errCode != 0) {
         spdlog::get("stderr_console")->error("find error from python: {}", resp->errCode);
@@ -679,8 +679,20 @@ int callBack(Response *resp, std::string sid) {
                       tmpData->status, sid);
     }
 
-    cb_(usrTag, headPtr, 0);
-    printf("ok\n:");
+    int cb_ret = cb_(usrTag, headPtr, 0);
+    while (headPtr != nullptr) {
+        pDataList ptr = headPtr;
+        headPtr = headPtr->next;
+        if (ptr->key) {
+            free(ptr->key);
+        }
+        if (ptr->data) {
+            free(ptr->data);
+        }
+        delete ptr;
+    }
+
+    spdlog::debug("call c's callback, usrTag:{} ret {}",usrTag, cb_ret);
     return 0;
 
 }

pyWrapper.h

@@ -66,8 +66,8 @@ class ResponseData {
     std::string key;
     py::bytes data;
     unsigned int len;
-    int status;
     int type;
+    int status;
 };
 
 class Response {

wrapper.cpp

@@ -54,9 +54,9 @@ void initlog(std::string logDir, std::string logpath) {
 
     // Creating a directory
     if (mkdir(logDir.c_str(), 0777) == -1) {
-        printf("log目录创建失败或已存在 %s\n", logpath);
+        printf("log目录创建失败或已存在 %s\n", logpath.c_str());
     } else {
-        printf("log目录已创建, %s \n", logpath);
+        printf("log目录已创建, %s \n", logpath.c_str());
     }
     auto file_logger = spdlog::rotating_logger_mt("mspper", logpath.c_str(), 1048576 * 10, 50);
     // Console logger
@@ -107,7 +107,7 @@ int WrapperAPI wrapperInit(pConfig cfg) {
     pyWrapper = new PyWrapper();
 
     setLog(loglvl);
-    printf("WrapperInit: 当前线程ID: %d \n", gettid());
+    printf("WrapperInit: 当前线程ID: %ld \n", gettid());
     if (global_metric_cb != NULL) {
         printf("Metric Custom func set! \n");
         pyWrapper->wrapperSetMetricFunc(CTMeterCustom, global_metric_cb);
@@ -122,7 +122,7 @@ int WrapperAPI wrapperInit(pConfig cfg) {
 }
 
 int WrapperAPI wrapperFini() {
-    printf("WrapperFini: 当前线程ID: %d \n", gettid());
+    printf("WrapperFini: 当前线程ID: %ld \n", gettid());
     pyWrapper->wrapperFini();
     return 0;
 }