For both ifconfig and route if we didn't get enough memory from the
prior sysctl due to the structure growing between calls try again. Also try again for deleting routes if things fail. We've seen route -f fail this way which does not actually flush all routes. This fixes it. It will whine but it will do the work. PR: 56732 Obtained from: IronPort
This commit is contained in:
parent
37e6803f0b
commit
643bbdb471
@ -417,7 +417,7 @@ main(int argc, char *argv[])
|
||||
{
|
||||
int c;
|
||||
int all, namesonly, downonly, uponly;
|
||||
int need_nl = 0;
|
||||
int need_nl = 0, count = 0;
|
||||
const struct afswtch *afp = 0;
|
||||
int addrcount, ifindex;
|
||||
struct if_msghdr *ifm, *nextifm;
|
||||
@ -543,6 +543,7 @@ main(int argc, char *argv[])
|
||||
afp = NULL; /* not a family, NULL */
|
||||
}
|
||||
|
||||
retry:
|
||||
mib[0] = CTL_NET;
|
||||
mib[1] = PF_ROUTE;
|
||||
mib[2] = 0;
|
||||
@ -558,8 +559,15 @@ main(int argc, char *argv[])
|
||||
errx(1, "iflist-sysctl-estimate");
|
||||
if ((buf = malloc(needed)) == NULL)
|
||||
errx(1, "malloc");
|
||||
if (sysctl(mib, 6, buf, &needed, NULL, 0) < 0)
|
||||
if (sysctl(mib, 6, buf, &needed, NULL, 0) < 0) {
|
||||
if (errno == ENOMEM && count++ < 10) {
|
||||
warnx("Routing table grew, retrying");
|
||||
free(buf);
|
||||
sleep(1);
|
||||
goto retry;
|
||||
}
|
||||
errx(1, "actual retrieval of interface table");
|
||||
}
|
||||
lim = buf + needed;
|
||||
|
||||
next = buf;
|
||||
|
@ -201,7 +201,7 @@ flushroutes(argc, argv)
|
||||
char *argv[];
|
||||
{
|
||||
size_t needed;
|
||||
int mib[6], rlen, seqno;
|
||||
int mib[6], rlen, seqno, count = 0;
|
||||
char *buf, *next, *lim;
|
||||
struct rt_msghdr *rtm;
|
||||
|
||||
@ -232,6 +232,7 @@ flushroutes(argc, argv)
|
||||
} else
|
||||
bad: usage(*argv);
|
||||
}
|
||||
retry:
|
||||
mib[0] = CTL_NET;
|
||||
mib[1] = PF_ROUTE;
|
||||
mib[2] = 0; /* protocol */
|
||||
@ -242,8 +243,15 @@ bad: usage(*argv);
|
||||
err(EX_OSERR, "route-sysctl-estimate");
|
||||
if ((buf = malloc(needed)) == NULL)
|
||||
errx(EX_OSERR, "malloc failed");
|
||||
if (sysctl(mib, 6, buf, &needed, NULL, 0) < 0)
|
||||
if (sysctl(mib, 6, buf, &needed, NULL, 0) < 0) {
|
||||
if (errno == ENOMEM && count++ < 10) {
|
||||
warnx("Routing table grew, retrying");
|
||||
sleep(1);
|
||||
free(buf);
|
||||
goto retry;
|
||||
}
|
||||
err(EX_OSERR, "route-sysctl-get");
|
||||
}
|
||||
lim = buf + needed;
|
||||
if (verbose)
|
||||
(void) printf("Examining routing table from sysctl\n");
|
||||
@ -268,6 +276,8 @@ bad: usage(*argv);
|
||||
if (rlen < (int)rtm->rtm_msglen) {
|
||||
warn("write to routing socket");
|
||||
(void) printf("got only %d for rlen\n", rlen);
|
||||
free(buf);
|
||||
goto retry;
|
||||
break;
|
||||
}
|
||||
seqno++;
|
||||
@ -1105,9 +1115,10 @@ interfaces()
|
||||
{
|
||||
size_t needed;
|
||||
int mib[6];
|
||||
char *buf, *lim, *next;
|
||||
char *buf, *lim, *next, count = 0;
|
||||
struct rt_msghdr *rtm;
|
||||
|
||||
retry2:
|
||||
mib[0] = CTL_NET;
|
||||
mib[1] = PF_ROUTE;
|
||||
mib[2] = 0; /* protocol */
|
||||
@ -1118,8 +1129,15 @@ interfaces()
|
||||
err(EX_OSERR, "route-sysctl-estimate");
|
||||
if ((buf = malloc(needed)) == NULL)
|
||||
errx(EX_OSERR, "malloc failed");
|
||||
if (sysctl(mib, 6, buf, &needed, NULL, 0) < 0)
|
||||
if (sysctl(mib, 6, buf, &needed, NULL, 0) < 0) {
|
||||
if (errno == ENOMEM && count++ < 10) {
|
||||
warnx("Routing table grew, retrying");
|
||||
sleep(1);
|
||||
free(buf);
|
||||
goto retry2;
|
||||
}
|
||||
err(EX_OSERR, "actual retrieval of interface table");
|
||||
}
|
||||
lim = buf + needed;
|
||||
for (next = buf; next < lim; next += rtm->rtm_msglen) {
|
||||
rtm = (struct rt_msghdr *)next;
|
||||
|
Loading…
Reference in New Issue
Block a user