geoip: make sure we null-terminate after read

Make sure raw is null-terminated after we read the buffer that we
receive from the remote end. Detected by coverty scan.

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
netsniff-ng: do not leak file descriptors on exit

Covertiy detected that when we redirect sdtin/stdout either via
``-i -'' or ``-o -'', we also need to properly close it when it
goes out of scope.

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
geoip: reorder statements, so that file handles are closed first

Coverty detected, when we are in an error condition, we do not close
file handles, but just return instead. Thus, reorder statements, so
that file handles are closed for sure.

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
flowtop: close dir handle first before returning back

Coverty detected that the variable "dir" going out of scope leaks the
storage it points to. Thus close the handle first!

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
astraceroute: fix getnameinfo arguments

Coverty detected a misuse of the getnameinfo API, thus, we
need to fix the length parameter.

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>