From c73113a12e2ed6cf8ea5be94b4337eb5aec881eb Mon Sep 17 00:00:00 2001
From: stasatdaglabs <39559713+stasatdaglabs@users.noreply.github.com>
Date: Tue, 27 Aug 2019 13:09:36 +0300
Subject: [PATCH] [NOD-295] Limit the length of GetData to 50 (#383)

* [NOD-295] Fixed bad break condition in addInvsToGetDataMessageFromQueue.

* [NOD-295] Fixed the fix for bad break condition in addInvsToGetDataMessageFromQueue.

* [NOD-295] Made the check for max invs refer to invsNum instead of MaxInvPerGetDataMsg.
---
 netsync/manager.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/netsync/manager.go b/netsync/manager.go
index fd6a3ad36..806bc51fa 100644
--- a/netsync/manager.go
+++ b/netsync/manager.go
@@ -1042,7 +1042,7 @@ func (sm *SyncManager) addInvsToGetDataMessageFromQueue(gdmsg *wire.MsgGetData,
 	}
 	invsToAdd := make([]*wire.InvVect, 0, invsNum)
 
-	for len(requestQueue) != 0 {
+	for len(requestQueue) != 0 && len(invsToAdd) < invsNum {
 		iv := requestQueue[0]
 		requestQueue[0] = nil
 		requestQueue = requestQueue[1:]