commit 56508259a620a1b5f5f2178ce7a2bf4c9135c1b5
author Christopher N. Hesse <raymanfx@gmail.com> Wed Jan 24 02:14:45 2018 +0100
committer Christopher N. Hesse <raymanfx@gmail.com> Mon Feb 05 14:05:28 2018 +0000
tree ba177cdfe0de5fd0a2a0763c3d6c4518cd788452
parent 5b6ec5682ec6ed5580db3f4a890f7c13f7a6edad

wifiloader: Make sure WiFi firmware path is writable

Change-Id: Ib0ae2a004d4207a4ccb3d551c9a6a092920595ca

wifiloader/Android.mk

diff --git a/wifiloader/Android.mk b/wifiloader/Android.mk
index 32b80b1..9d27fb7 100644
--- a/wifiloader/Android.mk
+++ b/wifiloader/Android.mk
@@ -9,6 +9,9 @@
 LOCAL_SHARED_LIBRARIES := \
     libcutils liblog libutils
 
+LOCAL_C_INCLUDES := \
+	system/core/include
+
 ifneq ($(WIFI_DRIVER_MODULE_NAME),)
 LOCAL_CFLAGS += -DWIFI_DRIVER_MODULE_NAME=\"$(WIFI_DRIVER_MODULE_NAME)\"
 endif
@@ -17,6 +20,10 @@
 LOCAL_CFLAGS += -DWIFI_DRIVER_MODULE_PATH=\"$(WIFI_DRIVER_MODULE_PATH)\"
 endif
 
+ifneq ($(WIFI_DRIVER_FW_PATH_PARAM),)
+LOCAL_CFLAGS += -DWIFI_DRIVER_FW_PATH_PARAM=\"$(WIFI_DRIVER_FW_PATH_PARAM)\"
+endif
+
 LOCAL_MODULE := wifiloader
 LOCAL_MODULE_TAGS := optional
 

wifiloader/wifiloader.c

diff --git a/wifiloader/wifiloader.c b/wifiloader/wifiloader.c
index c827aea..0aeec3e 100644
--- a/wifiloader/wifiloader.c
+++ b/wifiloader/wifiloader.c
@@ -28,6 +28,8 @@
 #include <sys/stat.h>
 #include <sys/syscall.h>
 
+#include <private/android_filesystem_config.h>
+
 #define DEFERRED_INITCALLS        "/proc/deferred_initcalls"
 
 #ifndef WIFI_DRIVER_MODULE_NAME
@@ -38,6 +40,10 @@
 #define WIFI_DRIVER_MODULE_PATH   "/system/lib/modules/" WIFI_DRIVER_MODULE_NAME ".ko"
 #endif
 
+#ifndef WIFI_DRIVER_FW_PATH_PARAM
+#define WIFI_DRIVER_FW_PATH_PARAM "/sys/module/wlan/parameters/fwpath"
+#endif
+
 #define finit_module(fd, params, flags) syscall(__NR_finit_module, fd, params, flags)
 
 
@@ -84,6 +90,14 @@
         return -errno;
     }
 
+    // setup proper permissions for firmware change
+    if (chown(WIFI_DRIVER_FW_PATH_PARAM, AID_WIFI, AID_WIFI) != 0) {
+        ALOGE("Failed to chown firmware path %s - error: %s",
+              WIFI_DRIVER_FW_PATH_PARAM, strerror(errno));
+        close(fd);
+        return -errno;
+    }
+
     // let wifi HAL know we succeeded
     ALOGV("Successfully loaded WLAN module: %s", WIFI_DRIVER_MODULE_NAME);
     property_set("wlan.driver.status", "ok");