nasa / CF

The Core Flight System (cFS) CFDP application.
Apache License 2.0
86 stars 46 forks source link

Registering 100+ event filters w/ 0x0000 values serves no purpose (and overflows a typical filter buffer) #260

Closed skliper closed 2 years ago

skliper commented 2 years ago

Checklist (Please check before submitting)

Is your feature request related to a problem? Please describe. Registering event filters just to register event filters doesn't do anything but load the system. If an event needs to be filtered operationally, just use the add filter command. Note default CFE_PLATFORM_EVS_MAX_EVENT_FILTERS is 8, so the rest would get discarded and cause a filter buffer overflow event.

Describe the solution you'd like Remove filter registration for those set to 0x0000 (all of them!) Add operationally if/when needed, or if eventually an actual filter needs to be applied then add it.

Describe alternatives you've considered None

Additional context None

Requester Info Jacob Hageman - NASA/GSFC