[iproute2,00/11] fix analyzer warnings

Message

Stephen Hemminger May 9, 2023, 9:21 p.m. UTC

Address some (but not all) of the issues reported by gcc 13
new analyzer.  These are mostly just issues like not checking
for malloc() failure.

Stephen Hemminger (11):
  lib/fs: fix file leak in task_get_name
  ipmaddr: fix dereference of NULL on malloc() failure
  iproute_lwtunnel: fix possible use of NULL when malloc() fails
  tc_filter: fix unitialized warning
  tc_util fix unitialized warning
  tc_exec: don't dereference NULL on calloc failure
  m_action: fix warning of overwrite of const string
  netem: fix NULL deref on allocation failure
  nstat: fix potential NULL deref
  rdma/utils: fix some analyzer warnings
  tc/prio: handle possible truncated kernel response

 ip/ipmaddr.c          |  9 ++++++++-
 ip/iproute_lwtunnel.c | 18 +++++++++++++-----
 lib/fs.c              |  4 +++-
 misc/nstat.c          |  6 ++++++
 rdma/utils.c          | 10 ++++++++++
 tc/m_action.c         |  4 ++--
 tc/q_netem.c          |  3 +++
 tc/q_prio.c           |  2 ++
 tc/tc_exec.c          |  4 ++++
 tc/tc_filter.c        |  7 ++++---
 tc/tc_util.c          |  2 +-
 11 files changed, 56 insertions(+), 13 deletions(-)

Comments

[e-mail] May 14, 2023, 2:20 a.m. UTC | #1

Hello:

This series was applied to iproute2/iproute2.git (main)
by Stephen Hemminger <[e-mail]>:

On Tue,  9 May 2023 14:21:14 -0700 you wrote:
> Address some (but not all) of the issues reported by gcc 13
> new analyzer.  These are mostly just issues like not checking
> for malloc() failure.
> 
> Stephen Hemminger (11):
>   lib/fs: fix file leak in task_get_name
>   ipmaddr: fix dereference of NULL on malloc() failure
>   iproute_lwtunnel: fix possible use of NULL when malloc() fails
>   tc_filter: fix unitialized warning
>   tc_util fix unitialized warning
>   tc_exec: don't dereference NULL on calloc failure
>   m_action: fix warning of overwrite of const string
>   netem: fix NULL deref on allocation failure
>   nstat: fix potential NULL deref
>   rdma/utils: fix some analyzer warnings
>   tc/prio: handle possible truncated kernel response
> 
> [...]

Here is the summary with links:
  - [iproute2,01/11] lib/fs: fix file leak in task_get_name
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=9f0fe8ee0900
  - [iproute2,02/11] ipmaddr: fix dereference of NULL on malloc() failure
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=8cda7a24a971
  - [iproute2,03/11] iproute_lwtunnel: fix possible use of NULL when malloc() fails
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=fa44c2d6f1da
  - [iproute2,04/11] tc_filter: fix unitialized warning
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=c73a41054638
  - [iproute2,05/11] tc_util fix unitialized warning
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=c9c1c9d59a6c
  - [iproute2,06/11] tc_exec: don't dereference NULL on calloc failure
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=0b9b9d659880
  - [iproute2,07/11] m_action: fix warning of overwrite of const string
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=b134c2c34458
  - [iproute2,08/11] netem: fix NULL deref on allocation failure
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=6c266d7c22a8
  - [iproute2,09/11] nstat: fix potential NULL deref
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=d348d1d6466a
  - [iproute2,10/11] rdma/utils: fix some analyzer warnings
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=33722349feb9
  - [iproute2,11/11] tc/prio: handle possible truncated kernel response
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=c90d25e96b01

You are awesome, thank you!