Check FILE* is not NULL before calling fileno function

yury · yury · commit eb1b3a08af67 · 2021-10-05T20:08:59.000+03:00
diff --git a/ios_system.m b/ios_system.m
@@ -1694,9 +1694,9 @@ int ios_dup2(int fd1, int fd2)
             case 2: child_stderr = stream1; return fd2;
         }
     }
-    if ((fd2 == 0) || (fd2 == fileno(thread_stdin))) { child_stdin = fdopen(fd1, "rb"); }
-    else if ((fd2 == 1) || (fd2 == fileno(thread_stdout))) { child_stdout = fdopen(fd1, "wb"); }
-    else if ((fd2 == 2) || (fd2 == fileno(thread_stderr))) {
+    if ((fd2 == 0) || (thread_stdin != NULL && fd2 == fileno(thread_stdin))) { child_stdin = fdopen(fd1, "rb"); }
+    else if ((fd2 == 1) || (thread_stdout != NULL && fd2 == fileno(thread_stdout))) { child_stdout = fdopen(fd1, "wb"); }
+    else if ((fd2 == 2) || (thread_stderr != NULL && fd2 == fileno(thread_stderr))) {
         if ((child_stdout != NULL) && (fileno(child_stdout) == fd1)) child_stderr = child_stdout;
         else child_stderr = fdopen(fd1, "wb"); }
     else if (fd1 != fd2) {

Original file line number	Diff line number	Diff line change
`@@ -1694,9 +1694,9 @@ int ios_dup2(int fd1, int fd2)`
`1694`	`1694`	`case 2: child_stderr = stream1; return fd2;`
`1695`	`1695`	`}`
`1696`	`1696`	`}`
`1697`		`- if ((fd2 == 0) \|\| (fd2 == fileno(thread_stdin))) { child_stdin = fdopen(fd1, "rb"); }`
`1698`		`- else if ((fd2 == 1) \|\| (fd2 == fileno(thread_stdout))) { child_stdout = fdopen(fd1, "wb"); }`
`1699`		`- else if ((fd2 == 2) \|\| (fd2 == fileno(thread_stderr))) {`
	`1697`	`+ if ((fd2 == 0) \|\| (thread_stdin != NULL && fd2 == fileno(thread_stdin))) { child_stdin = fdopen(fd1, "rb"); }`
	`1698`	`+ else if ((fd2 == 1) \|\| (thread_stdout != NULL && fd2 == fileno(thread_stdout))) { child_stdout = fdopen(fd1, "wb"); }`
	`1699`	`+ else if ((fd2 == 2) \|\| (thread_stderr != NULL && fd2 == fileno(thread_stderr))) {`
`1700`	`1700`	`if ((child_stdout != NULL) && (fileno(child_stdout) == fd1)) child_stderr = child_stdout;`
`1701`	`1701`	`else child_stderr = fdopen(fd1, "wb"); }`
`1702`	`1702`	`else if (fd1 != fd2) {`