mbox series

[ovs-dev,v2,0/8] Coverity fixes.

Message ID 20240528114552.730389-1-echaudro@redhat.com
Headers show
Series Coverity fixes. | expand

Message

Eelco Chaudron May 28, 2024, 11:39 a.m. UTC 
This series addresses several high-priority Coverity issues.

Delta from v1 -> v2:

- Add cover letter.
- Split first patch and update commit subject.
- Added ds_destroy() to context_done().
- Changed invalid gso_type handling by returning an error + log message.
- Changed sFlowRcvrTimeout to be using uint32_t to avoid TIME_T_MAKE.

Eelco Chaudron (8):
  netdev-linux: Fix possible int overflow in tc_add_matchall_policer().
  cfm: Fix possible integer overflow in tc_add_matchall_policer().
  sflow: Replace libc's random() function with the OVS's random_range().
  sflow: Use uint32_t instead of time_t for tick handling in the poller.
  sflow: Fix check for disabled receive time.
  ofproto-dpif: Define age as time_t in ofproto_unixctl_fdb_add().
  db-ctl-base: Initialize the output variable in the ctx structure.
  netdev-linux: Fix uninitialized gso_type case.

 lib/cfm.c                    |  2 +-
 lib/db-ctl-base.c            |  2 ++
 lib/netdev-linux.c           |  9 +++++++--
 lib/sflow_api.h              | 10 +++++-----
 lib/sflow_poller.c           |  3 ++-
 lib/sflow_receiver.c         |  7 ++++---
 ofproto/ofproto-dpif-sflow.c |  2 +-
 ofproto/ofproto-dpif.c       |  2 +-
 8 files changed, 23 insertions(+), 14 deletions(-)