Hashtable size in configuration is not checked

[constcast]

Hashtablesize is a uint32_t. No boundary checks are performed when reading hashBits from the configuration file. This will result in overflows and even segmentation faults (e.g. if hashbits == 63).

[nickbroon]

This looks to be referring to several use of uint32_t hashBits:

src/modules/analysis/RBSWormDetectorCfg.cpp: hashBits = getInt("hashbits");
src/modules/analysis/TRWPortscanDetectorCfg.cpp: hashBits = getInt("hashbits");
src/modules/analysis/AutoFocusCfg.cpp: hashBits = getInt("hashbits");

The aggregator has a similar issue:

src/modules/ipfix/aggregator/AggregatorBaseCfg.cpp: htableBits = getInt("hashtableBits", HT_DEFAULT_BITSIZE);

as htableBits is a uint8_t.