From 31de4a40e758f384e89b592806a64130833cac35 Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Thu, 28 May 2020 13:03:02 +0200 Subject: broadcom-wl: don't inherit lock descriptor in nas process Add a local hack to prevent the Broadcom WPA authenticator process from inheriting the lock descriptor 1000 used to prevent concurrent executions of the init script. Without this fix, repeated invocations of /etc/init.d/network, e.g. for obtaining the enabled state, would hang forever. Signed-off-by: Jo-Philipp Wich (cherry picked from commit a03d6d2fab13c478a0f6cfc3082bec141f2adcf1) --- package/kernel/broadcom-wl/files/lib/wifi/broadcom.sh | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'package/kernel/broadcom-wl/files/lib/wifi/broadcom.sh') diff --git a/package/kernel/broadcom-wl/files/lib/wifi/broadcom.sh b/package/kernel/broadcom-wl/files/lib/wifi/broadcom.sh index 3643f6b1c6..33447341b2 100644 --- a/package/kernel/broadcom-wl/files/lib/wifi/broadcom.sh +++ b/package/kernel/broadcom-wl/files/lib/wifi/broadcom.sh @@ -443,7 +443,10 @@ EOF txpower=${txpower:-$vif_txpower} [ -z "$txpower" ] || iwconfig $device txpower ${txpower}dBm - eval "$nas_cmd" + # fd 1000 is an inherited lock file descriptor for preventing concurrent + # init script executions. Close it here to prevent the nas daemon from + # inheriting it further to avoid holding the lock indefinitely. + eval "$nas_cmd 1000>&-" } -- cgit v1.2.3