This cleanup patchset addresses several high and medium-impact Coverity defects.
Unit tests have been successfully executed via `make check` and they successfully passed. The list of revisions so far: v2: Fix some apply-robot checkpatch errors and warnings. v3: Fix some apply-robot checkpatch errors and warnings. v4: - Fix some apply-robot checkpatch errors and warnings. - Fix github-robot build failure: linux clang test asan. v5: Improve commit message to better describe the intent of this patch. v6: - Convert some explicit null pointer checks to assertions since they are checking for impossible conditions. - Use `nullable_memset()` and `nullable_memcpy()` instead of manually performing null checks for `memset()` and `memcpy()`. v7: - Split the single commit into a patch series which consists of 8 patches to make it easier to review and help keep things organized. - Incorporated feedback from Aaron Conole. v8: Incorporated feedback from Simon Horman and Ilya Maximets. James Raphael Tiovalen (8): lib: Add non-null assertions to some return values of `dp_packet_data` lib, ovs-vsctl: Add zero-initializations shash, simap, smap: Add assertions to `*_count` functions ovsdb: Assert and check return values of `ovsdb_table_schema_get_column` file, monitor: Add null pointer assertions for old and new ovsdb_rows ovs-vsctl: Fix crash when routing is enabled lib, ovsdb: Add various null pointer checks treewide: Add `ovs_assert` to check for null pointers lib/dp-packet.c | 24 +++++++++++++++--------- lib/dpctl.c | 4 ++++ lib/latch-unix.c | 2 +- lib/netdev-native-tnl.c | 17 +++++++++++------ lib/odp-execute.c | 4 ++++ lib/pcap-file.c | 4 +++- lib/rtnetlink.c | 4 ++-- lib/sflow_agent.c | 11 +++++++++-- lib/sflow_api.h | 2 +- lib/shash.c | 11 +++++++++-- lib/simap.c | 2 ++ lib/smap.c | 1 + lib/sset.c | 2 ++ ovsdb/condition.c | 5 ++++- ovsdb/file.c | 2 ++ ovsdb/jsonrpc-server.c | 6 +++++- ovsdb/monitor.c | 14 ++++++++++++-- ovsdb/ovsdb-client.c | 14 +++++++++++--- ovsdb/ovsdb-server.c | 2 ++ ovsdb/ovsdb-util.c | 6 ++++++ ovsdb/ovsdb.c | 8 +++++++- ovsdb/query.c | 2 ++ ovsdb/row.c | 4 +++- ovsdb/transaction.c | 2 ++ utilities/ovs-vsctl.c | 30 +++++++++++++++++++++--------- vtep/vtep-ctl.c | 5 ++++- 26 files changed, 145 insertions(+), 43 deletions(-) -- 2.40.1 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev