Fix panic caused by nil filter function

Make sure we don't add nil to the filter list. Fixes google#1283
ben-clayton · Oct 30, 2017 · e3a7bdf · e3a7bdf
1 parent 4764f77
commit e3a7bdf
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/gapis/resolve/events.go b/gapis/resolve/events.go
@@ -45,7 +45,9 @@ func Events(ctx context.Context, p *path.Events) (*service.Events, error) {
 	filters := CommandFilters{filter}
 	for _, e := range extensions.Get() {
 		if f := e.EventFilter; f != nil {
-			filters = append(filters, CommandFilter(f(ctx, p)))
+			if filter := CommandFilter(f(ctx, p)); filter != nil {
+				filters = append(filters, filter)
+			}
 		}
 	}
 	filter = filters.All