From d5720b799995b55a93d18597a422c44ccebee8f1 Mon Sep 17 00:00:00 2001 From: Armin Novak Date: Tue, 29 Jan 2019 17:06:08 +0100 Subject: [PATCH] Fixed clang scanbuild false positive memory leak. The memset did confuse the analyzer. Remove it as the variable is already initialized to zero. --- channels/rdpgfx/client/rdpgfx_codec.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/channels/rdpgfx/client/rdpgfx_codec.c b/channels/rdpgfx/client/rdpgfx_codec.c index d2dfbb3e9..cf96f44c9 100644 --- a/channels/rdpgfx/client/rdpgfx_codec.c +++ b/channels/rdpgfx/client/rdpgfx_codec.c @@ -243,10 +243,7 @@ static UINT rdpgfx_decode_AVC444(RDPGFX_PLUGIN* gfx, RDPGFX_SURFACE_COMMAND* cmd h264.bitstream[1].length = Stream_GetRemainingLength(s); } else - { h264.bitstream[0].length = Stream_GetRemainingLength(s); - memset(&h264.bitstream[1], 0, sizeof(h264.bitstream[1])); - } cmd->extra = (void*) &h264;