diag_states_summary # States and # States twice (explain one is per protocol)
diag_states_summary is not intuitive as each table has two columns
with same "# States" header.
first is for $ipinfo['seen'];
and second is for $protoinfo['seen'];
Maybe have the header say "Protocol counts" over the last three fields
or add a documentation line at top (or bottom) to explain that.
(My list of bugs is at http://reedmedia.net/books/pfsense/bugs-20160813.html )