android: For keyingtries > 0 notify the GUI if the limit is reached when reestablishing

The IKE_SA is destroyed anyway, so letting the GUI remain in
"connecting" state would be incorrect.

We still use keyingtries=0 for now, though. And we still abort after the
first failed attempt initially, in case there is a configuration error.

diff --git a/src/frontends/android/jni/libandroidbridge/backend/android_service.c b/src/frontends/android/jni/libandroidbridge/backend/android_service.c
index 5a85d30264f871948dabaa3a9af990ddd3b96448..e60c491c16d972a9f62613b0db4c49f5cfefb556 100644 (file)
--- a/src/frontends/android/jni/libandroidbridge/backend/android_service.c
+++ b/src/frontends/android/jni/libandroidbridge/backend/android_service.c
@@ -502,6 +502,23 @@ METHOD(listener_t, alert, bool,
                                                        (callback_job_cb_t)terminate, id, free,
                                                        (callback_job_cancel_t)return_false, JOB_PRIO_HIGH));
                                }
+                               else
+                               {
+                                       peer_cfg_t *peer_cfg;
+                                       u_int32_t tries, try;
+
+                                       /* when reestablishing and if keyingtries is not %forever
+                                        * the IKE_SA is destroyed after the set number of tries,
+                                        * so notify the GUI */
+                                       peer_cfg = ike_sa->get_peer_cfg(ike_sa);
+                                       tries = peer_cfg->get_keyingtries(peer_cfg);
+                                       try = va_arg(args, u_int32_t);
+                                       if (tries != 0 && try == tries-1)
+                                       {
+                                               charonservice->update_status(charonservice,
+                                                                                       CHARONSERVICE_UNREACHABLE_ERROR);
+                                       }
+                               }
                                this->lock->unlock(this->lock);
                                break;
                        default: