[net][at] Modify AT socket object release judgment

Lawlieta · Lawlieta · commit 33a5cf15ee01 · 2018-08-31T14:45:40.000+08:00
diff --git a/components/net/at/at_socket/at_socket.c b/components/net/at/at_socket/at_socket.c
@@ -404,11 +404,13 @@ int at_closesocket(int socket)
         if (at_dev_ops->at_closesocket(socket) != 0)
         {
             LOG_E("AT socket (%d) closesocket failed!", socket);
+            free_socket(sock);
             return -1;
         }
     }
 
-    return free_socket(sock);
+    free_socket(sock); 
+    return 0;
 }
 
 int at_shutdown(int socket, int how)
@@ -431,11 +433,13 @@ int at_shutdown(int socket, int how)
         if (at_dev_ops->at_closesocket(socket) != 0)
         {
             LOG_E("AT socket (%d) shutdown failed!", socket);
+            free_socket(sock);
             return -1;
         }
     }
 
-    return free_socket(sock);
+    free_socket(sock);
+    return 0;
 }
 
 int at_bind(int socket, const struct sockaddr *name, socklen_t namelen)

Original file line number	Diff line number	Diff line change
`@@ -404,11 +404,13 @@ int at_closesocket(int socket)`
`404`	`404`	`if (at_dev_ops->at_closesocket(socket) != 0)`
`405`	`405`	`{`
`406`	`406`	`LOG_E("AT socket (%d) closesocket failed!", socket);`
	`407`	`+ free_socket(sock);`
`407`	`408`	`return -1;`
`408`	`409`	`}`
`409`	`410`	`}`
`410`	`411`
`411`		`- return free_socket(sock);`
	`412`	`+ free_socket(sock);`
	`413`	`+ return 0;`
`412`	`414`	`}`
`413`	`415`
`414`	`416`	`int at_shutdown(int socket, int how)`
`@@ -431,11 +433,13 @@ int at_shutdown(int socket, int how)`
`431`	`433`	`if (at_dev_ops->at_closesocket(socket) != 0)`
`432`	`434`	`{`
`433`	`435`	`LOG_E("AT socket (%d) shutdown failed!", socket);`
	`436`	`+ free_socket(sock);`
`434`	`437`	`return -1;`
`435`	`438`	`}`
`436`	`439`	`}`
`437`	`440`
`438`		`- return free_socket(sock);`
	`441`	`+ free_socket(sock);`
	`442`	`+ return 0;`
`439`	`443`	`}`
`440`	`444`
`441`	`445`	`int at_bind(int socket, const struct sockaddr *name, socklen_t namelen)`