]> git.armaanb.net Git - opendoas.git/blobdiff - parse.y
add more restrictive permissions and root:root as owner for binary
[opendoas.git] / parse.y
diff --git a/parse.y b/parse.y
index f1e90ab9039fc555fd05e4a0acf6890d922888fd..6166ceef908353e56ff965b7110dff68aa43f66e 100644 (file)
--- a/parse.y
+++ b/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.5 2015/07/19 22:09:08 benno Exp $ */
+/* $OpenBSD: parse.y,v 1.16 2016/06/05 00:46:34 djm Exp $ */
 /*
  * Copyright (c) 2015 Ted Unangst <tedu@openbsd.org>
  *
 #include <stdint.h>
 #include <stdarg.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 #include <err.h>
 
+#include "openbsd.h"
+
 #include "doas.h"
 
 typedef struct {
@@ -32,10 +35,15 @@ typedef struct {
                struct {
                        int action;
                        int options;
+                       const char *cmd;
+                       const char **cmdargs;
                        const char **envlist;
+                       const char **setenvlist;
                };
                const char *str;
        };
+       int lineno;
+       int colno;
 } yystype;
 #define YYSTYPE yystype
 
@@ -43,6 +51,7 @@ FILE *yyfp;
 
 struct rule **rules;
 int nrules, maxrules;
+int parse_errors = 0;
 
 void yyerror(const char *, ...);
 int yylex(void);
@@ -50,8 +59,8 @@ int yyparse(void);
 
 %}
 
-%token TPERMIT TDENY TAS TCMD
-%token TNOPASS TKEEPENV
+%token TPERMIT TDENY TAS TCMD TARGS
+%token TNOPASS TKEEPENV TSETENV
 %token TSTRING
 
 %%
@@ -59,6 +68,7 @@ int yyparse(void);
 grammar:       /* empty */
                | grammar '\n'
                | grammar rule '\n'
+               | error '\n'
                ;
 
 rule:          action ident target cmd {
@@ -69,9 +79,11 @@ rule:                action ident target cmd {
                        r->action = $1.action;
                        r->options = $1.options;
                        r->envlist = $1.envlist;
+                       r->setenvlist = $1.setenvlist;
                        r->ident = $2.str;
                        r->target = $3.str;
-                       r->cmd = $4.str;
+                       r->cmd = $4.cmd;
+                       r->cmdargs = $4.cmdargs;
                        if (nrules == maxrules) {
                                if (maxrules == 0)
                                        maxrules = 63;
@@ -88,31 +100,50 @@ action:            TPERMIT options {
                        $$.action = PERMIT;
                        $$.options = $2.options;
                        $$.envlist = $2.envlist;
+                       $$.setenvlist = $2.setenvlist;
                } | TDENY {
                        $$.action = DENY;
                } ;
 
-options:       /* none */
-               | options option {
+options:       /* none */ {
+                       $$.options = 0;
+                       $$.envlist = NULL;
+               } | options option {
                        $$.options = $1.options | $2.options;
                        $$.envlist = $1.envlist;
                        if ($2.envlist) {
-                               if ($$.envlist)
-                                       errx(1, "can't have two keepenv sections");
-                               else
+                               if ($$.envlist) {
+                                       yyerror("can't have two keepenv sections");
+                                       YYERROR;
+                               } else
                                        $$.envlist = $2.envlist;
                        }
+                       $$.setenvlist = $1.setenvlist;
+                       if ($2.setenvlist) {
+                               if ($$.setenvlist) {
+                                       yyerror("can't have two setenv sections");
+                                       YYERROR;
+                               } else
+                                       $$.setenvlist = $2.setenvlist;
+                       }
                } ;
 option:                TNOPASS {
                        $$.options = NOPASS;
+                       $$.envlist = NULL;
                } | TKEEPENV {
                        $$.options = KEEPENV;
+                       $$.envlist = NULL;
                } | TKEEPENV '{' envlist '}' {
                        $$.options = KEEPENV;
                        $$.envlist = $3.envlist;
+               } | TSETENV '{' setenvlist '}' {
+                       $$.options = SETENV;
+                       $$.setenvlist = NULL;
+                       $$.setenvlist = $3.setenvlist;
                } ;
 
 envlist:       /* empty */ {
+                       $$.envlist = NULL;
                        if (!($$.envlist = calloc(1, sizeof(char *))))
                                errx(1, "can't allocate envlist");
                } | envlist TSTRING {
@@ -124,6 +155,28 @@ envlist:   /* empty */ {
                        $$.envlist[nenv + 1] = NULL;
                }
 
+setenvlist:    /* empty */ {
+                       if (!($$.setenvlist = calloc(1, sizeof(char *))))
+                               errx(1, "can't allocate setenvlist");
+               } | setenvlist TSTRING '=' TSTRING {
+                       int nenv = arraylen($1.setenvlist);
+                       char *cp = NULL;
+
+                       if (*$2.str == '\0' || strchr($2.str, '=') != NULL) {
+                               yyerror("invalid setenv expression");
+                               YYERROR;
+                       }
+                       if (!($$.setenvlist = reallocarray($1.setenvlist,
+                           nenv + 2, sizeof(char *))))
+                               errx(1, "can't allocate envlist");
+                       $$.setenvlist[nenv] = NULL;
+                       if (asprintf(&cp, "%s=%s", $2.str, $4.str) <= 0 ||
+                           cp == NULL)
+                               errx(1,"asprintf failed");
+                       $$.setenvlist[nenv] = cp;
+                       $$.setenvlist[nenv + 1] = NULL;
+               }
+
 
 ident:         TSTRING {
                        $$.str = $1.str;
@@ -136,9 +189,30 @@ target:            /* optional */ {
                } ;
 
 cmd:           /* optional */ {
-                       $$.str = NULL;
-               } | TCMD TSTRING {
-                       $$.str = $2.str;
+                       $$.cmd = NULL;
+                       $$.cmdargs = NULL;
+               } | TCMD TSTRING args {
+                       $$.cmd = $2.str;
+                       $$.cmdargs = $3.cmdargs;
+               } ;
+
+args:          /* empty */ {
+                       $$.cmdargs = NULL;
+               } | TARGS argslist {
+                       $$.cmdargs = $2.cmdargs;
+               } ;
+
+argslist:      /* empty */ {
+                       $$.cmdargs = NULL;
+                       if (!($$.cmdargs = calloc(1, sizeof(char *))))
+                               errx(1, "can't allocate args");
+               } | argslist TSTRING {
+                       int nargs = arraylen($1.cmdargs);
+                       if (!($$.cmdargs = reallocarray($1.cmdargs, nargs + 2,
+                           sizeof(char *))))
+                               errx(1, "can't allocate args");
+                       $$.cmdargs[nargs] = $2.str;
+                       $$.cmdargs[nargs + 1] = NULL;
                } ;
 
 %%
@@ -148,8 +222,12 @@ yyerror(const char *fmt, ...)
 {
        va_list va;
 
+       fprintf(stderr, "doas: ");
        va_start(va, fmt);
-       verrx(1, fmt, va);
+       vfprintf(stderr, fmt, va);
+       va_end(va);
+       fprintf(stderr, " at line %d\n", yylval.lineno + 1);
+       parse_errors++;
 }
 
 struct keyword {
@@ -160,69 +238,138 @@ struct keyword {
        { "permit", TPERMIT },
        { "as", TAS },
        { "cmd", TCMD },
+       { "args", TARGS },
        { "nopass", TNOPASS },
        { "keepenv", TKEEPENV },
+       { "setenv", TSETENV },
 };
 
 int
 yylex(void)
 {
        char buf[1024], *ebuf, *p, *str;
-       int i, c, next;
+       int c, quotes = 0, escape = 0, qpos = -1, nonkw = 0;
 
        p = buf;
        ebuf = buf + sizeof(buf);
+
 repeat:
-       c = getc(yyfp);
+       /* skip whitespace first */
+       for (c = getc(yyfp); c == ' ' || c == '\t'; c = getc(yyfp))
+               yylval.colno++;
+
+       /* check for special one-character constructions */
        switch (c) {
-               case ' ':
-               case '\t':
-                       goto repeat; /* skip spaces */
-               case '\\':
-                       next = getc(yyfp);
-                       if (next == '\n')
-                               goto repeat;
-                       else
-                               c = next;
                case '\n':
+                       yylval.colno = 0;
+                       yylval.lineno++;
+                       /* FALLTHROUGH */
                case '{':
                case '}':
+               case '=':
                        return c;
                case '#':
-                       while ((c = getc(yyfp)) != '\n' && c != EOF)
-                               ; /* skip comments */
-                       if (c == EOF)
-                               return 0;
+                       /* skip comments; NUL is allowed; no continuation */
+                       while ((c = getc(yyfp)) != '\n')
+                               if (c == EOF)
+                                       goto eof;
+                       yylval.colno = 0;
+                       yylval.lineno++;
                        return c;
                case EOF:
-                       return 0;
+                       goto eof;
        }
-       while (1) {
+
+       /* parsing next word */
+       for (;; c = getc(yyfp), yylval.colno++) {
                switch (c) {
+               case '\0':
+                       yyerror("unallowed character NUL in column %d",
+                           yylval.colno + 1);
+                       escape = 0;
+                       continue;
+               case '\\':
+                       escape = !escape;
+                       if (escape)
+                               continue;
+                       break;
                case '\n':
+                       if (quotes)
+                               yyerror("unterminated quotes in column %d",
+                                   qpos + 1);
+                       if (escape) {
+                               nonkw = 1;
+                               escape = 0;
+                               yylval.colno = 0;
+                               yylval.lineno++;
+                               continue;
+                       }
+                       goto eow;
+               case EOF:
+                       if (escape)
+                               yyerror("unterminated escape in column %d",
+                                   yylval.colno);
+                       if (quotes)
+                               yyerror("unterminated quotes in column %d",
+                                   qpos + 1);
+                       goto eow;
+                       /* FALLTHROUGH */
                case '{':
                case '}':
                case '#':
                case ' ':
                case '\t':
-               case EOF:
-                       goto eow;
+               case '=':
+                       if (!escape && !quotes)
+                               goto eow;
+                       break;
+               case '"':
+                       if (!escape) {
+                               quotes = !quotes;
+                               if (quotes) {
+                                       nonkw = 1;
+                                       qpos = yylval.colno;
+                               }
+                               continue;
+                       }
                }
                *p++ = c;
-               if (p == ebuf)
-                       yyerror("too much stuff");
-               c = getc(yyfp);
+               if (p == ebuf) {
+                       yyerror("too long line");
+                       p = buf;
+               }
+               escape = 0;
        }
+
 eow:
        *p = 0;
        if (c != EOF)
                ungetc(c, yyfp);
-       for (i = 0; i < sizeof(keywords) / sizeof(keywords[0]); i++) {
-               if (strcmp(buf, keywords[i].word) == 0)
-                       return keywords[i].token;
+       if (p == buf) {
+               /*
+                * There could be a number of reasons for empty buffer,
+                * and we handle all of them here, to avoid cluttering
+                * the main loop.
+                */
+               if (c == EOF)
+                       goto eof;
+               else if (qpos == -1)    /* accept, e.g., empty args: cmd foo args "" */
+                       goto repeat;
+       }
+       if (!nonkw) {
+               size_t i;
+               for (i = 0; i < sizeof(keywords) / sizeof(keywords[0]); i++) {
+                       if (strcmp(buf, keywords[i].word) == 0)
+                               return keywords[i].token;
+               }
        }
        if ((str = strdup(buf)) == NULL)
                err(1, "strdup");
        yylval.str = str;
        return TSTRING;
+
+eof:
+       if (ferror(yyfp))
+               yyerror("input error reading config");
+       return 0;
 }