Merge pull request #3 from abelsensors/feature/VV-2986_Fix_watchdog_timeout_after_multipart_line_too_long

MarkDeBrabander1976 · web-flow · commit 6cf1aed525a2 · 2024-09-09T13:02:34.000+02:00
VV-2986 Also stop the peekBoundary() loop in nextField() when the pee…
diff --git a/src/HTTPMultipartBodyParser.cpp b/src/HTTPMultipartBodyParser.cpp
@@ -169,10 +169,14 @@ bool HTTPMultipartBodyParser::peekBoundary() {
 
 int32_t HTTPMultipartBodyParser::nextField() {
   fillBuffer(MAXLINESIZE);
-  while(!peekBoundary()) {
+  while (!peekBoundary()) {
+    if (peekBufferSize == 0) {
+      HTTPS_LOGE("Multipart missing last boundary1");
+      return -1;
+    }
     std::string dummy = readLine();
     if (endOfBody()) {
-      HTTPS_LOGE("Multipart missing last boundary");
+      HTTPS_LOGE("Multipart missing last boundary2");
       return -1;
     }
     fillBuffer(MAXLINESIZE);
