CodeQL findings — Svc/TlmPacketizer

devel @ d3003862b5fd · generated 2026-11-03T21:41:25Z
FileLineRuleSeverityMessage
Svc/TlmPacketizer/TlmPacketizer.cpp:615615cpp/fprime/checking-parameter-valuesmediumParameter pkt is used as an array index or divisor without first being checked (e.g. with FW_ASSERT or a condition).
Svc/TlmPacketizer/TlmPacketizer.cpp:643643cpp/fprime/jpl-c/use-of-assertions-non-booleanmediumAssertions should be defined as Boolean tests.
Svc/TlmPacketizer/TlmPacketizer.cpp:797797cpp/fprime/checking-parameter-valuesmediumParameter pkt is used as an array index or divisor without first being checked (e.g. with FW_ASSERT or a condition).
Svc/TlmPacketizer/TlmPacketizer.cpp:6161cpp/jpl-c/function-size-limitslowsetPacketList has too many lines (118, while 60 are allowed).
Svc/TlmPacketizer/TlmPacketizer.cpp:240240cpp/jpl-c/function-size-limitslowTlmGet_handler has too many lines (61, while 60 are allowed).
Svc/TlmPacketizer/TlmPacketizer.cpp:302302cpp/jpl-c/function-size-limitslowRun_handler has too many lines (113, while 60 are allowed).
Svc/TlmPacketizer/TlmPacketizer.cpp:454454cpp/fprime/jpl-c/use-of-assertions-densitylowAll functions of more than 10 lines should have at least one assertion, unless they handle errors by returning a status enum or bool, or by emitting a WARNING/FATAL event.
Svc/TlmPacketizer/TlmPacketizer.cpp:524524cpp/fprime/jpl-c/use-of-assertions-densitylowAll functions of more than 10 lines should have at least one assertion, unless they handle errors by returning a status enum or bool, or by emitting a WARNING/FATAL event.
Svc/TlmPacketizer/TlmPacketizer.cpp:549549cpp/fprime/jpl-c/use-of-assertions-densitylowAll functions of more than 10 lines should have at least one assertion, unless they handle errors by returning a status enum or bool, or by emitting a WARNING/FATAL event.
Svc/TlmPacketizer/TlmPacketizer.cpp:549549cpp/jpl-c/function-size-limitslowOVERRIDE_PACKET_RATES_cmdHandler has too many parameters (7, while 6 are allowed).
Svc/TlmPacketizer/TlmPacketizer.cpp:612612cpp/fprime/jpl-c/use-of-assertions-densitylowAll functions of more than 10 lines should have at least one assertion, unless they handle errors by returning a status enum or bool, or by emitting a WARNING/FATAL event.

Dismissed findings (1)

FileLineRuleDismissal reasonComment
Svc/TlmPacketizer/TlmPacketizer.cpp505cpp/jpl-c/function-size-limitswon't fixOnly 5 command arguments, the other two are default/autocode