From 613f04c195049213b5941a593003d7548b52d3fc Mon Sep 17 00:00:00 2001
From: adams13x13 <adams13x13@users.noreply.github.com>
Date: Sun, 20 Aug 2017 20:52:19 +0200
Subject: [PATCH] Update WString.cpp

realloc() is called with newSize > 0 (at least 16), so newbuffer==0 means the old memory was not deallocated. Therefore, the pointer should still point to the old buffer. This change should resolve issue #3516.
---
 cores/esp8266/WString.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cores/esp8266/WString.cpp b/cores/esp8266/WString.cpp
index c66f5da95..acd62ce9b 100644
--- a/cores/esp8266/WString.cpp
+++ b/cores/esp8266/WString.cpp
@@ -159,7 +159,6 @@ unsigned char String::changeBuffer(unsigned int maxStrLen) {
         buffer = newbuffer;
         return 1;
     }
-    buffer = newbuffer;
     return 0;
 }