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. v9: Fix warning for patch 2/8 due to using `sizeof` on a void pointer. v10: Resolve merge conflict for patch 8/8. v11: Fix order of calling `ovs_assert` before `memset` in patch 2/8. v12: Fix failure of some tests in patch 4/8 by checking for warning logs in OVSDB_SERVER_SHUTDOWN. v13: - Incorporated feedback from Eelco Chaudron and Ilya Maximets. - Regrouped some changes between patches 3/4 and 4/4. James Raphael Tiovalen (4): lib: Add non-null assertions to some return values of `dp_packet_data` lib, ovs-vsctl: Add zero-initializations lib, ovsdb, vtep: Add various null pointer checks treewide: Add `ovs_assert` to check for null pointers lib/dp-packet.c | 24 +++++++++++++++--------- lib/dpctl.c | 12 ++++++++++++ lib/latch-unix.c | 2 +- lib/netdev-native-tnl.c | 6 +++++- lib/odp-execute.c | 4 ++++ lib/pcap-file.c | 4 +++- lib/rtnetlink.c | 4 ++-- lib/sflow_agent.c | 12 ++++++++++-- lib/sflow_api.h | 2 +- lib/shash.c | 6 +++++- lib/sset.c | 5 +++++ ovsdb/jsonrpc-server.c | 6 +++++- ovsdb/monitor.c | 14 ++++++++++++-- ovsdb/ovsdb-client.c | 7 ++++++- ovsdb/ovsdb-server.c | 2 ++ ovsdb/row.c | 5 ++++- ovsdb/transaction.c | 2 ++ utilities/ovs-vsctl.c | 6 +++--- vtep/vtep-ctl.c | 6 ++++++ 19 files changed, 103 insertions(+), 26 deletions(-) -- 2.41.0 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev