diff --git a/android/hal-audio.c b/android/hal-audio.c
index 4326ccd..52f8894 100644
--- a/android/hal-audio.c
+++ b/android/hal-audio.c
static int listen_sk = -1;
static int audio_sk = -1;
-static bool close_thread = false;
static pthread_t ipc_th = 0;
-static pthread_mutex_t close_mutex = PTHREAD_MUTEX_INITIALIZER;
static pthread_mutex_t sk_mutex = PTHREAD_MUTEX_INITIALIZER;
#if __BYTE_ORDER == __LITTLE_ENDIAN
return bytes;
}
-static void audio_ipc_cleanup(void)
-{
- if (audio_sk >= 0) {
- shutdown(audio_sk, SHUT_RDWR);
- audio_sk = -1;
- }
-}
-
static int audio_ipc_cmd(uint8_t service_id, uint8_t opcode, uint16_t len,
void *param, size_t *rsp_len, void *rsp, int *fd)
{
struct hal_status s;
size_t s_len = sizeof(s);
+ pthread_mutex_lock(&sk_mutex);
+
if (audio_sk < 0) {
error("audio: Invalid cmd socket passed to audio_ipc_cmd");
goto failed;
msg.msg_iov = iv;
msg.msg_iovlen = 2;
- pthread_mutex_lock(&sk_mutex);
-
ret = sendmsg(audio_sk, &msg, 0);
if (ret < 0) {
error("audio: Sending command failed:%s", strerror(errno));
- pthread_mutex_unlock(&sk_mutex);
goto failed;
}
if (ret < 0) {
error("audio: Receiving command response failed:%s",
strerror(errno));
- pthread_mutex_unlock(&sk_mutex);
goto failed;
}
- pthread_mutex_unlock(&sk_mutex);
-
if (ret < (ssize_t) sizeof(cmd)) {
error("audio: Too small response received(%zd bytes)", ret);
goto failed;
goto failed;
}
+ pthread_mutex_unlock(&sk_mutex);
+
return s->code;
}
+ pthread_mutex_unlock(&sk_mutex);
+
/* Receive auxiliary data in msg */
if (fd) {
struct cmsghdr *cmsg;
failed:
/* Some serious issue happen on IPC - recover */
shutdown(audio_sk, SHUT_RDWR);
- audio_sk = -1;
+ pthread_mutex_unlock(&sk_mutex);
+
return AUDIO_STATUS_FAILED;
}
DBG("");
- pthread_mutex_lock(&close_mutex);
- audio_ipc_cleanup();
- close_thread = true;
- pthread_mutex_unlock(&close_mutex);
+ unregister_endpoints();
+
+ shutdown(listen_sk, SHUT_RDWR);
+ shutdown(audio_sk, SHUT_RDWR);
pthread_join(ipc_th, NULL);
{
bool done = false;
struct pollfd pfd;
+ int sk;
DBG("");
while (!done) {
DBG("Waiting for connection ...");
- audio_sk = accept(listen_sk, NULL, NULL);
- if (audio_sk < 0) {
+
+ sk = accept(listen_sk, NULL, NULL);
+ if (sk < 0) {
int err = errno;
- error("audio: Failed to accept socket: %d (%s)", err,
- strerror(err));
- continue;
+
+ if (err == EINTR)
+ continue;
+
+ if (err != ECONNABORTED && err != EINVAL)
+ error("audio: Failed to accept socket: %d (%s)",
+ err, strerror(err));
+
+ break;
}
+ pthread_mutex_lock(&sk_mutex);
+ audio_sk = sk;
+ pthread_mutex_unlock(&sk_mutex);
+
DBG("Audio IPC: Connected");
if (register_endpoints() != AUDIO_STATUS_SUCCESS) {
unregister_endpoints();
+ pthread_mutex_lock(&sk_mutex);
shutdown(audio_sk, SHUT_RDWR);
+ close(audio_sk);
+ audio_sk = -1;
+ pthread_mutex_unlock(&sk_mutex);
+
continue;
}
if (pfd.revents & (POLLHUP | POLLERR | POLLNVAL)) {
info("Audio HAL: Socket closed");
+
+ pthread_mutex_lock(&sk_mutex);
+ close(audio_sk);
audio_sk = -1;
+ pthread_mutex_unlock(&sk_mutex);
}
-
- /*Check if audio_dev is closed */
- pthread_mutex_lock(&close_mutex);
- done = close_thread;
- close_thread = false;
- pthread_mutex_unlock(&close_mutex);
}
unregister_endpoints();