coova / coova-chilli

CoovaChilli is an open-source software access controller for captive portal hotspots.
Other
518 stars 260 forks source link

Allow --enable-location without --enable-proxyvsa #282

Closed timri closed 8 years ago

timri commented 8 years ago

Previously, --enable-location was dependant on --enable-proxyvsa, i.e. it used fields/functions ifdef-ed away on ENABLE_PROXYVSA

This patch replaces most of them by "#if defined(ENABLE_LOCATION) || defined(ENABLE_PROXYVSA)"

Since I don't use proxyvsa (mostly because of issue #262 ), I don't know if proxyvsa needs alls these fields, or if they could be guarded by "#ifdef ENABLE_LOCATION"

Also I only tested this with proxyloc=32 (NASIdentifier) on a cisco-AP, which is a non-VSA. Tracking location with a VSA might eventually not work.

Signed-off-by: Tim Riemenschneider git@tim-riemenschneider.de