openwrt / packages

Community maintained packages for OpenWrt. Documentation for submitting pull requests is in CONTRIBUTING.md
GNU General Public License v2.0
4.01k stars 3.48k forks source link

choova-chilli - missing parameters #1221

Closed obsy closed 8 years ago

obsy commented 9 years ago

Coova-chilli's config defines option dhcpif. This parameter does not exists, because init script parses option network, not dhcpif.

teslamint commented 9 years ago

option network just converts interface name(e.g. wan, lan) to interface name(ethx.x) and assigns to option dhcpif afaik.

obsy commented 9 years ago

In coova-chilli config, but not in /etc/config/chilli

teslamint commented 9 years ago

Well, if i set network option with bridge network interface(e.g: br-wifi) with multiple devices(wlanX.X), I got this:

Mon May 18 13:34:58 2015 local6.err coova-chilli[13843]: net.c: 80: 19 (No such device) ioctl(SIOCSIFFLAGS) failed on
Mon May 18 13:34:58 2015 local6.err coova-chilli[13843]: net.c: 57: 19 (No such device) ioctl(SIOCSIFFLAGS) failed
Mon May 18 13:34:58 2015 local6.err coova-chilli[13843]: net.c: 120: 19 (No such device) ioctl(SIOCSIFADDR) failed
Mon May 18 13:34:58 2015 local6.err coova-chilli[13843]: net.c: 1291: 19 (No such device) ioctl(d=5, request=35111) failed
Mon May 18 13:34:58 2015 local6.err coova-chilli[13843]: chilli.c: 7339: Failed to create dhcp listener on

so I think it's better use dhcpif option than network option in that case.

obsy commented 9 years ago

Strange, the script gets the physical interface from network option. In coova config exists only dhcpif. Ask coovachilli developer about this.

teslamint commented 9 years ago

It would be OK if we use single physical interface(e.g. eth1.0, wlan0.1, etc.) on network option. EDIT: 'wifi', 'lan' works, but not actual interface name such as 'br-wifi', 'br-lan', etc.