fix(server): server closing deallocation fixed

examples/server/server_pyrfc_thread.py

@@ -36,7 +36,7 @@ def my_auth_check(func_name=False, request_context=None):
 
 
 # create server
-server = Server({"dest": "gateway"}, {"dest": "MME"}, {"port": 8081, "server_log": False})
+server = Server({"dest": "MME_GATEWAY"}, {"dest": "MME"}, { "server_log": True})
 
 # expose python function my_stfc_structure as ABAP function STFC_STRUCTURE, to be called by ABAP system
 server.add_function("STFC_STRUCTURE", my_stfc_structure)
@@ -49,6 +49,5 @@ def my_auth_check(func_name=False, request_context=None):
 
 input("Press Enter to stop server...")  # WPS110
 
-# stop server
-server.stop()
-print("Server stoped")
+# shutdown server
+server.close()

src/pyrfc/_cyrfc.pyx

@@ -1995,13 +1995,18 @@ cdef class Server:
         is delayed by the garbage collection, problems may occur when too many
         servers are registered.
         """
+
+        if self._server_handle == NULL:
+            return
+
         self.stop()
 
         # Server connection close
-        if self._server_handle != NULL:
-            with nogil:
-                RfcDestroyServer(self._server_handle, NULL)
-            self._server_handle = NULL
+        server_handle = self.server_handle
+        with nogil:
+            RfcDestroyServer(self._server_handle, NULL)
+        self._server_handle = NULL
+        _server_log("Server", f"{server_handle} closed")
 
         # Remove all installed server functions
         global server_functions
@@ -2010,7 +2015,6 @@ cdef class Server:
             if func_data["server"] != self:
                 after_remove[func_name] = func_data
         server_functions = after_remove
-        _server_log("Server", f"{self.server_handle} closed")
 
     def __dealloc__(self):
         self.close()