From: Florian Eckert <redacted>
Date: Fri, 24 Jul 2020 06:02:49 +0000 (+0200)
Subject: mwan3: revert: also cleanup lock on mwan3 stop
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=b5bd6d757b8ac9b3eb7fd1e19880924057cce7a7;p=openwrt-packages.git

mwan3: revert: also cleanup lock on mwan3 stop

This reverts commit cde2a77ed3b3b0df9e693e121dccdc97ef163156.

Applying this change has shown that it is even quicker to provoke the
race condtition on simultan mwan3 commands execution.
By reversing the change we have the same behaviour as before.

But the race condition on mwan3 execute at the same time still exists.

Signed-off-by: Florian Eckert <redacted>
---

diff --git a/net/mwan3/files/lib/mwan3/mwan3.sh b/net/mwan3/files/lib/mwan3/mwan3.sh
index 2ec760f9a..ab21ad3b2 100644
--- a/net/mwan3/files/lib/mwan3/mwan3.sh
+++ b/net/mwan3/files/lib/mwan3/mwan3.sh
@@ -204,13 +204,6 @@ mwan3_unlock() {
 }
 
 mwan3_lock_clean() {
-	for pid in $(pgrep -f "lock /var/run/mwan3.lock"); do
-		kill -TERM "$pid" > /dev/null 2>&1
-	done
-	sleep 1
-	for pid in $(pgrep -f "lock /var/run/mwan3.lock"); do
-		kill -KILL "$pid" > /dev/null 2>&1
-	done
 	rm -rf /var/run/mwan3.lock
 }