ClientContext:_write_some: release buffer once data really accepted f…

…or sending (#4265) may fix #1872
esp8266 · Feb 1, 2018 · 199fe0f · 199fe0f
1 parent 9ba8f77
commit 199fe0f
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/libraries/ESP8266WiFi/src/include/ClientContext.h b/libraries/ESP8266WiFi/src/include/ClientContext.h
@@ -447,11 +447,13 @@ class ClientContext
             }
             err_t err = tcp_write(_pcb, buf, next_chunk, TCP_WRITE_FLAG_COPY);
             DEBUGV(":wrc %d %d %d\r\n", next_chunk, will_send, (int) err);
-            _datasource->release_buffer(buf, next_chunk);
             if (err == ERR_OK) {
+                _datasource->release_buffer(buf, next_chunk);
                 _written += next_chunk;
                 need_output = true;
             } else {
+		// ERR_MEM(-1) is a valid error meaning
+		// "come back later". It leaves state() opened
                 break;
             }
             will_send -= next_chunk;