Fix sending duplicate speaker/microphone gains to the headset
Current code only prevent duplicate D-Bus signals, so in case headset
changes the volume a client may set the same volume level again which
would be send as new volume level.
To fix this headset_set_gain now return -EALREADY if nothing has changed
so code using it can just ignore the change instead of sending to remote
device.
diff --git a/audio/headset.c b/audio/headset.c
index 55bdc61..01f91db 100644
--- a/audio/headset.c
+++ b/audio/headset.c
@@ -949,7 +949,7 @@
case HEADSET_GAIN_SPEAKER:
if (slc->sp_gain == gain) {
DBG("Ignoring no-change in speaker gain");
- return 0;
+ return -EALREADY;
}
name = "SpeakerGainChanged";
property = "SpeakerGain";
@@ -958,7 +958,7 @@
case HEADSET_GAIN_MICROPHONE:
if (slc->mic_gain == gain) {
DBG("Ignoring no-change in microphone gain");
- return 0;
+ return -EALREADY;
}
name = "MicrophoneGainChanged";
property = "MicrophoneGain";
@@ -995,7 +995,7 @@
gain = (dbus_uint16_t) strtol(&buf[7], NULL, 10);
err = headset_set_gain(device, gain, buf[5]);
- if (err < 0)
+ if (err < 0 && err != -EALREADY)
return err;
return headset_send(hs, "\r\nOK\r\n");
@@ -1873,10 +1873,14 @@
return btd_error_not_connected(msg);
err = headset_set_gain(device, gain, type);
- if (err < 0)
+ if (err < 0) {
+ /* Ignore if nothing has changed */
+ if (err == -EALREADY)
+ return dbus_message_new_method_return(msg);
return g_dbus_create_error(msg, ERROR_INTERFACE
".InvalidArgument",
"Must be less than or equal to 15");
+ }
reply = dbus_message_new_method_return(msg);
if (!reply)
