diff options
| author | Sebastian Benoit <benno@cvs.openbsd.org> | 2016-06-21 21:35:26 +0000 |
|---|---|---|
| committer | Sebastian Benoit <benno@cvs.openbsd.org> | 2016-06-21 21:35:26 +0000 |
| commit | f2ada2db1464df13335d236529c834c378ced993 (patch) | |
| tree | a15cd06b17ca32de9b75238620437773f018ecdf /usr.sbin/smtpd | |
| parent | 05fc6735d59f44db060a9d3a428944db3e862ebe (diff) | |
do not allow whitespace in macro names, i.e. "this is" = "a variable".
change this in all config parsers in our tree that support macros.
problem reported by sven falempin.
feedback from henning@, stsp@, deraadt@
ok florian@ mikeb@
Diffstat (limited to 'usr.sbin/smtpd')
| -rw-r--r-- | usr.sbin/smtpd/parse.y | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/usr.sbin/smtpd/parse.y b/usr.sbin/smtpd/parse.y index ab0271920ec..d4c0fcbc1b7 100644 --- a/usr.sbin/smtpd/parse.y +++ b/usr.sbin/smtpd/parse.y @@ -1,4 +1,4 @@ -/* $OpenBSD: parse.y,v 1.184 2016/04/21 14:27:41 jsing Exp $ */ +/* $OpenBSD: parse.y,v 1.185 2016/06/21 21:35:25 benno Exp $ */ /* * Copyright (c) 2008 Gilles Chehade <gilles@poolp.org> @@ -213,6 +213,14 @@ include : INCLUDE STRING { ; varset : STRING '=' STRING { + char *s = $1; + while (*s++) { + if (isspace((unsigned char)*s)) { + yyerror("macro name cannot contain " + "whitespace"); + YYERROR; + } + } if (symset($1, $3, 0) == -1) fatal("cannot store variable"); free($1); |