Re: [PATCH] replay: use spice_malloc0 instead of raw malloc

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

 



On Fri, Jan 15, 2016 at 11:02 AM, Victor Toso <lists@xxxxxxxxxxxxxx> wrote:
> Hi,
>
> On Fri, Jan 15, 2016 at 09:51:58AM +0000, Frediano Ziglio wrote:
>> This function report memory allocation errors and reset memory
>> after allocation.
>>
>> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
>> ---
>>  server/red-replay-qxl.c | 2 +-
>
> It seems that there are other places that could use spice_malloc in
> red-replay-qxl, no?

And why not g_malloc0() instead of spice_malloc0()?

>
> Patch looks good
> Reviewed-by: Victor Toso <victortoso@xxxxxxxxxx>
>
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/server/red-replay-qxl.c b/server/red-replay-qxl.c
>> index 8e0b28c..fb7b9b7 100644
>> --- a/server/red-replay-qxl.c
>> +++ b/server/red-replay-qxl.c
>> @@ -1215,7 +1215,7 @@ SpiceReplay *spice_replay_new(FILE *file, int nsurfaces)
>>          }
>>      }
>>
>> -    replay = malloc(sizeof(SpiceReplay));
>> +    replay = spice_malloc0(sizeof(SpiceReplay));
>>
>>      replay->eof = 0;
>>      replay->fd = file;
>> --
>> 2.4.3
>>
>> _______________________________________________
>> Spice-devel mailing list
>> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
>> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]