Android: Set ctrl_iface client socket group (AID_WIFI) separately

Split chown() call in wpa_ctrl_open() and wpa_ctrl_open2() to allow the
group id to be set even if the process does not have privileges to
change the owner. This is needed for modules that need to communicate
with wpa_supplicant since without the group change, wpa_supplicant may
not have privileges to send the response to a control interface command.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>

diff --git a/src/common/wpa_ctrl.c b/src/common/wpa_ctrl.c
index 820dd12..5733aa6 100644 (file)
--- a/src/common/wpa_ctrl.c
+++ b/src/common/wpa_ctrl.c
@@ -152,6 +152,8 @@ try_again:
 
 #ifdef ANDROID
        chmod(ctrl->local.sun_path, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP);
+       /* Set group even if we do not have privileges to change owner */
+       chown(ctrl->local.sun_path, -1, AID_WIFI);
        chown(ctrl->local.sun_path, AID_SYSTEM, AID_WIFI);
 
        if (os_strncmp(ctrl_path, "@android:", 9) == 0) {