Before calling service_connect(), __connman_service_connet() has cleared
any errors the service has had. With that in mind it makes no sense to
compare against something that can not happen.
---
 src/service.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/service.c b/src/service.c
index ac7d49f..8003a68 100644
--- a/src/service.c
+++ b/src/service.c
@@ -5875,10 +5875,9 @@ static int service_connect(struct connman_service 
*service)
                                if (!service->wps ||
                                        
!connman_network_get_bool(service->network, "WiFi.UseWPS"))
                                        return -ENOKEY;
-                       } else if (service->error ==
-                                       CONNMAN_SERVICE_ERROR_INVALID_KEY)
-                               return -ENOKEY;
+                       }
                        break;
+
                case CONNMAN_SERVICE_SECURITY_8021X:
                        if (!service->eap)
                                return -EINVAL;
-- 
2.1.1

_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman

Reply via email to