isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use INADDR_NONE instead of casting -1 to in_addr_t (which may not exist).

Browse Source
This commit is contained in:
Todd C. Miller
2010-10-12 10:31:21 -04:00
parent a178a430a2
commit c615ca742a
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
plugins/sudoers/interfaces.c
View File

@@ -48,6 +48,10 @@
#include "sudoers.h" #include "sudoers.h"
#include "interfaces.h" #include "interfaces.h"
#ifndef INADDR_NONE
# define INADDR_NONE ((unsigned int)-1)
#endif
/* /*
* Parse a space-delimited list of IP address/netmask pairs and * Parse a space-delimited list of IP address/netmask pairs and
* store in a list of interface structures. * store in a list of interface structures.
@@ -83,8 +87,8 @@ set_interfaces(const char *ai)
ifp->family = AF_INET; ifp->family = AF_INET;
ifp->addr.ip4.s_addr = inet_addr(addr); ifp->addr.ip4.s_addr = inet_addr(addr);
ifp->netmask.ip4.s_addr = inet_addr(mask); ifp->netmask.ip4.s_addr = inet_addr(mask);
if (ifp->addr.ip4.s_addr == (in_addr_t)-1 || if (ifp->addr.ip4.s_addr == INADDR_NONE ||
ifp->netmask.ip4.s_addr == (in_addr_t)-1) { ifp->netmask.ip4.s_addr == INADDR_NONE) {
efree(ifp); efree(ifp);
continue; continue;
} }