fix AsyncClient::connect memory leak

serek4 · serek4 · commit 5b8a4ae35620 · 2024-02-05T13:20:31.000+01:00
[pull request 138](me-no-dev#138)
diff --git a/src/ESPAsyncTCP.cpp b/src/ESPAsyncTCP.cpp
@@ -264,6 +264,9 @@ bool AsyncClient::connect(IPAddress ip, uint16_t port){
   tcp_arg(pcb, this);
   tcp_err(pcb, &_s_error);
   size_t err = tcp_connect(pcb, &addr, port,(tcp_connected_fn)&_s_connected);
+  if (err != ERR_OK){
+    tcp_close(pcb);
+  }
   return (ERR_OK == err);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -264,6 +264,9 @@ bool AsyncClient::connect(IPAddress ip, uint16_t port){`
`264`	`264`	`tcp_arg(pcb, this);`
`265`	`265`	`tcp_err(pcb, &_s_error);`
`266`	`266`	`size_t err = tcp_connect(pcb, &addr, port,(tcp_connected_fn)&_s_connected);`
	`267`	`+ if (err != ERR_OK){`
	`268`	`+ tcp_close(pcb);`
	`269`	`+ }`
`267`	`270`	`return (ERR_OK == err);`
`268`	`271`	`}`
`269`	`272`