cmd/ingest: fix nil pointer deref

When the previous block returns an error, ingestRes may be nil, which caused a panic. Signed-off-by: Tomás Senart <[email protected]>
axiomhq · Aug 30, 2023 · 81b964d · 81b964d
1 parent 2d5cdac
commit 81b964d
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/internal/cmd/ingest/ingest.go b/internal/cmd/ingest/ingest.go
@@ -299,7 +299,11 @@ func run(ctx context.Context, opts *options, flushEverySet bool) error {
 		} else {
 			ingestRes, err = ingestReader(ctx, client, r, typ, opts)
 		}
-		res.Add(ingestRes)
+
+		// Error handling below, so we need to check for nil.
+		if ingestRes != nil {
+			res.Add(ingestRes)
+		}
 
 		if err != nil && !errors.Is(err, context.Canceled) {
 			_ = rc.Close()