commit db0f917217979bf7547e27b47faf1b49d25c8d9c
author Paul Stewart <pstew@chromium.org> Fri Nov 30 16:48:09 2012 -0800
committer Gerrit <chrome-bot@google.com> Tue Dec 04 12:45:46 2012 -0800
tree 214da6e1eeefeeb895fc59cfad6a99e735f38d89
parent 03c15ce0583d0c89bbd762b1140ace8d537c7625

shill: WiFi: Emit WiFiService errors on EAP Failures

Do not wait for wpa_supplicant to timeout on EAP failures.
Instead, report EAP failures immediately by setting the
most specific failure possible on the service and disconnecting.
By doing so, this empowers the UI (and in turn the user) to
take appropriate action to correct the authentication problem.

CQ-DEPEND=Ifae8eeb45c1f9ef5a9f0406482a15c0c9adca12c
BUG=chromium-os:31466
TEST=Unit tests, network_WiFiSecMat/network_WiFiSecMat.073CheckWPA_1x_TTLS,
network_WiFiSecMat/network_WiFiSecMat.019CheckWPA_1x_AES, observing that
the service enters the Failed state and the error is set appropriately
to the cause of failure.

Change-Id: I1d019f3148c64ed41d173aeae0a91409eb73e906
Reviewed-on: https://gerrit.chromium.org/gerrit/39026
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>
Commit-Ready: Paul Stewart <pstew@chromium.org>

supplicant_interface_proxy.cc

diff --git a/supplicant_interface_proxy.cc b/supplicant_interface_proxy.cc
index ad581a2..8df3727 100644
--- a/supplicant_interface_proxy.cc
+++ b/supplicant_interface_proxy.cc
@@ -173,6 +173,13 @@
   wifi_->Certification(properties);
 }
 
+void SupplicantInterfaceProxy::Proxy::EAP(
+    const string &status, const string &parameter) {
+  SLOG(DBus, 2) << __func__ << ": status " << status
+                << ", parameter " << parameter;
+  wifi_->EAPEvent(status, parameter);
+}
+
 void SupplicantInterfaceProxy::Proxy::BSSRemoved(const ::DBus::Path &BSS) {
   SLOG(DBus, 2) << __func__;
   wifi_->BSSRemoved(BSS);