Merge pull request #619 from mcnewton/jsonbuffer

don't process JSON past the end of the buffer

Author: matjack1

src/config.esp

@@ -12,7 +12,7 @@ bool ICACHE_FLASH_ATTR loadConfiguration(Config &config)
 	std::unique_ptr<char[]> buf(new char[size]);
 	configFile.readBytes(buf.get(), size);
 	DynamicJsonDocument json(2048);
-	auto error = deserializeJson(json, buf.get());
+	auto error = deserializeJson(json, buf.get(), size);
 	if (error)
 	{
 #ifdef DEBUG

src/mqtt.esp

@@ -372,7 +372,7 @@ void getUserList()
 		std::unique_ptr<char[]> buf(new char[size]);
 		f.readBytes(buf.get(), size);
 		DynamicJsonDocument json(512);
-		auto error = deserializeJson(json, buf.get());
+		auto error = deserializeJson(json, buf.get(), size);
 		if (!error)
 		{
 			mqttPublishEvent(&json);

src/rfid.esp

@@ -70,7 +70,7 @@ void wiegandRead()
 				f.readBytes(buf.get(), size);
 				f.close();
 				DynamicJsonDocument json(512);
-				auto error = deserializeJson(json, buf.get());
+				auto error = deserializeJson(json, buf.get(), size);
 				if (error)
 				{
 					processingState = notValid;
@@ -299,7 +299,8 @@ void rfidProcess()
 	f.readBytes(buf.get(), size);
 	f.close();
 	DynamicJsonDocument json(512);
-	auto error = deserializeJson(json, buf.get());
+	auto error = deserializeJson(json, buf.get(), size);
+
 	if (error)
 	{
 		processingState = notValid;

src/wsResponses.esp

@@ -21,7 +21,7 @@ void ICACHE_FLASH_ATTR sendUserList(int page, AsyncWebSocketClient *client)
 			std::unique_ptr<char[]> buf(new char[size]);
 			f.readBytes(buf.get(), size);
 			DynamicJsonDocument json(512);
-			auto error = deserializeJson(json, buf.get());
+			auto error = deserializeJson(json, buf.get(), size);
 			if (!error)
 			{
 				String username = json["user"];
@@ -199,4 +199,4 @@ void ICACHE_FLASH_ATTR sendTime(AsyncWebSocketClient *client)
 			ws.textAll(buffer);
 		}
 	}
-}
+}