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 esp8266#3516.
d-a-v · Sep 29, 2017 · 47baf29 · 47baf29
1 parent 92978a7
commit 47baf29
Showing 1 changed file with 0 additions and 1 deletion.
diff --git 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;
 }