From de7b2f984a924f6b8a8b7be46972a03dd30decbc Mon Sep 17 00:00:00 2001 From: mimimi085181 Date: Fri, 8 Jul 2016 23:15:38 +0200 Subject: [PATCH] Make tries a u32 Before this variable was an u8, which could theoretically result in desyncs with a large buffer(greater than 255*120/200=153) filled with blank inputs. If this could actually happen, i don't know. But this part of the code on its own looks like it could break. --- Source/Core/Core/NetPlayClient.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/Core/Core/NetPlayClient.cpp b/Source/Core/Core/NetPlayClient.cpp index 5d8aee550a..e0639e9c1e 100644 --- a/Source/Core/Core/NetPlayClient.cpp +++ b/Source/Core/Core/NetPlayClient.cpp @@ -989,7 +989,7 @@ bool NetPlayClient::WiimoteUpdate(int _number, u8* data, const u8 size) // until we reach a good input if (nw.size() != size) { - u8 tries = 0; + u32 tries = 0; // Clear the buffer and wait for new input, since we probably just changed reporting mode. while (nw.size() != size) {