From d74628a00973c3f6408424ad421c269e35f0372e Mon Sep 17 00:00:00 2001 From: Simon Cropp Date: Thu, 20 Mar 2025 15:04:23 +1100 Subject: [PATCH] fix null ref in Audit fixes #4893 --- src/ServiceControl.Audit/Auditing/AuditIngestor.cs | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/ServiceControl.Audit/Auditing/AuditIngestor.cs b/src/ServiceControl.Audit/Auditing/AuditIngestor.cs index f783a61dea..8f6f453bdb 100644 --- a/src/ServiceControl.Audit/Auditing/AuditIngestor.cs +++ b/src/ServiceControl.Audit/Auditing/AuditIngestor.cs @@ -70,8 +70,7 @@ public async Task Ingest(List contexts, CancellationToken cancel Task Forward(IReadOnlyCollection messageContexts, string forwardingAddress, CancellationToken cancellationToken) { - var transportOperations = new TransportOperation[messageContexts.Count]; //We could allocate based on the actual number of ProcessedMessages but this should be OK - var index = 0; + var transportOperations = new List(); MessageContext anyContext = null; foreach (var messageContext in messageContexts) { @@ -90,13 +89,12 @@ Task Forward(IReadOnlyCollection messageContexts, string forward // Forwarded messages should last as long as possible outgoingMessage.Headers.Remove(Headers.TimeToBeReceived); - transportOperations[index] = new TransportOperation(outgoingMessage, new UnicastAddressTag(forwardingAddress)); - index++; + transportOperations.Add(new TransportOperation(outgoingMessage, new UnicastAddressTag(forwardingAddress))); } return anyContext != null ? messageDispatcher.Value.Dispatch( - new TransportOperations(transportOperations), + new TransportOperations(transportOperations.ToArray()), anyContext.TransportTransaction, cancellationToken) : Task.CompletedTask; }