Fix wrong variable was used in the loop. Causing potential memory leaks.

syoyo · syoyo · commit 3627ab306059 · 2023-06-06T02:17:12.000+09:00
Issue found by Code scanning: https://github.com/syoyo/tinyexr/security/code-scanning/105
diff --git a/tinyexr.h b/tinyexr.h
@@ -8638,7 +8638,7 @@ int ParseEXRMultipartHeaderFromMemory(EXRHeader ***exr_headers,
     if (!ConvertHeader(exr_header, infos[i], &warn, &_err)) {
 
       // Free malloc-allocated memory here.
-      for (size_t k = 0; k < infos[i].attributes.size(); i++) {
+      for (size_t k = 0; k < infos[i].attributes.size(); k++) {
         if (infos[i].attributes[k].value) {
           free(infos[i].attributes[k].value);
         }

Original file line number	Diff line number	Diff line change
`@@ -8638,7 +8638,7 @@ int ParseEXRMultipartHeaderFromMemory(EXRHeader ***exr_headers,`
`8638`	`8638`	`if (!ConvertHeader(exr_header, infos[i], &warn, &_err)) {`
`8639`	`8639`
`8640`	`8640`	`// Free malloc-allocated memory here.`
`8641`		`- for (size_t k = 0; k < infos[i].attributes.size(); i++) {`
	`8641`	`+ for (size_t k = 0; k < infos[i].attributes.size(); k++) {`
`8642`	`8642`	`if (infos[i].attributes[k].value) {`
`8643`	`8643`	`free(infos[i].attributes[k].value);`
`8644`	`8644`	`}`