CheckWriteRead should fail if payloadBase38Representation fails to pa…

…rse the QRCode (#27554) * CheckWriteRead should fail if payloadBase38Representation fails to parse the QRCode * Restyled by clang-format --------- Co-authored-by: Restyled.io <[email protected]>
project-chip · Feb 29, 2024 · 4399491 · 4399491
1 parent 7c99e89
commit 4399491
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/setup_payload/tests/TestHelpers.h b/src/setup_payload/tests/TestHelpers.h
@@ -141,7 +141,12 @@ inline bool CheckWriteRead(SetupPayload & inPayload, bool allowInvalidPayload =
     memset(optionalInfo, 0xFF, sizeof(optionalInfo));
     auto generator = QRCodeSetupPayloadGenerator(inPayload);
     generator.SetAllowInvalidPayload(allowInvalidPayload);
-    generator.payloadBase38Representation(result, optionalInfo, sizeof(optionalInfo));
+    CHIP_ERROR err = generator.payloadBase38Representation(result, optionalInfo, sizeof(optionalInfo));
+
+    if (err != CHIP_NO_ERROR)
+    {
+        return false;
+    }
 
     outPayload = {};
     QRCodeSetupPayloadParser(result).populatePayload(outPayload);