Fix JSON serialization in “node-wifi-mqtt” again, there was an off-by…

…-one-error. Thanks, Giuseppe!
hiveeyes · Mar 31, 2017 · 9c86aa1 · 9c86aa1
1 parent 34c6ffa
commit 9c86aa1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -17,6 +17,7 @@ in-progress
 - Improve firmwares :ref:`node-wifi-mqtt-homie` and :ref:`node-wifi-mqtt-homie-battery`
 - :ref:`node-wifi-mqtt`: Fix JSON serialization: Transmit sensor readings as float values. Thanks, Matthias!
 - :ref:`node-wifi-mqtt`: Improve configuration and documentation
+- :ref:`node-wifi-mqtt`: Fix JSON serialization again, there was an off-by-one-error. Thanks, Giuseppe!
 
 
 2017-03-17 0.13.0

diff --git a/node-wifi-mqtt/node-wifi-mqtt.ino b/node-wifi-mqtt/node-wifi-mqtt.ino
@@ -365,9 +365,8 @@ void loop() {
   getHumidityTemperature();
 
 
-  // Prepare data, build JSON object
-
-  StaticJsonBuffer<256> jsonBuffer;
+  // Build JSON object containing sensor readings
+  StaticJsonBuffer<512> jsonBuffer;
 
   JsonObject& root = jsonBuffer.createObject();
   root["weight"]                    = String(weightChar).toFloat();
@@ -384,7 +383,8 @@ void loop() {
 
 
   // Serialize data
-  char payload[256];
+  int json_length = root.measureLength();
+  char payload[json_length+1];
   root.printTo(payload, sizeof(payload));