From f3129a3cc4800321ed2157eb400da8b43398606e Mon Sep 17 00:00:00 2001 From: Patrick McHardy Date: Tue, 4 Feb 2014 08:09:27 +0000 Subject: parser: recover from errors in any block Move error recovery to the common_block definition to handle errors in any block. Queue those errors and abort parsing once a threshold is reached. With this in place, we can continue parsing when errors occur and show all of them to the user at once. tests/error.1:3:8-8: Error: syntax error, unexpected '{', expecting string filter { ^ tests/error.1:4:13-13: Error: syntax error, unexpected newline filter input ^ tests/error.1:5:17-17: Error: syntax error, unexpected newline filter input tcp ^ tests/error.1:6:23-23: Error: syntax error, unexpected newline filter input tcp dport Signed-off-by: Patrick McHardy --- src/main.c | 3 ++- src/parser.y | 7 ++++++- 2 files changed, 8 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/main.c b/src/main.c index 28ce1aa6..2320a826 100644 --- a/src/main.c +++ b/src/main.c @@ -26,6 +26,7 @@ #include #include +unsigned int max_errors = 10; unsigned int numeric_output; unsigned int handle_output; #ifdef DEBUG @@ -219,7 +220,7 @@ int nft_run(void *scanner, struct parser_state *state, struct list_head *msgs) int ret = 0; ret = nft_parse(scanner, state); - if (ret != 0) + if (ret != 0 || state->nerrs > 0) return -1; memset(&ctx, 0, sizeof(ctx)); diff --git a/src/parser.y b/src/parser.y index fa33b570..0dad036c 100644 --- a/src/parser.y +++ b/src/parser.y @@ -518,6 +518,12 @@ common_block : INCLUDE QUOTED_STRING stmt_seperator symbol_bind(current_scope(state), $2, $4); xfree($2); } + | error stmt_seperator + { + if (++state->nerrs == max_errors) + YYABORT; + yyerrok; + } ; line : common_block { $$ = NULL; } @@ -542,7 +548,6 @@ line : common_block { $$ = NULL; } YYACCEPT; } - | base_cmd error { $$ = $1; } ; base_cmd : /* empty */ add_cmd { $$ = $1; } -- cgit v1.2.3