proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-08-08 09:30:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

lib: Rewrite ipv4 address and prefix validator

Browse Source 
Simplify ipv4 prefix and address matcher / validator to use standard
Linux networking functions instead of a state machine.

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
Reviewed-by: Don Slice <dslice@cumulusnetworks.com>
This commit is contained in:
Quentin Young 2016-06-27 14:51:17 +00:00
parent 4d91343a45
commit d4dc41b6a2
1 changed files with 40 additions and 112 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

132
lib/command.c
View File

@ -865,138 +865,66 @@ enum match_type
exact_match exact_match
}; };
#define IPV4_ADDR_STR "0123456789."
#define IPV4_PREFIX_STR "0123456789./"
/**
* Determines whether a string is a valid ipv4 token.
*
* @param[in] str the string to match
* @return exact_match if the string is an exact match, no_match/partly_match
* otherwise
*/
static enum match_type static enum match_type
cmd_ipv4_match (const char *str) cmd_ipv4_match (const char *str)
{ {
const char *sp; struct sockaddr_in sin_dummy;
int dots = 0, nums = 0;
char buf[4];
if (str == NULL) if (str == NULL)
return partly_match; return partly_match;
for (;;) if (strspn (str, IPV4_ADDR_STR) != strlen (str))
{
memset (buf, 0, sizeof (buf));
sp = str;
while (*str != '\0')
{
if (*str == '.')
{
if (dots >= 3)
return no_match; return no_match;
if (*(str + 1) == '.') if (inet_pton(AF_INET, str, &sin_dummy.sin_addr) != 1)
return no_match; return no_match;
if (*(str + 1) == '\0')
return partly_match;
dots++;
break;
}
if (!isdigit ((int) *str))
return no_match;
str++;
}
if (str - sp > 3)
return no_match;
strncpy (buf, sp, str - sp);
if (atoi (buf) > 255)
return no_match;
nums++;
if (*str == '\0')
break;
str++;
}
if (nums < 4)
return partly_match;
return exact_match; return exact_match;
} }
static enum match_type static enum match_type
cmd_ipv4_prefix_match (const char *str) cmd_ipv4_prefix_match (const char *str)
{ {
const char *sp; struct sockaddr_in sin_dummy;
int dots = 0; const char *delim = "/\0";
char buf[4]; char *dupe, *prefix, *mask, *context, *endptr;
int nmask = -1;
if (str == NULL) if (str == NULL)
return partly_match; return partly_match;
for (;;) if (strspn (str, IPV4_PREFIX_STR) != strlen (str))
{
memset (buf, 0, sizeof (buf));
sp = str;
while (*str != '\0' && *str != '/')
{
if (*str == '.')
{
if (dots == 3)
return no_match; return no_match;
if (*(str + 1) == '.' || *(str + 1) == '/') /* tokenize to address + mask */
return no_match; dupe = XMALLOC(MTYPE_TMP, strlen(str)+1);
strncpy(dupe, str, strlen(str)+1);
prefix = strtok_r(dupe, delim, &context);
mask = strtok_r(NULL, delim, &context);
if (*(str + 1) == '\0') if (!mask)
return partly_match; return partly_match;
dots++; /* validate prefix */
break; if (inet_pton(AF_INET, prefix, &sin_dummy.sin_addr) != 1)
}
if (!isdigit ((int) *str))
return no_match; return no_match;
str++; /* validate mask */
} nmask = strtol (mask, &endptr, 10);
if (*endptr != '\0' || nmask < 0 || nmask > 32)
if (str - sp > 3)
return no_match; return no_match;
strncpy (buf, sp, str - sp); XFREE(MTYPE_TMP, dupe);
if (atoi (buf) > 255)
return no_match;
if (dots == 3)
{
if (*str == '/')
{
if (*(str + 1) == '\0')
return partly_match;
str++;
break;
}
else if (*str == '\0')
return partly_match;
}
if (*str == '\0')
return partly_match;
str++;
}
sp = str;
while (*str != '\0')
{
if (!isdigit ((int) *str))
return no_match;
str++;
}
if (atoi (sp) > 32)
return no_match;
return exact_match; return exact_match;
} }