summaryrefslogtreecommitdiff
path: root/sbin
diff options
context:
space:
mode:
authorHans-Joerg Hoexer <hshoexer@cvs.openbsd.org>2007-08-11 00:20:31 +0000
committerHans-Joerg Hoexer <hshoexer@cvs.openbsd.org>2007-08-11 00:20:31 +0000
commite39600f4ea3a9ac88cb8ea617740727a2c80eb31 (patch)
tree61d4db9bc5b621783ab7cae1ec1026f169974e35 /sbin
parent2b5b05caec66b26f75b391f959cf30d39c86b58c (diff)
Do not complain about being not able to read non-existing files. Minor
glitch introduced by previous commit. ok markus@
Diffstat (limited to 'sbin')
-rw-r--r--sbin/isakmpd/monitor.c16
1 files changed, 11 insertions, 5 deletions
diff --git a/sbin/isakmpd/monitor.c b/sbin/isakmpd/monitor.c
index 033e17652ca..8e1e105ebb0 100644
--- a/sbin/isakmpd/monitor.c
+++ b/sbin/isakmpd/monitor.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: monitor.c,v 1.70 2007/08/07 20:09:39 markus Exp $ */
+/* $OpenBSD: monitor.c,v 1.71 2007/08/11 00:20:30 hshoexer Exp $ */
/*
* Copyright (c) 2003 Håkan Olsson. All rights reserved.
@@ -500,7 +500,7 @@ m_priv_getfd(void)
{
char path[MAXPATHLEN];
size_t len;
- int v, flags;
+ int v, flags, ret;
int err = 0;
mode_t mode;
@@ -516,8 +516,10 @@ m_priv_getfd(void)
must_read(&flags, sizeof flags);
must_read(&mode, sizeof mode);
- if (m_priv_local_sanitize_path(path, sizeof path, flags) != 0) {
- log_print("m_priv_getfd: illegal path \"%s\"", path);
+ if ((ret = m_priv_local_sanitize_path(path, sizeof path, flags))
+ != 0) {
+ if (ret == 1)
+ log_print("m_priv_getfd: illegal path \"%s\"", path);
err = EACCES;
v = -1;
} else {
@@ -694,8 +696,12 @@ m_priv_local_sanitize_path(char *path, size_t pmax, int flags)
if (realpath(path, new_path) == NULL ||
realpath("/var/run", var_run) == NULL) {
+ /*
+ * We could not decide wether the path is ok or not.
+ * Indicate this be returning 2.
+ */
if (errno == ENOENT)
- return 1;
+ return 2;
goto bad_path;
}
strlcat(var_run, "/", sizeof(var_run));