From: Daniel Wagner <[email protected]> Hi,
Here is the secound round of small bug fixes reported by coverity. There isn't any particular ordering or sorting of the fixes. I just send fixes which are obviously easy to understand and fix. The only common pattern between these fixes is that they are classified by coverity as medium. Happy reviewing :) cheers, daniel Daniel Wagner (14): gsupplicant: Remove useless NULL pointer check gsupplicant: Fix empty string test vpn-manager: __connman_error_failed() expects positive error codes provider: __connman_error_failed() expects positive error codes dnsproxy: Check return value of get_req_udp_socket() 6to4: Return error code when tunnel_create() failes client: strerror expects positive error codes gdhcp: Remove dead code ipconfig: Remove dead code iptables: Remove dead code l2tp: Close file descriptors after use hh2serial-gps: Check error returns when during init() nat: Free all resources in error case network: Do not access pointer after testing its valitiy client/commands.c | 2 +- gdhcp/common.c | 5 +---- gsupplicant/supplicant.c | 6 +++--- plugins/hh2serial-gps.c | 20 ++++++++++++++++---- src/6to4.c | 4 +++- src/dnsproxy.c | 3 +++ src/ipconfig.c | 4 +--- src/iptables.c | 5 ----- src/nat.c | 7 +++++-- src/network.c | 6 ++---- src/provider.c | 2 +- vpn/plugins/l2tp.c | 2 ++ vpn/vpn-manager.c | 2 +- 13 files changed, 39 insertions(+), 29 deletions(-) -- 1.8.4.474.g128a96c _______________________________________________ connman mailing list [email protected] https://lists.connman.net/mailman/listinfo/connman
