Completed state handling in isakmp_cert_pre

Should not be a problem, but makes static analyzers happy.
2012-09-28 19:01:09 +02:00 · 2012-09-28 19:01:09 +02:00 · 6c10cecebf
parent 9ff9c3d11b
commit 6c10cecebf
1 changed files with 4 additions and 0 deletions
--- 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)