Fix a bug in ProcessRunner:

Pass empty stdin to the child processes instead of allowing them to inherit stdin from the parent process.

Author: PolarGoose

src/DOpusScriptingExtensions/DOpusScriptingExtensions.vcxproj

@@ -200,11 +200,11 @@
   </Target>
   <!-- Download ExifTool binaries -->
   <Target Name="DownloadExifTool" AfterTargets="Build" Condition="!Exists('$(BuildFolder)exiftool.zip')">
-    <DownloadFile SourceUrl="https://exiftool.org/exiftool-13.30_64.zip" DestinationFolder="$(BuildFolder)" DestinationFileName="exiftool.zip" />
+    <DownloadFile SourceUrl="https://netix.dl.sourceforge.net/project/exiftool/exiftool-13.33_64.zip?viasf=1" DestinationFolder="$(BuildFolder)" DestinationFileName="exiftool.zip" />
     <Unzip SourceFiles="$(BuildFolder)exiftool.zip" DestinationFolder="$(BuildFolder)" />
   </Target>
   <Target Name="CopyExifToolToOutputFolder" AfterTargets="DownloadExifTool" Condition="!Exists('$(OutputPath)exiftool')">
-    <Exec Command="robocopy &quot;$(BuildFolder)exiftool-13.30_64&quot; &quot;$(OutputPath)exiftool&quot; /e &gt; nul" ContinueOnError="True" />
+    <Exec Command="robocopy &quot;$(BuildFolder)exiftool-13.33_64&quot; &quot;$(OutputPath)exiftool&quot; /e &gt; nul" ContinueOnError="True" />
     <Exec Command="ren &quot;$(OutputPath)exiftool\exiftool(-k).exe&quot; &quot;exiftool.exe&quot;" />
   </Target>
   <!-- Download language files for MediaInfoLib -->
@@ -215,4 +215,4 @@
   <Target Name="CopyMediaInfoLanguagesToOutputFolder" AfterTargets="DownloadMediaInfo" Condition="!Exists('$(OutputPath)MediaInfoLanguages')">
     <Exec Command="robocopy &quot;$(BuildFolder)MediaInfo-25.04/Source/Resource/Plugin/Language&quot; &quot;$(OutputPath)MediaInfoLanguages&quot; /e &gt; nul" ContinueOnError="True" />
   </Target>
-</Project>
+</Project>

src/DOpusScriptingExtensions/ProcessRunner/ProcessRunner.h

@@ -29,13 +29,11 @@ class ATL_NO_VTABLE CProcessRunner :
   static std::tuple<std::wstring, std::wstring, int> RunProcess(const boost::filesystem::path& exePath,
                                                                 const std::vector<std::string>& args,
                                                                 const boost::filesystem::path& workingDirectory) {
-    if (!boost::filesystem::exists(exePath))
-    {
+    if (!boost::filesystem::exists(exePath)) {
       THROW_WEXCEPTION(L"The executable not found '{}'", exePath);
     }
 
-    if (!boost::process::v2::environment::detail::is_exec_type(exePath.c_str()))
-    {
+    if (!boost::process::v2::environment::detail::is_exec_type(exePath.c_str())) {
       THROW_WEXCEPTION(L"The file '{}' is not executable", exePath);
     }
 
@@ -44,7 +42,7 @@ class ATL_NO_VTABLE CProcessRunner :
     boost::asio::readable_pipe stdOutPipe(ctx), stdErrPipe(ctx);
     boost::process::v2::process proc(
       ctx, boost::filesystem::canonical(exePath), args,
-      boost::process::v2::process_stdio{ {}, stdOutPipe, stdErrPipe },
+      boost::process::v2::process_stdio{ nullptr, stdOutPipe, stdErrPipe },
       boost::process::v2::windows::show_window_hide,
       boost::process::process_start_dir(workingDirectory));
 
@@ -56,8 +54,7 @@ class ATL_NO_VTABLE CProcessRunner :
           if (!ec) {
             readStdOut();
           }
-        }
-      );
+        });
       };
 
     std::function<void()> readStdErr;
@@ -68,8 +65,7 @@ class ATL_NO_VTABLE CProcessRunner :
           if (!ec) {
             readStdErr();
           }
-        }
-      );
+        });
       };
 
     readStdOut();
@@ -81,7 +77,7 @@ class ATL_NO_VTABLE CProcessRunner :
 
     return { ToUtf16({ GetDataPointer(outBuffer), outBuffer.size() }),
              ToUtf16({ GetDataPointer(errBuffer), errBuffer.size() }),
-            exitCode };
+             exitCode };
   }
 };