fix: correct base64 data URI segment count validation in readImage

ionkorol · ionkorol · commit c5f18b498760 · 2026-02-15T12:10:54.000-05:00
diff --git a/packages/react-native-executorch/common/rnexecutorch/data_processing/ImageProcessing.cpp b/packages/react-native-executorch/common/rnexecutorch/data_processing/ImageProcessing.cpp
@@ -86,7 +86,7 @@ cv::Mat readImage(const std::string &imageURI) {
     while (std::getline(uriStream, stringData, ',')) {
       ++segmentIndex;
     }
-    if (segmentIndex != 1) {
+    if (segmentIndex != 2) {
       throw RnExecutorchError(RnExecutorchErrorCode::FileReadFailed,
                               "Read image error: invalid base64 URI");
     }

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ cv::Mat readImage(const std::string &imageURI) {`
`86`	`86`	`while (std::getline(uriStream, stringData, ',')) {`
`87`	`87`	`++segmentIndex;`
`88`	`88`	`}`
`89`		`- if (segmentIndex != 1) {`
	`89`	`+ if (segmentIndex != 2) {`
`90`	`90`	`throw RnExecutorchError(RnExecutorchErrorCode::FileReadFailed,`
`91`	`91`	`"Read image error: invalid base64 URI");`
`92`	`92`	`}`