Skip to content

QA check with cppcheck

Thomas Jarosch netdev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest11 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/11Fix pipe I/O stream descriptor leak in init_service_resolver()misc/ss.c+1 -00 replies
2/11Fix file descriptor leak on error in rtnl_hash_initialize()lib/rt_names.c+2 -00 replies
3/11Fix wrong comparison in cmp_print_eopt()tc/em_cmp.c+1 -10 replies
4/11Fix wrong sanity check in choke_parse_opt()tc/q_choke.c+1 -10 replies
5/11Fix memory leak of lname variable in get_target_name()tc/m_ipt.c+3 -00 replies
6/11Add missing closedir() call in do_show()ip/iptuntap.c+1 -00 replies
7/11Fix file descriptor leak on error in iproute_flush_cache()ip/iproute.c+1 -00 replies
8/11Fix file descriptor leak on error in read_viftable()ip/ipmroute.c+3 -20 replies
9/11Fix file descriptor leak on error in read_mroute_list()ip/ipmroute.c+3 -10 replies
10/11Fix file descriptor leak in do_tunnels_list()ip/iptunnel.c+3 -00 replies
11/11Fix file descriptor leak on error in read_igmp()ip/ipmaddr.c+3 -11 reply

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [iproute2 PATCH 00/11] QA check with cppcheck

Hi there,

I've given iproute2 a good run with the free
cppcheck static code analysis tool.
(http://cppcheck.sourceforge.net/)

Following are eleven patches correcting minor
resource leaks and two real bugs.

Cheers,
Thomas

Thomas Jarosch (11):
Fix pipe I/O stream descriptor leak in init_service_resolver() Fix file descriptor leak on error in rtnl_hash_initialize() Fix wrong comparison in cmp_print_eopt()
Fix wrong sanity check in choke_parse_opt()
Fix memory leak of lname variable in get_target_name() Add missing closedir() call in do_show()
Fix file descriptor leak on error in iproute_flush_cache() Fix file descriptor leak on error in read_viftable() Fix file descriptor leak on error in read_mroute_list() Fix file descriptor leak in do_tunnels_list()
Fix file descriptor leak on error in read_igmp()

 ip/ipmaddr.c   |    4 +++-
 ip/ipmroute.c  |    9 ++++++---
 ip/iproute.c   |    1 +
 ip/iptunnel.c  |    3 +++
 ip/iptuntap.c  |    1 +
 lib/rt_names.c |    2 ++
 misc/ss.c      |    1 +
 tc/em_cmp.c    |    2 +-
 tc/m_ipt.c     |    3 +++
 tc/q_choke.c   |    2 +-
 10 files changed, 22 insertions(+), 6 deletions(-)
Signature · 1 line
1.7.4.4