After coverity tool scan, some warnings about signed/unsigned conflicts in
sscanf format string were found. Not a serious problem, but better to be
correct.

RHBZ#1032563

Patch from TAKADA Yoshihito
https://bugzilla.redhat.com/show_bug.cgi?id=1021109

They are needed to be able to install binaries into
/usr/bin & /usr/sbin instead of /bin & /sbin.

1) Fall-back to numeric address if getnameinfo fails.
   Reverse lookup is not mandatory, therefore its fail
   is not an error. Just return numeric address in that case.
   This makes netstat/route show IPv6 address instead of
   [UNKNOWN] in case of DNS problems.

2) Pass length of 'name' buffer into function.
   'name' is a pointer and therefore sizeof(name)
   returns size of pointer and not size of the buffer.
   see http://stackoverflow.com/questions/14298710/c-pointers-and-arrays-sizeof-operator
   The sizeof() usage was added with commit 604785ad,
   so I checked all the other changes in that commit
   and they seem to be OK.

3) remove unused 's' variable

Thanks to Elias Probst for mentioning the problem.

Signed-off-by: Lehner Florian <dev@der-flo.net>

Signed-off-by: Lehner Florian <dev@der-flo.net>

[PATCH] replace strcpy() with safe_strncpy()

Signed-off-by: Lehner Florian <dev@der-flo.net>

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Currently, we pass around sockaddr to various locations where they
might case things to IPv4 or IPv6 addresses.  In the latter case,
this struct is not large enough to hold an IPv6 address, so we end
up smashing the stack.

For example:
	ifconfig sit1 inet6 pointopoint ::1.2.3.4
This calls ap->input() with an IPv6 address but a sockaddr struct.
Internally, we cast this to sockaddr_in6 and then write too much.

Rather than fixing this one cast, change the code to use the new
sockaddr_storage struct.  Then point sockaddr_in and sockaddr_in6
to that location.  This way we can use "sa" everywhere and then
use "sin" and "sin6" automatically (since they point to the same
storage).

URL: http://sourceforge.net/p/net-tools/bugs/3/Signed-off-by: Mike Frysinger <vapier@gentoo.org>

An IPv4 address is 32bits.  Using unsigned long to represent that results
in misbehavior on 64bit hosts as sizeof(ulong) == 8.  Use in_addr_t in its
place to represent IPv4 addresses.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

They've started linking libselinux against more things (like libpcre),
so use pkg-config to get its dependency info.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Libs go into LIBS and after objects, not LDFLAGS.

We don't want to pass -DHAVE_SELINUX via the command line as the generated
config.h already defines it for us.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>

There are many CFLAGS which implicitly affect linking too (like
-fsanitize=address), so include them when linking.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Rmi Verschelde contributed french translation if ifconfig(8). https://sourceforge.net/p/net-tools/bugs/6/

https://sourceforge.net/p/net-tools/bugs/7/

Signed-off-by: Gilles Espinasse <g.esp@free.fr>

to avoid string overflow and to make sure the string
is terminated.

- link mii-tool and nameif against lib/
- remove xmalloc() from nameif because it's defined also in util.c

Read and dump all registers, including those not listed
in linux/mii.h.
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>

Document command line "-p addr" / "--phy=addr".
Modify comments describing command line parameters.
Signed-off-by: Antonio Borneo <borneo.antonio@...>