Adds required comparison function to order_vector_of_count to avoid potential runtime error

[epeterso]

Fixes an edge case runtime error observed when monitoring certain network traffic that causes the following reporter error:

Reporter::ERROR comparison function required for order() with non-numeric types (order(FINGERPRINT::input))

[J0eJ0h]

I'm not sure how a vector of count ends up holding something non-numeric without a bug elsewhere. This is going to mask that bug without fixing it.

[vlvkobal]

Thanks for pointing this out. I checked the Zeek scripts and couldn’t find a case where a non-numeric value could end up in a vector of count. The vectors here are strictly typed and filled only from count values.

The error seems to come from Zeek requiring an explicit comparison function for order() in some situations, rather than from incorrect data. Because of that, this change should be safe and not hide another bug.

I’m not a Zeek expert, but I’m fine with merging this unless there is a concrete repro that points to a real type issue.