#149 - Fixed index out of range issue with negative ids

credfeto · credfeto · commit fb572f87294d · 2019-08-21T10:31:12.000+01:00
diff --git a/src/Serilog.Extensions.Logging/Extensions/Logging/SerilogLogger.cs b/src/Serilog.Extensions.Logging/Extensions/Logging/SerilogLogger.cs
@@ -139,7 +139,7 @@ internal static LogEventProperty CreateEventIdProperty(EventId eventId)
 
             if (eventId.Id != 0)
             {
-                if (eventId.Id < LowEventIdValues.Length)
+                if (eventId.Id >= 0 && eventId.Id < LowEventIdValues.Length)
                     // Avoid some allocations
                     properties.Add(LowEventIdValues[eventId.Id]);
                 else

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,7 @@ internal static LogEventProperty CreateEventIdProperty(EventId eventId)`
`139`	`139`
`140`	`140`	`if (eventId.Id != 0)`
`141`	`141`	`{`
`142`		`- if (eventId.Id < LowEventIdValues.Length)`
	`142`	`+ if (eventId.Id >= 0 && eventId.Id < LowEventIdValues.Length)`
`143`	`143`	`// Avoid some allocations`
`144`	`144`	`properties.Add(LowEventIdValues[eventId.Id]);`
`145`	`145`	`else`