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

Merge pull request #4603 from opensourcerouting/vty-recursion

Browse Source 
lib: add a check for {[...]} in DEFUNs
This commit is contained in:
Donald Sharp 2019-06-25 09:19:57 -04:00 committed by GitHub
commit 0d7b9179a1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 36 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
lib/command_parse.y
View File

@ -46,6 +46,7 @@
%code requires { %code requires {
#include "config.h" #include "config.h"
#include <stdbool.h>
#include <stdlib.h> #include <stdlib.h>
#include <string.h> #include <string.h>
#include <ctype.h> #include <ctype.h>
@ -140,6 +141,8 @@
static void static void
cleanup (struct parser_ctx *ctx); cleanup (struct parser_ctx *ctx);
static void loopcheck(struct parser_ctx *ctx, struct subgraph *sg);
#define scanner ctx->scanner #define scanner ctx->scanner
} }
@ -335,6 +338,7 @@ selector: '{' selector_seq_seq '}' varname_token
* just use [{a|b}] if neccessary, that will work perfectly fine, and reason * just use [{a|b}] if neccessary, that will work perfectly fine, and reason
* #1 is good enough to keep it this way. */ * #1 is good enough to keep it this way. */
loopcheck(ctx, &$$);
cmd_token_varname_set ($2.end->data, $4); cmd_token_varname_set ($2.end->data, $4);
XFREE (MTYPE_LEX, $4); XFREE (MTYPE_LEX, $4);
}; };
@ -396,6 +400,38 @@ cmd_graph_parse (struct graph *graph, struct cmd_element *cmd)
/* parser helper functions */ /* parser helper functions */
static bool loopcheck_inner(struct graph_node *start, struct graph_node *node,
struct graph_node *end, size_t depth)
{
size_t i;
bool ret;
/* safety check */
if (depth++ == 64)
return true;
for (i = 0; i < vector_active(node->to); i++) {
struct graph_node *next = vector_slot(node->to, i);
struct cmd_token *tok = next->data;
if (next == end || next == start)
return true;
if (tok->type < SPECIAL_TKN)
continue;
ret = loopcheck_inner(start, next, end, depth);
if (ret)
return true;
}
return false;
}
static void loopcheck(struct parser_ctx *ctx, struct subgraph *sg)
{
if (loopcheck_inner(sg->start, sg->start, sg->end, 0))
zlog_err("FATAL: '%s': {} contains an empty path! Use [{...}]",
ctx->el->string);
}
void void
yyerror (CMD_YYLTYPE *loc, struct parser_ctx *ctx, char const *msg) yyerror (CMD_YYLTYPE *loc, struct parser_ctx *ctx, char const *msg)
{ {