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 <commits@restyled.io>
project-chip · Sep 22, 2023 · 8481f3b · 8481f3b
1 parent 482de7c
commit 8481f3b
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);