[pkg-wpa-devel] r1078 - in /wpasupplicant/trunk/debian: changelog ifupdown/functions.sh

kelmo-guest at users.alioth.debian.org kelmo-guest at users.alioth.debian.org
Thu Jan 31 01:56:32 UTC 2008


Author: kelmo-guest
Date: Thu Jan 31 01:56:32 2008
New Revision: 1078

URL: http://svn.debian.org/wsvn/pkg-wpa/?sc=1&rev=1078
Log:
* Start daemon in quiet mode by default, the scan event is being written to
  logfile far too often. [debian/ifupdown/functions.sh]
* When starting daemon with debug option, include timestamps in logfile.
* Simplfy return check in init_wpa_supplicant(), init_wpa_cli()
  kill_wpa_supplicant() and kill_wpa_cli() reduce one level of indentation
  for each function. [debian/ifupdown/functions.sh]

Modified:
    wpasupplicant/trunk/debian/changelog
    wpasupplicant/trunk/debian/ifupdown/functions.sh

Modified: wpasupplicant/trunk/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-wpa/wpasupplicant/trunk/debian/changelog?rev=1078&op=diff
==============================================================================
--- wpasupplicant/trunk/debian/changelog (original)
+++ wpasupplicant/trunk/debian/changelog Thu Jan 31 01:56:32 2008
@@ -82,8 +82,15 @@
     CTRL-EVENT-SCAN-RESULTS.
       - debian/patches/90_wpa_gui_qt4_update_scan_results_on_event.patch
       - debian/patches/91_wpa_gui_qt4_scanres_bss_iterative.patch
-
- -- Kel Modderman <kel at otaku42.de>  Thu, 31 Jan 2008 04:03:31 +1000
+  * Start daemon in quiet mode by default, the scan event is being written to
+    logfile far too often. [debian/ifupdown/functions.sh]
+  * When starting daemon with debug option, include timestamps in logfile.
+    [debian/ifupdown/functions.sh]
+  * Simplfy return check in init_wpa_supplicant(), init_wpa_cli()
+    kill_wpa_supplicant() and kill_wpa_cli() reduce one level of indentation
+    for each function. [debian/ifupdown/functions.sh]
+
+ -- Kel Modderman <kel at otaku42.de>  Thu, 31 Jan 2008 11:50:14 +1000
 
 wpasupplicant (0.6.2-1) experimental; urgency=low
 

Modified: wpasupplicant/trunk/debian/ifupdown/functions.sh
URL: http://svn.debian.org/wsvn/pkg-wpa/wpasupplicant/trunk/debian/ifupdown/functions.sh?rev=1078&op=diff
==============================================================================
--- wpasupplicant/trunk/debian/ifupdown/functions.sh (original)
+++ wpasupplicant/trunk/debian/ifupdown/functions.sh Thu Jan 31 01:56:32 2008
@@ -187,112 +187,124 @@
 # -f	path to log file
 #
 init_wpa_supplicant () {
-	if [ -n "$WPA_SUP_CONF" ]; then
-		local WPA_SUP_OPTIONS
-
-		if [ -n "$WPA_ACTION_SCRIPT" ]; then
-			if [ -x "$WPA_ACTION_SCRIPT" ]; then
-				WPA_SUP_OPTIONS="-W -B -P $WPA_SUP_PIDFILE -i $WPA_IFACE"
-				wpa_msg verbose "wait for wpa_cli to attach"
-			else
-				wpa_msg stderr "action script \"$WPA_ACTION_SCRIPT\" not executable"
-				return 1
-			fi
+	[ -n "$WPA_SUP_CONF" ] || return 0
+
+	local WPA_SUP_OPTIONS
+
+	if [ -n "$WPA_ACTION_SCRIPT" ]; then
+		if [ -x "$WPA_ACTION_SCRIPT" ]; then
+			WPA_SUP_OPTIONS="-W -B -P $WPA_SUP_PIDFILE -i $WPA_IFACE"
+			wpa_msg verbose "wait for wpa_cli to attach"
 		else
-			WPA_SUP_OPTIONS="-B -P $WPA_SUP_PIDFILE -i $WPA_IFACE"
-		fi
-
-		if [ -n "$IF_WPA_BRIDGE" ]; then
-			WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -b $IF_WPA_BRIDGE"
-			wpa_msg verbose "wpa-bridge $IF_WPA_BRIDGE"
-		fi
-
-		if [ -n "$IF_WPA_DRIVER" ]; then
-			WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -D $IF_WPA_DRIVER"
-			wpa_msg verbose "wpa-driver $IF_WPA_DRIVER"
-		else
-			WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -D wext"
-			wpa_msg verbose "using default driver type: wpa-driver wext"
-		fi
-
-		if [ -n "$IF_WPA_DEBUG_LEVEL" ]; then
-			case "$IF_WPA_DEBUG_LEVEL" in
-				3)
-					WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -ddd"
-					;;
-				2)
-					WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -dd"
-					;;
-				1)
-					WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -d"
-					;;
-				0)
-					# default verbosity
-					;;
-				-1)
-					WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -q"
-					;;
-				-2)
-					WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -qq"
-					;;
-			esac
-			wpa_msg verbose "using debug level: $IF_WPA_DEBUG_LEVEL"
-		fi
-
-		if [ -n "$IF_WPA_LOGFILE" ]; then
-			# custom log file
-			WPA_SUP_LOGFILE="$IF_WPA_LOGFILE"
-			wpa_msg verbose "logging to $IF_WPA_LOGFILE"
-		fi
-
-		if [ -d "${WPA_SUP_LOGFILE%/*}" ]; then
-			WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -f $WPA_SUP_LOGFILE"
-		else
-			wpa_msg stderr "unable to log output to $WPA_SUP_LOGFILE"
-		fi
-
-		wpa_msg verbose "$WPA_SUP_BIN $WPA_SUP_OPTIONS $WPA_SUP_CONF"
-			
-		start-stop-daemon --start --oknodo $DAEMON_VERBOSITY \
-			--name $WPA_SUP_PNAME --startas $WPA_SUP_BIN --pidfile $WPA_SUP_PIDFILE \
-			-- $WPA_SUP_OPTIONS $WPA_SUP_CONF
-
-		if [ "$?" -ne 0 ]; then
-			wpa_msg stderr "$WPA_SUP_BIN daemon failed to start"
+			wpa_msg stderr "action script \"$WPA_ACTION_SCRIPT\" not executable"
 			return 1
 		fi
-
-		if [ -s "$WPA_SUP_PIDFILE" ]; then
-			if [ -n "$WPA_SUP_OMIT_PIDFILE" ]; then
-				wpa_msg verbose "creating sendsigs omission pidfile: $WPA_SUP_OMIT_PIDFILE"
-				cat "$WPA_SUP_PIDFILE" > "$WPA_SUP_OMIT_PIDFILE"
-			else
-				wpa_msg verbose "sendsigs omission pidfile not created"
-			fi
-		else
-			wpa_msg stderr "pidfile missing or empty: $WPA_SUP_PIDFILE"
-		fi
-		
-		local WPA_SOCKET_WAIT MAX_WPA_SOCKET_WAIT
-		WPA_SOCKET_WAIT="0"
-		MAX_WPA_SOCKET_WAIT="5"
-		until [ -S "$WPA_CTRL_DIR/$WPA_IFACE" ]; do
-			if [ "$WPA_SOCKET_WAIT" -ge "$MAX_WPA_SOCKET_WAIT" ]; then
-				wpa_msg stderr "ctrl_interface socket not found at $WPA_CTRL_DIR/$WPA_IFACE"
+	else
+		WPA_SUP_OPTIONS="-B -P $WPA_SUP_PIDFILE -i $WPA_IFACE"
+	fi
+
+	if [ -n "$IF_WPA_BRIDGE" ]; then
+		WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -b $IF_WPA_BRIDGE"
+		wpa_msg verbose "wpa-bridge $IF_WPA_BRIDGE"
+	fi
+
+	if [ -n "$IF_WPA_DRIVER" ]; then
+		WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -D $IF_WPA_DRIVER"
+		wpa_msg verbose "wpa-driver $IF_WPA_DRIVER"
+	else
+		WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -D wext"
+		wpa_msg verbose "using default driver type: wpa-driver wext"
+	fi
+
+	if [ -n "$IF_WPA_DEBUG_LEVEL" ]; then
+		case "$IF_WPA_DEBUG_LEVEL" in
+			3)
+				WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -t -ddd"
+				;;
+			2)
+				WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -t -dd"
+				;;
+			1)
+				WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -t -d"
+				;;
+			0)
+				# wpa_supplicant default verbosity
+				;;
+			-1)
+				WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -q"
+				;;
+			-2)
+				WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -qq"
+				;;
+		esac
+		wpa_msg verbose "using debug level: $IF_WPA_DEBUG_LEVEL"
+	else
+		# log warnings only by default
+		WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -q"
+	fi
+
+	if [ -n "$IF_WPA_LOGFILE" ]; then
+		# custom log file
+		WPA_SUP_LOGFILE="$IF_WPA_LOGFILE"
+		wpa_msg verbose "logging to $IF_WPA_LOGFILE"
+	fi
+
+	if [ -d "${WPA_SUP_LOGFILE%/*}" ]; then
+		WPA_SUP_OPTIONS="$WPA_SUP_OPTIONS -f $WPA_SUP_LOGFILE"
+	else
+		wpa_msg stderr "unable to log output to $WPA_SUP_LOGFILE"
+	fi
+
+	wpa_msg verbose "$WPA_SUP_BIN $WPA_SUP_OPTIONS $WPA_SUP_CONF"
+		
+	start-stop-daemon --start --oknodo $DAEMON_VERBOSITY \
+		--name $WPA_SUP_PNAME --startas $WPA_SUP_BIN --pidfile $WPA_SUP_PIDFILE \
+		-- $WPA_SUP_OPTIONS $WPA_SUP_CONF
+
+	if [ "$?" -ne 0 ]; then
+		wpa_msg stderr "$WPA_SUP_BIN daemon failed to start"
+		return 1
+	fi
+
+	if [ -n "$WPA_SUP_OMIT_PIDFILE" ]; then
+		local WPA_PIDFILE_WAIT MAX_WPA_PIDFILE_WAIT
+		WPA_PIDFILE_WAIT="0"
+		MAX_WPA_PIDFILE_WAIT="5"
+		until [ -s "$WPA_SUP_PIDFILE" ]; do
+			if [ "$WPA_PIDFILE_WAIT" -ge "$MAX_WPA_PIDFILE_WAIT" ]; then
+				wpa_msg stderr "timed out waiting for creation of $WPA_SUP_PIDFILE"
 				return 1
 			else
-				wpa_msg verbose "waiting for \"$WPA_CTRL_DIR/$WPA_IFACE\": $WPA_SOCKET_WAIT (max. $MAX_WPA_SOCKET_WAIT)"
+				wpa_msg verbose "waiting for \"$WPA_SUP_PIDFILE\": " \
+					"$WPA_PIDFILE_WAIT (max. $MAX_WPA_PIDFILE_WAIT)"
 			fi
-			
-			WPA_SOCKET_WAIT=$(($WPA_SOCKET_WAIT + 1))
+
+			WPA_PIDFILE_WAIT=$(($WPA_PIDFILE_WAIT + 1))
 			sleep 1
 		done
-		
-		wpa_msg verbose "ctrl_interface socket located at $WPA_CTRL_DIR/$WPA_IFACE"
-	else
-		wpa_msg stderr "init_wpa_supplicant() called without WPA_SUP_CONF"
-		return 1
-	fi
+		wpa_msg verbose "creating sendsigs omission pidfile: $WPA_SUP_OMIT_PIDFILE"
+		cat "$WPA_SUP_PIDFILE" > "$WPA_SUP_OMIT_PIDFILE"
+	else
+		wpa_msg verbose "sendsigs omission pidfile not created"
+	fi
+
+	local WPA_SOCKET_WAIT MAX_WPA_SOCKET_WAIT
+	WPA_SOCKET_WAIT="0"
+	MAX_WPA_SOCKET_WAIT="5"
+	until [ -S "$WPA_CTRL_DIR/$WPA_IFACE" ]; do
+		if [ "$WPA_SOCKET_WAIT" -ge "$MAX_WPA_SOCKET_WAIT" ]; then
+			wpa_msg stderr "ctrl_interface socket not found at $WPA_CTRL_DIR/$WPA_IFACE"
+			return 1
+		else
+			wpa_msg verbose "waiting for \"$WPA_CTRL_DIR/$WPA_IFACE\": " \
+				"$WPA_SOCKET_WAIT (max. $MAX_WPA_SOCKET_WAIT)"
+		fi
+		
+		WPA_SOCKET_WAIT=$(($WPA_SOCKET_WAIT + 1))
+		sleep 1
+	done
+	
+	wpa_msg verbose "ctrl_interface socket located at $WPA_CTRL_DIR/$WPA_IFACE"
 }
 
 #####################################################################
@@ -301,20 +313,20 @@
 # the pidfile or ctrl_interface socket path and interface name
 #
 kill_wpa_supplicant () {
-	if test_wpa_supplicant; then
-		wpa_msg verbose "terminating $WPA_SUP_PNAME daemon via pidfile $WPA_SUP_PIDFILE"
-
-		start-stop-daemon --stop --oknodo $DAEMON_VERBOSITY \
-			--exec $WPA_SUP_BIN --pidfile $WPA_SUP_PIDFILE
-	
-		if [ -f "$WPA_SUP_PIDFILE" ]; then
-			rm -f "$WPA_SUP_PIDFILE"
-		fi
-
-		if [ -f "$WPA_SUP_OMIT_PIDFILE" ]; then
-			wpa_msg verbose "removing $WPA_SUP_OMIT_PIDFILE"
-			rm -f "$WPA_SUP_OMIT_PIDFILE"
-		fi
+	test_wpa_supplicant || return 0
+
+	wpa_msg verbose "terminating $WPA_SUP_PNAME daemon via pidfile $WPA_SUP_PIDFILE"
+
+	start-stop-daemon --stop --oknodo $DAEMON_VERBOSITY \
+		--exec $WPA_SUP_BIN --pidfile $WPA_SUP_PIDFILE
+
+	if [ -f "$WPA_SUP_PIDFILE" ]; then
+		rm -f "$WPA_SUP_PIDFILE"
+	fi
+
+	if [ -f "$WPA_SUP_OMIT_PIDFILE" ]; then
+		wpa_msg verbose "removing $WPA_SUP_OMIT_PIDFILE"
+		rm -f "$WPA_SUP_OMIT_PIDFILE"
 	fi
 }
 
@@ -344,31 +356,42 @@
 # -B	background process
 #
 init_wpa_cli () {
-	if [ -n "$WPA_ACTION_SCRIPT" ]; then
-		local WPA_CLI_OPTIONS
-		WPA_CLI_OPTIONS="-B -P $WPA_CLI_PIDFILE -i $WPA_IFACE"
-
-		wpa_msg verbose "$WPA_CLI_BIN $WPA_CLI_OPTIONS -p $WPA_CTRL_DIR -a $WPA_ACTION_SCRIPT"
-			
-		start-stop-daemon --start --oknodo $DAEMON_VERBOSITY \
-			--name $WPA_CLI_PNAME --startas $WPA_CLI_BIN --pidfile $WPA_CLI_PIDFILE \
-			-- $WPA_CLI_OPTIONS -p $WPA_CTRL_DIR -a $WPA_ACTION_SCRIPT
-
-		if [ "$?" -ne 0 ]; then
-			wpa_msg stderr "$WPA_CLI_BIN daemon failed to start"
-			return 1
-		fi
-
-		if [ -s "$WPA_CLI_PIDFILE" ]; then
-			if [ -n "$WPA_CLI_OMIT_PIDFILE" ]; then
-				wpa_msg verbose "creating sendsigs omission pidfile: $WPA_CLI_OMIT_PIDFILE"
-				cat "$WPA_CLI_PIDFILE" > "$WPA_CLI_OMIT_PIDFILE"
+	[ -n "$WPA_ACTION_SCRIPT" ] || return 0
+
+	local WPA_CLI_OPTIONS
+	WPA_CLI_OPTIONS="-B -P $WPA_CLI_PIDFILE -i $WPA_IFACE"
+
+	wpa_msg verbose "$WPA_CLI_BIN $WPA_CLI_OPTIONS -p $WPA_CTRL_DIR -a $WPA_ACTION_SCRIPT"
+		
+	start-stop-daemon --start --oknodo $DAEMON_VERBOSITY \
+		--name $WPA_CLI_PNAME --startas $WPA_CLI_BIN --pidfile $WPA_CLI_PIDFILE \
+		-- $WPA_CLI_OPTIONS -p $WPA_CTRL_DIR -a $WPA_ACTION_SCRIPT
+
+	if [ "$?" -ne 0 ]; then
+		wpa_msg stderr "$WPA_CLI_BIN daemon failed to start"
+		return 1
+	fi
+
+	if [ -n "$WPA_CLI_OMIT_PIDFILE" ]; then
+		local WPA_PIDFILE_WAIT MAX_WPA_PIDFILE_WAIT
+		WPA_PIDFILE_WAIT="0"
+		MAX_WPA_PIDFILE_WAIT="5"
+		until [ -s "$WPA_CLI_PIDFILE" ]; do
+			if [ "$WPA_PIDFILE_WAIT" -ge "$MAX_WPA_PIDFILE_WAIT" ]; then
+				wpa_msg stderr "timed out waiting for creation of $WPA_CLI_PIDFILE"
+				return 1
 			else
-				wpa_msg verbose "sendsigs omission pidfile not created"
+				wpa_msg verbose "waiting for \"$WPA_CLI_PIDFILE\": " \
+					"$WPA_PIDFILE_WAIT (max. $MAX_WPA_PIDFILE_WAIT)"
 			fi
-		else
-			wpa_msg stderr "empty or missing: $WPA_CLI_PIDFILE"
-		fi
+
+			WPA_PIDFILE_WAIT=$(($WPA_PIDFILE_WAIT + 1))
+			sleep 1
+		done
+		wpa_msg verbose "creating sendsigs omission pidfile: $WPA_CLI_OMIT_PIDFILE"
+		cat "$WPA_CLI_PIDFILE" > "$WPA_CLI_OMIT_PIDFILE"
+	else
+		wpa_msg verbose "sendsigs omission pidfile not created"
 	fi
 }
 
@@ -378,19 +401,19 @@
 # pidfile
 #
 kill_wpa_cli () {
-	if test_wpa_cli; then
-		wpa_msg verbose "terminating $WPA_CLI_PNAME daemon via pidfile $WPA_CLI_PIDFILE"
-		
-		start-stop-daemon --stop --oknodo $DAEMON_VERBOSITY \
-			--exec $WPA_CLI_BIN --pidfile $WPA_CLI_PIDFILE
-		
-		if [ -f "$WPA_CLI_PIDFILE" ]; then
-			rm -f "$WPA_CLI_PIDFILE"
-		fi
-
-		if [ -f "$WPA_CLI_OMIT_PIDFILE" ]; then
-			rm -f "$WPA_CLI_OMIT_PIDFILE"
-		fi
+	test_wpa_cli || return 0
+	
+	wpa_msg verbose "terminating $WPA_CLI_PNAME daemon via pidfile $WPA_CLI_PIDFILE"
+	
+	start-stop-daemon --stop --oknodo $DAEMON_VERBOSITY \
+		--exec $WPA_CLI_BIN --pidfile $WPA_CLI_PIDFILE
+	
+	if [ -f "$WPA_CLI_PIDFILE" ]; then
+		rm -f "$WPA_CLI_PIDFILE"
+	fi
+
+	if [ -f "$WPA_CLI_OMIT_PIDFILE" ]; then
+		rm -f "$WPA_CLI_OMIT_PIDFILE"
 	fi
 }
 




More information about the Pkg-wpa-devel mailing list