From 35879fc3b13c9dee5de9883e3e26927ee686e5bc Mon Sep 17 00:00:00 2001 From: Ethan Chiu Date: Wed, 8 Jul 2020 12:33:22 -0400 Subject: [PATCH] Fix null errors --- src/Runner.Worker/ActionManifestManager.cs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/Runner.Worker/ActionManifestManager.cs b/src/Runner.Worker/ActionManifestManager.cs index a0071c9bf..a9eac9d9d 100644 --- a/src/Runner.Worker/ActionManifestManager.cs +++ b/src/Runner.Worker/ActionManifestManager.cs @@ -74,7 +74,8 @@ namespace GitHub.Runner.Worker var fileId = templateContext.GetFileId(fileRelativePath); // Add this file to the FileTable in executionContext if it hasn't been added already - if (fileId > executionContext.FileTable.Count) + // we use > since fileID is zero indexed + if (fileId > executionContext.FileTable?.Count) { executionContext.FileTable.Add(fileRelativePath); } @@ -297,7 +298,7 @@ namespace GitHub.Runner.Worker } // Add the file table from the Execution Context - if (executionContext.FileTable.Count > 0) + if (executionContext.FileTable?.Count > 0) { for (var i = 0; i < executionContext.FileTable.Count; i++) {