On 01/14/2015 10:55 PM, David Lechner wrote:
The technology_set implementation in the bluetooth plugin returns
-EINPROGRESS which causes problems.

1.  When setting the Tethering property via D-Bus, D-Bus returns an error
     instead of completing successfuly.
2.  When using the PersistentTetheringMode option, bluetooth tethering
     is not remembered. This is because /var/lib/connman/settings is not
     updated if technology_set returns an error.

This fixes the issue by returning 0 when bluetooth_tech_set_tethering
completes successfully.
---
  plugins/bluetooth.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/bluetooth.c b/plugins/bluetooth.c
index c6f387e..636c7bf 100644
--- a/plugins/bluetooth.c
+++ b/plugins/bluetooth.c
@@ -899,7 +899,7 @@ static int bluetooth_tech_set_tethering(struct 
connman_technology *technology,
        if (i == 0)
                return -ENODEV;
- return -EINPROGRESS;
+       return 0;
  }
static struct connman_technology_driver tech_driver = {
Just want to make sure this patch did not get overlooked.
_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Reply via email to