Fixed occasionally passing null to StringBuilder.Append (#11)

mysticmind · Aug 30, 2019 · 3768229 · 3768229
1 parent 6f7470e
commit 3768229
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/src/MysticMind.PostgresEmbed/Utils.cs b/src/MysticMind.PostgresEmbed/Utils.cs
@@ -147,8 +147,17 @@ public static ProcessResult RunProcess(string filename, List<string> args)
                 p.StartInfo.RedirectStandardOutput = true;
                 p.StartInfo.UseShellExecute = false;
                 p.EnableRaisingEvents = true;
-                p.OutputDataReceived += (sender, e) => outputBuilder.Append(e.Data);
-                p.ErrorDataReceived += (sender, e) => errorBuilder.Append(e.Data);
+                p.OutputDataReceived += (sender, e) =>
+                {
+                    if (!string.IsNullOrEmpty(e.Data))
+                        outputBuilder.Append(e.Data);
+                };
+
+                p.ErrorDataReceived += (sender, e) =>
+                {
+                    if (!string.IsNullOrEmpty(e.Data))
+                        errorBuilder.Append(e.Data);
+                };
 
                 p.StartInfo.FileName = filename;
                 p.StartInfo.Arguments = string.Join(" ", args);