[LU-6021] 5 new static analysis issues in lnetconfig

Related — Wed, 30 Aug 2023 21:32:18 +0000

Found 5 new static analysis issues in 2.6.91-26-g6b0fa76:

Result of function that may return NULL will be dereferenced
- lnet/utils/cyaml/cyaml.c: in print_value, Pointer 'node' returned from call to function 'cYAML_ll_pop' at line 886 may be NULL and will be dereferenced at line 888.
Null pointer may be dereferenced
- lnet/utils/lnetconfig/liblnetconfig.c: in lustre_lnet_show_stats, Null pointer 'root' that comes from line 1120 may be dereferenced at line 1197 (after "goto out" in 1133).
Result of function that can return NULL may be dereferenced
- lnet/utils/lnetconfig/liblnetconfig.c: in lustre_lnet_show_stats, Pointer 'root' returned from call to function 'cYAML_create_object' at line 1136 may be NULL and may be dereferenced at line 1197.
Possible Buffer Overflow in Following String Operations
- lnet/utils/lnetconfig/liblnetconfig.c: in liblnetconfig.c, function 'strncpy' will fill whole buffer 'data.cfg_config_u.cfg_net.net_intf' of fixed size (128) with string value and will not leave place for NULL-terminator. Possible buffer boundaries violation in following string operations.
Pointer may be dereferenced after it was positively checked for NULL
- lnet/utils/lnetconfig/liblnetconfig.c: in lustre_lnet_show_stats, Pointer 'root' checked for NULL at line 1137 may be dereferenced at line 1197.

Whamcloud Community JIRA