From 21f3cf257e78b24d362f5d5e50c43942fa0ab440 Mon Sep 17 00:00:00 2001 From: schwarze <> Date: Sun, 9 Oct 2011 16:46:55 +0000 Subject: [PATCH] When the _RC_RUNFILE (by some ill chance) happened to be empty or reading it failed, ${pexp} ended up as the empty string and the script would send SIGTERM to init(1), which was really inconvenient. Fix that by never allowing pexp to become empty. My patch considerably simplified by and ok ajacoutot@. --- src/etc/rc.d/rc.subr | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/etc/rc.d/rc.subr b/src/etc/rc.d/rc.subr index 5c45d5fa..58af023f 100644 --- a/src/etc/rc.d/rc.subr +++ b/src/etc/rc.d/rc.subr @@ -1,4 +1,4 @@ -# $OpenBSD: rc.subr,v 1.52 2011/10/09 08:48:53 ajacoutot Exp $ +# $OpenBSD: rc.subr,v 1.53 2011/10/09 16:46:55 schwarze Exp $ # # Copyright (c) 2010, 2011 Antoine Jacoutot # Copyright (c) 2010, 2011 Ingo Schwarze @@ -34,7 +34,8 @@ rc_write_runfile() { } rc_read_runfile() { - [ -f ${_RC_RUNFILE} ] && pexp=$(< ${_RC_RUNFILE}) + [ -f ${_RC_RUNFILE} ] && local _new_pexp=$(< ${_RC_RUNFILE}) + [ -n "${_new_pexp}" ] && pexp="${_new_pexp}" } rc_rm_runfile() {