From 9de0241df1381a89cb07b4f3407b0580600b1d27 Mon Sep 17 00:00:00 2001
From: Edgar Aroutiounian <edgar.factorial@gmail.com>
Date: Tue, 12 May 2020 23:34:24 -0700
Subject: [PATCH] [validation] Use label for break

---
 validation.go | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/validation.go b/validation.go
index 364591d..3e235c8 100644
--- a/validation.go
+++ b/validation.go
@@ -270,12 +270,13 @@ func (v *validation) validate(vals []*topicVal, src peer.ID, msg *Message) {
 
 	// apply inline (synchronous) validators
 	result := ValidationAccept
+loop:
 	for _, val := range inline {
 		switch val.validateMsg(v.p.ctx, src, msg) {
 		case ValidationAccept:
 		case ValidationReject:
 			result = ValidationReject
-			break
+			break loop
 		case ValidationIgnore:
 			result = ValidationIgnore
 		}
@@ -377,12 +378,13 @@ func (v *validation) validateTopic(vals []*topicVal, src peer.ID, msg *Message)
 	}
 
 	result := ValidationAccept
+loop:
 	for i := 0; i < rcount; i++ {
 		switch <-rch {
 		case ValidationAccept:
 		case ValidationReject:
 			result = ValidationReject
-			return result
+			break loop
 		case ValidationIgnore:
 			// throttled validation has the same effect, but takes precedence over Ignore as it is not
 			// known whether the throttled validator would have signaled rejection.