Skip to contentSkip to conversation

Fix OCSP_check.sh

PATCH In review Steffan Karger 2 participants 2 messages Feb 25, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Fix OCSP_check.sh

As reported in trac #582, the OCSP_check.sh script should use grep -E, instead of grep -F when it uses ^ in the expression.

---
 contrib/OCSP_check/OCSP_check.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
contrib/OCSP_check/OCSP_check.sh +2 -2
index 6876c6d..2675788 100644
@@ -105,9 +105,9 @@ if [ $check_depth -eq -1 ] || [ $cur_depth -eq $check_depth ]; then
           exit 1
       fi
       # check that the reported status of certificate is ok
-      if echo "$status" | grep -Fq "^${serial}: good"; then
+      if echo "$status" | grep -Eq "^${serial}: good"; then
         # check if signature on the OCSP response verified correctly
-        if echo "$status" | grep -Fq "^Response verify OK"; then
+        if echo "$status" | grep -Eq "^Response verify OK"; then
             exit 0
         fi
       fi
Signature · 1 line
2.5.0
#1 lore raw

ACK, thanks.

Your patch has been applied to the master and release/2.3 branch.

commit ab0f846de6991345c30f5b69817304183d347e0e (master) commit 22f399f5509a56c7aaa57779b14a650d33d30e5e (master)

Author: Steffan Karger
Date:   Thu Feb 25 15:10:34 2016 +0100
     Fix OCSP_check.sh

     Signed-off-by: Steffan Karger <[email protected]...>
     Acked-by: Gert Doering <[email protected]...>
     Message-Id: <[email protected]...>
     URL: http://article.gmane.org/gmane.network.openvpn.devel/11254
     Signed-off-by: Gert Doering <[email protected]...>

--
kind regards,

Gert Doering