diff options
| author | Ingo Schwarze <schwarze@cvs.openbsd.org> | 2014-07-25 19:36:03 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@cvs.openbsd.org> | 2014-07-25 19:36:03 +0000 |
| commit | 4c59e7da970a9c59d1e75d5d9173f64307ca9929 (patch) | |
| tree | 1fb38a879d720944451cba3f9641745b017ce9b7 /usr.bin/mandoc | |
| parent | 2afbb0e9dde74b7dc457e8648d1e02630bdfcd31 (diff) | |
oops, we must not try to validate a manpath we don't have;
fixing an oversight introduced in rev. 1.17
Diffstat (limited to 'usr.bin/mandoc')
| -rw-r--r-- | usr.bin/mandoc/cgi.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/usr.bin/mandoc/cgi.c b/usr.bin/mandoc/cgi.c index 8f89965c55d..2c0b8c2aebe 100644 --- a/usr.bin/mandoc/cgi.c +++ b/usr.bin/mandoc/cgi.c @@ -1,4 +1,4 @@ -/* $Id: cgi.c,v 1.28 2014/07/25 18:19:33 schwarze Exp $ */ +/* $Id: cgi.c,v 1.29 2014/07/25 19:36:02 schwarze Exp $ */ /* * Copyright (c) 2011, 2012 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2014 Ingo Schwarze <schwarze@usta.de> @@ -1082,7 +1082,8 @@ main(void) if (NULL != (querystring = getenv("QUERY_STRING"))) http_parse(&req, querystring); - if ( ! validate_manpath(&req, req.q.manpath)) { + if ( ! (NULL == req.q.manpath || + validate_manpath(&req, req.q.manpath))) { pg_error_badrequest( "You specified an invalid manpath."); return(EXIT_FAILURE); |