mirror of
https://github.com/ntop/ntopng.git
synced 2026-10-21 13:57:02 +00:00
Rework the `cppcheck:` recipe so it is usable in practice:
- `--enable=all` -> `--enable=warning,performance,portability`
(drops the `style`/`information`/`unusedFunction` groups, which produced
hundreds of opinion-level hits)
- drop `--force` (was re-analyzing every file once per #ifdef combination;
single files took 40+ min). Analyzes the default build config only.
- add `-j $(shell nproc)` + `--cppcheck-build-dir=.cppcheck-build` for
parallelism and incremental caching; `mkdir -p` the cache dir in the recipe
- suppress checks that are noise for this codebase:
* useInitializationList - micro perf, POD members
* duplInheritedMember - intentional non-virtual name shadowing
(operator[], swap, enqueue, incUses, lua, ...)
* noCopyConstructor / noOperatorEq - the flagged classes
(NetworkInterface, Ntop, Flow, ...) are non-copyable by design and
always held by pointer; the fix cppcheck suggests (add operator=)
is meaningless for them
- .gitignore: ignore the new .cppcheck-build/ cache directory
FlowAlertsLoader::registerRisk (src/FlowAlertsLoader.cpp)
Missing `else`: `alert_to_risk[alert_type.id] = risk` was executed even when
`alert_type.id >= MAX_DEFINED_FLOW_ALERT_TYPE` -> out-of-bounds array write.
The sibling registerAlert() already had the `else`.
Uninitialized ndpi_protocol structs (src/NetworkInterface.cpp x6,
src/LuaEngineInterface.cpp x1)
Local `ndpi_protocol` values were only partially initialized (proto.* set,
but breed/fpc/state/category/custom_category_userdata left indeterminate)
and then passed by value to get_ndpi_full_proto_name() /
get_ndpi_proto_category(). Added `memset(&x, 0, sizeof(x))` at each
declaration, matching the existing convention elsewhere in the tree.
Uninitialized locals
- src/ParserInterface.cpp: `src2dst_direction` was read on the
`flow == NULL` path (getFlow() not called). Initialize to a defined value;
the value is not otherwise observable on that path.
- include/Bitmap.h (toHexString): if `buf_len <= 0` the first loop never
runs and `strlen(buf)` reads uninitialized memory; also `strlen(buf) - 2`
underflows (unsigned) when the string is shorter than 2 bytes. Null-
terminate up front and hoist strlen into a `size_t`, iterate with
`pos + 2 < len`.
src/NetworkInterface.cpp incNumHosts()/decNumHosts()
`if (isShuttingDown() || (!host->isUnicastHost())) return;` - the second
term is always false, the function already returned on `!isUnicastHost()`
a few lines above. Dropped it.
include/NetworkInterface.h getActiveFlowsStats()
Declaration and definition disagreed on the first two parameter *names*
(`stats, status_stats` vs `ndpi_stats, stats`); types matched. Renamed the
header params to match the implementation.
No behavioural change; each argument was already non-negative.
- Interface id (`get_id()` returns int; kept int because 3 call sites test
`== -1`) formatted with %u in redis-key macros -> cast the argument
`(u_int)` at the call site:
include/AutonomousSystem.h, include/Country.h,
include/ObservationPoint.h, include/VLAN.h,
src/BroadcastDomains.cpp (x2), src/HostPools.cpp, src/LocalHost.cpp,
src/LuaEngineInterface.cpp (x3), src/NetworkInterface.cpp (x2),
src/SyslogParserInterface.cpp
- src/HTTPserver.cpp: `%d` with get_http_port()/get_https_port() (u_int)
-> `%u`; the scanf-width builder `"%%%u[^|]..."` fed `int` macros -> `%d`
- src/Flow.cpp: `"%u.%u.%u.%u"` with `int` octets from a `%d` sscanf -> `%d`;
`%u` (arg 13) with get_protocol_category() enum -> cast `(u_int)`
- src/MacManufacturers.cpp: `int _mac[3]` scanned with `%02x`
(wants `unsigned int*`) -> declare `u_int _mac[3]`
- src/StatsManager.cpp: `%lu` with `static_cast<long int>(key)` epoch values
-> `%ld` (respects the explicit signed cast)
- src/Trace.cpp: `"%s.%u"` with loop `int i` / `i + 1` -> `%d`
- src/Prefs.cpp: `%u` in usage() help text fed CONST_DEFAULT_* /
MAX_NUM_* / CLICKHOUSE_MAX_* int macros -> cast `(u_int)`
std::string / std::vector parameters taken by value, changed to `const &`
(declaration and definition kept in sync; none are virtual or overrides, so
callers are source-compatible):
- AggregatedFlowsStats::setInfoKey
- Flow::setL7JSON
- CustomFlowLuaScriptAlert::setAlertMessage
- CustomHostLuaScriptAlert::setAlertMessage, ctor (_msg)
- ScanRealtimeAlert ctor (_alerts, std::vector)
- CustomHostLuaScript::allocAlert (_msg)
- OtherAlertableEntity::getAlertCachedValue / setAlertCacheValue /
triggerAlert / releaseAlert (key, value)
- HostAlert::init and HostAlert(HostCheckID, ...) ctor (_check_name)
- Ntop::luaFlowCheckInfo / luaHostCheckInfo (check_name)
- Ntop::incBlacklisHits (listname)
- Ntop::setLuaCache (key, val)
109 lines
3.5 KiB
C++
109 lines
3.5 KiB
C++
/*
|
|
*
|
|
* (C) 2013-26 - ntop.org
|
|
*
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License as published by
|
|
* the Free Software Foundation; either version 3 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program; if not, write to the Free Software Foundation,
|
|
* Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
|
|
*
|
|
*/
|
|
|
|
#ifndef _OBSERVATION_POINT_H
|
|
#define _OBSERVATION_POINT_H
|
|
|
|
#include "ntop_includes.h"
|
|
|
|
class Score;
|
|
|
|
struct ndpi_in6_addr_less {
|
|
bool operator()(const struct ndpi_in6_addr& a, const struct ndpi_in6_addr& b) const {
|
|
return memcmp(&a, &b, sizeof(struct ndpi_in6_addr)) < 0;
|
|
}
|
|
};
|
|
|
|
class ObservationPoint : public GenericHashEntry,
|
|
public GenericTrafficElement,
|
|
public Score {
|
|
private:
|
|
/* Note: country name can be more then 2 chars, see
|
|
* https://www.iso.org/iso-3166-country-codes.html
|
|
*/
|
|
u_int16_t obs_point;
|
|
NetworkStats dirstats;
|
|
u_int64_t num_flows;
|
|
std::set<struct ndpi_in6_addr, ndpi_in6_addr_less> exporter_list;
|
|
std::atomic<bool> delete_requested;
|
|
bool remove_entry;
|
|
|
|
inline void incSentStats(time_t t, u_int64_t num_pkts, u_int64_t num_bytes) {
|
|
if (first_seen == 0)
|
|
first_seen = t, last_seen = iface->getTimeLastPktRcvd();
|
|
sent.incStats(t, num_pkts, num_bytes);
|
|
}
|
|
|
|
inline void incRcvdStats(time_t t, u_int64_t num_pkts, u_int64_t num_bytes) {
|
|
rcvd.incStats(t, num_pkts, num_bytes);
|
|
}
|
|
|
|
public:
|
|
ObservationPoint(NetworkInterface* _iface, u_int16_t obs_point);
|
|
~ObservationPoint();
|
|
|
|
void set_hash_entry_state_idle();
|
|
bool is_hash_entry_state_idle_transition_ready();
|
|
|
|
inline u_int64_t getNumFlows() { return num_flows; }
|
|
inline u_int32_t getNumHosts() { return getUses(); }
|
|
inline u_int32_t key() { return obs_point; }
|
|
inline u_int32_t getObsPoint() { return obs_point; }
|
|
inline void addProbeIp(struct ndpi_in6_addr* probe_ip) {
|
|
exporter_list.insert(*probe_ip);
|
|
}
|
|
|
|
bool equal(u_int16_t _obs_point) { return (obs_point == _obs_point); }
|
|
inline bool equal(ObservationPoint* _obs_point) {
|
|
return (obs_point == _obs_point->key());
|
|
}
|
|
|
|
inline void incFlows() { num_flows++; }
|
|
|
|
inline void incStats(time_t when, u_int16_t proto_id, u_int64_t sent_packets,
|
|
u_int64_t sent_bytes, u_int64_t rcvd_packets,
|
|
u_int64_t rcvd_bytes) {
|
|
if (!remove_entry) {
|
|
if (ndpiStats || (ndpiStats = new nDPIStats()))
|
|
ndpiStats->incStats(when, proto_id, sent_packets, sent_bytes,
|
|
rcvd_packets, rcvd_bytes);
|
|
incSentStats(when, sent_packets, sent_bytes);
|
|
incRcvdStats(when, rcvd_packets, rcvd_bytes);
|
|
}
|
|
}
|
|
|
|
virtual void updateStats(const struct timeval* tv);
|
|
|
|
void lua(lua_State* vm, DetailsLevel details_level, bool asListElement);
|
|
|
|
inline char* getSerializationKey(char* buf, u_int bufsize) {
|
|
snprintf(buf, bufsize, OBS_POINT_SERIALIZED_KEY, (u_int)iface->get_id(),
|
|
obs_point);
|
|
return (buf);
|
|
}
|
|
|
|
inline void setDeleteRequested(bool to_be_deleted) {
|
|
delete_requested = to_be_deleted;
|
|
}
|
|
inline void deleteObsStats() { remove_entry = true; }
|
|
};
|
|
|
|
#endif /* _OBSERVATION_POINT_H */
|