Re: [spice-gtk 1/4] coverity: fix unitialized use of mem.data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 4, 2016 at 11:08 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> Where is the mem.data use you mention? In the memcpy?

Exactly. mem.data use mentioned is in the memcpy.

>
> Christophe
>
> On Mon, Apr 04, 2016 at 09:59:50AM +0200, Fabiano Fidêncio wrote:
>> ---
>>  src/channel-main.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/channel-main.c b/src/channel-main.c
>> index 4a1f58a..93537d2 100644
>> --- a/src/channel-main.c
>> +++ b/src/channel-main.c
>> @@ -1021,9 +1021,9 @@ static void agent_msg_queue_many(SpiceMainChannel *channel, int type, const void
>>      }
>>      va_end(args);
>>
>> +    memset(&msg, 0, sizeof(VDAgentMessage));
>>      msg.protocol = VD_AGENT_PROTOCOL;
>>      msg.type = type;
>> -    msg.opaque = 0;
>>      msg.size = size;
>>
>>      paysize = MIN(VD_AGENT_MAX_DATA_SIZE, size + sizeof(VDAgentMessage));
>> --
>> 2.7.3
>>
>> _______________________________________________
>> Spice-devel mailing list
>> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]