From 650da22c33975dcde5c5880b791de1f795f8eca1 Mon Sep 17 00:00:00 2001 From: Nick Bolton Date: Wed, 10 Apr 2013 17:30:21 +0000 Subject: [PATCH] fixed buffer overrun bugs in unit tests. --- src/test/unittests/client/CServerProxyTests.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/unittests/client/CServerProxyTests.cpp b/src/test/unittests/client/CServerProxyTests.cpp index dbfad3e0..7fe6a219 100644 --- a/src/test/unittests/client/CServerProxyTests.cpp +++ b/src/test/unittests/client/CServerProxyTests.cpp @@ -55,7 +55,7 @@ TEST(CServerProxyTests, mouseMove) EXPECT_CALL(client, mouseMove(1, 2)).Times(1); const char data[] = "DSOP\0\0\0\0DMMV\0\1\0\2"; - memcpy(mouseMove_buffer, data, sizeof(data)); + memcpy(mouseMove_buffer, data, mouseMove_bufferLen); CServerProxy serverProxy(&client, &stream, &eventQueue); serverProxy.handleDataForTest(); @@ -73,7 +73,7 @@ TEST(CServerProxyTests, cryptoIv) ON_CALL(client, setCryptoIv(_)).WillByDefault(Invoke(cryptoIv_setCryptoIv)); const char data[] = "DSOP\0\0\0\0DCIV\0\0\0\4mock"; - memcpy(cryptoIv_buffer, data, sizeof(data)); + memcpy(cryptoIv_buffer, data, cryptoIv_bufferLen); CServerProxy serverProxy(&client, &stream, &eventQueue); serverProxy.handleDataForTest();