Merge pull request #40 from microsoft/jschick/fix_description_indexerror

Added property count check in FormatDescription to prevent IndexOutOfRange error
microsoft · May 16, 2023 · c0a807b · c0a807b
2 parents e9991a2 + b247bba
commit c0a807b
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/EventLogExpert.Library/EventResolvers/EventResolverBase.cs b/src/EventLogExpert.Library/EventResolvers/EventResolverBase.cs
@@ -48,6 +48,9 @@ protected EventResolverBase(Action<string> tracer)
 
                 sb.Append(description.AsSpan(lastIndex, matches[i].Index - lastIndex));
                 var propIndex = int.Parse(matches[i].Value.Trim(new[] { '{', '}', '%' }));
+
+                if (propIndex - 1 >= eventRecord.Properties.Count) { return "Unable to format description"; }
+
                 var propValue = eventRecord.Properties[propIndex - 1].Value;
                 if (propValue is DateTime)
                 {