فهرست منبع

package/mdnsd/S50mdnsd: do not clobber exit code

As pointed out by shellcheck, the exit code of the start/stop/restart/reload
command is clobbered by the 'echo "FAIL'" statement:

In package/mdnsd/S50mdnsd line 52:
exit $?
     ^-- SC2320 (warning): This $? refers to echo/printf, not a previous command. Assign to variable to avoid it being overwritten.

So introduce a $status variable to keep track of it, similar to how it is
done in S40iwd.

Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
(cherry picked from commit be20c12e157c0e67875d8031de17df41a9c55679)
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
Peter Korsgaard 5 ماه پیش
والد
کامیت
2f23cd8680
1فایلهای تغییر یافته به همراه4 افزوده شده و 2 حذف شده
  1. 4 2
      package/mdnsd/S50mdnsd

+ 4 - 2
package/mdnsd/S50mdnsd

@@ -36,7 +36,9 @@ reload() {
 
 case "$1" in
 	start|stop|restart|reload)
-		if "$1"; then
+		"$1"
+		status=$?
+		if [ "$status" -eq 0 ]; then
 			echo "OK"
 		else
 			echo "FAIL"
@@ -48,4 +50,4 @@ case "$1" in
 		;;
 esac
 
-exit $?
+exit "$status"