From 638787d3dd1e438dd4b500994004f9b305aae215 Mon Sep 17 00:00:00 2001 From: Cameron Gutman Date: Mon, 20 Jan 2014 22:05:14 -0500 Subject: [PATCH] Fix misuse of sizeof on dynamic allocation --- limelight-common/VideoStream.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/limelight-common/VideoStream.c b/limelight-common/VideoStream.c index 991e39e..3d9c489 100644 --- a/limelight-common/VideoStream.c +++ b/limelight-common/VideoStream.c @@ -3,6 +3,8 @@ #include "PlatformThreads.h" #include "LinkedBlockingQueue.h" +#define FIRST_FRAME_MAX 1500 + PDECODER_RENDERER_CALLBACKS callbacks; PSTREAM_CONFIGURATION configuration; IP_ADDRESS remoteHost; @@ -140,14 +142,14 @@ int readFirstFrame(void) { return LastSocketError(); } - firstFrame = (char*)malloc(1500); + firstFrame = (char*) malloc(FIRST_FRAME_MAX); if (firstFrame == NULL) { return -1; } Limelog("Waiting for first frame\n"); for (;;) { - err = recv(firstFrameSocket, &firstFrame[offset], sizeof(firstFrame) - offset, 0); + err = recv(firstFrameSocket, &firstFrame[offset], FIRST_FRAME_MAX - offset, 0); if (err <= 0) { break; }