From ebe5f39bfbb64921d5d31e0a6e49287356a5e6e2 Mon Sep 17 00:00:00 2001
From: Julian Andres Klode <julian.klode@canonical.com>
Date: Tue, 28 Jan 2020 23:06:08 +0100
Subject: patterns: Allow bare words only in arguments

This changes the syntax from approximately

	expr = unary
	unary = '!'? primary
	primary = pattern | short-pattern | word | quoted-word
	pattern = '?' name [ '(' expr [',' expr]* ')' ]
	short-pattern = ~ name | ~name expr
to:

        primary = pattern | short-pattern
        argument = word | quoted-word | expr
        pattern = '?' name [ '(' argument [',' argument]* ')' ]
        short-pattern = ~ name | ~name argument
---
 apt-pkg/cachefilter-patterns.cc | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

(limited to 'apt-pkg/cachefilter-patterns.cc')

diff --git a/apt-pkg/cachefilter-patterns.cc b/apt-pkg/cachefilter-patterns.cc
index 9fab0281d..cf3e59ac6 100644
--- a/apt-pkg/cachefilter-patterns.cc
+++ b/apt-pkg/cachefilter-patterns.cc
@@ -58,6 +58,9 @@ std::unique_ptr<PatternTreeParser::Node> PatternTreeParser::parseTop()
    auto node = parse();
    skipSpace();
 
+   if (node == nullptr)
+      throw Error{Node{0, sentence.size()}, "Expected pattern"};
+
    if (node->end != sentence.size())
       throw Error{Node{node->end, sentence.size()}, "Expected end of file"};
 
@@ -98,10 +101,19 @@ std::unique_ptr<PatternTreeParser::Node> PatternTreeParser::parsePrimary()
       return node;
    if ((node = parsePattern()) != nullptr)
       return node;
+
+   return nullptr;
+}
+
+std::unique_ptr<PatternTreeParser::Node> PatternTreeParser::parseArgument()
+{
+   std::unique_ptr<Node> node;
    if ((node = parseQuotedWord()) != nullptr)
       return node;
    if ((node = parseWord()) != nullptr)
       return node;
+   if ((node = parse()) != nullptr)
+      return node;
 
    throw Error{Node{state.offset, sentence.size()},
 	       "Expected pattern, quoted word, or word"};
@@ -125,7 +137,7 @@ std::unique_ptr<PatternTreeParser::Node> PatternTreeParser::parseShortPattern()
       state.offset += sp.shortName.size() + 1;
       if (sp.takesArgument)
       {
-	 node->arguments.push_back(parse());
+	 node->arguments.push_back(parseArgument());
 	 node->haveArgumentList = true;
       }
       node->end = state.offset;
@@ -173,7 +185,7 @@ std::unique_ptr<PatternTreeParser::Node> PatternTreeParser::parsePattern()
       return node;
    }
 
-   node->arguments.push_back(parse());
+   node->arguments.push_back(parseArgument());
    skipSpace();
    while (sentence[state.offset] == ',')
    {
@@ -182,7 +194,7 @@ std::unique_ptr<PatternTreeParser::Node> PatternTreeParser::parsePattern()
       // This was a trailing comma - allow it and break the loop
       if (sentence[state.offset] == ')')
 	 break;
-      node->arguments.push_back(parse());
+      node->arguments.push_back(parseArgument());
       skipSpace();
    }
 
-- 
cgit v1.2.3-70-g09d2