From 87e9cff065c9add436e4a9488239a89835b1bc48 Mon Sep 17 00:00:00 2001 From: Michael Orlitzky <michael@orlitzky.com> Date: Sun, 15 Jul 2018 16:49:49 -0400 Subject: [PATCH] files/fail2ban-openrc.init.in: remove redundant "return" from start_pre. OpenRC functions will exit with the return code from the last command by default, so there's no need for the "|| return 1" in our single-line start_pre() phase. --- files/fail2ban-openrc.init.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/files/fail2ban-openrc.init.in b/files/fail2ban-openrc.init.in index a2f4d34f..ad977274 100755 --- a/files/fail2ban-openrc.init.in +++ b/files/fail2ban-openrc.init.in @@ -44,7 +44,7 @@ depend() { } start_pre() { - checkpath -d "${FAIL2BAN_RUNDIR}" || return 1 + checkpath -d "${FAIL2BAN_RUNDIR}" } reload() {