From: Tobias Brunner <tobias@strongswan.org>
Date: Fri, 28 Sep 2012 17:01:09 +0000 (+0200)
Subject: Completed state handling in isakmp_cert_pre
X-Git-Tag: 5.0.1~10
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=6c10cecebf67d3d9ab52053ee663345a732c87c7;p=thirdparty%2Fstrongswan.git

Completed state handling in isakmp_cert_pre

Should not be a problem, but makes static analyzers happy.
---

diff --git a/src/libcharon/sa/ikev1/tasks/isakmp_cert_pre.c b/src/libcharon/sa/ikev1/tasks/isakmp_cert_pre.c
index a40bdf176b..d48484f090 100644
--- a/src/libcharon/sa/ikev1/tasks/isakmp_cert_pre.c
+++ b/src/libcharon/sa/ikev1/tasks/isakmp_cert_pre.c
@@ -410,6 +410,8 @@ METHOD(task_t, process_r, status_t,
 					process_certreqs(this, message);
 					process_certs(this, message);
 					return SUCCESS;
+				default:
+					return FAILED;
 			}
 		}
 		case AGGRESSIVE:
@@ -455,6 +457,8 @@ METHOD(task_t, build_r, status_t,
 					return NEED_MORE;
 				case CR_AUTH:
 					return NEED_MORE;
+				default:
+					return FAILED;
 			}
 		case AGGRESSIVE:
 			switch (this->state)