[pkg-wpa-devel] Bug#615821: hostapd: init file check for readable conf file breaks when specifying multiple conf files

[Johnny Casey]

Package: hostapd
Version: 1:0.6.10-2
Severity: normal


Hostapd supports running with multiple configuration files.  (I'm pretty sure
I didn't modify the init file before, so this is a change in the start up
process.)  However the [ -r "$DAEMON_CONF" ] check assumes that only one file
would ever be used.  Either it needs to just check for non-empty string, or
filter the contents of the variable (which seems like overkill).  A message
(as #568181) would help too.

As an aside, I understand that my configuration is a little odd.  I have
madwifi configured with two interfaces to avoid confusing Apple's OS X stack.
Having one interface exposing both PSK and Enterprise was just too confusing
for it.

-- System Information:
Debian Release: 6.0
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 2.6.18-skas3-styx
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/bash

Versions of packages hostapd depends on:
ii  libc6                   2.11.2-10        Embedded GNU C Library: Shared lib
ii  libnl1                  1.1-6            library for dealing with netlink s
ii  libssl0.9.8             0.9.8o-4squeeze1 SSL shared libraries
ii  lsb-base                3.2-23.2squeeze1 Linux Standard Base 3.2 init scrip

hostapd recommends no packages.

hostapd suggests no packages.

-- Configuration Files:
/etc/default/hostapd changed:
DAEMON_CONF="/etc/hostapd/hostapd.conf /etc/hostapd/hostapd-dev.conf"

/etc/hostapd/hostapd.conf [Errno 13] Permission denied: u'/etc/hostapd/hostapd.conf'
/etc/init.d/hostapd changed:
PATH=/sbin:/bin:/usr/sbin:/usr/bin
DAEMON_SBIN=/usr/sbin/hostapd
DAEMON_DEFS=/etc/default/hostapd
DAEMON_CONF=
NAME=hostapd
DESC="advanced IEEE 802.11 management"
PIDFILE=/var/run/hostapd.pid
[ -x "$DAEMON_SBIN" ] || exit 0
[ -s "$DAEMON_DEFS" ] && . /etc/default/hostapd
DAEMON_OPTS="-B -P $PIDFILE $DAEMON_OPTS $DAEMON_CONF"
.. /lib/lsb/init-functions
case "$1" in
  start)
	log_daemon_msg "Starting $DESC" "$NAME"
	start-stop-daemon --start --oknodo --quiet --exec "$DAEMON_SBIN" \
		--pidfile "$PIDFILE" -- $DAEMON_OPTS >/dev/null
	log_end_msg "$?"
	;;
  stop)
	log_daemon_msg "Stopping $DESC" "$NAME"
	start-stop-daemon --stop --oknodo --quiet --exec "$DAEMON_SBIN" \
		--pidfile "$PIDFILE"
	log_end_msg "$?"
	;;
  reload)
  	log_daemon_msg "Reloading $DESC" "$NAME"
	start-stop-daemon --stop --signal HUP --exec "$DAEMON_SBIN" \
		--pidfile "$PIDFILE"
	log_end_msg "$?"
	;;
  restart|force-reload)
  	$0 stop
	sleep 8
	$0 start
	;;
  status)
	status_of_proc "$DAEMON_SBIN" "$NAME"
	exit $?
	;;
  *)
	N=/etc/init.d/$NAME
	echo "Usage: $N {start|stop|restart|force-reload|reload|status}" >&2
	exit 1
	;;
esac
exit 0


-- no debconf information