[Spice-devel] [PATCH v2] Use spice_malloc instead of malloc

Christophe Fergeau cfergeau at redhat.com
Tue Jun 16 00:55:14 PDT 2015


On Thu, Jun 11, 2015 at 04:52:09PM +0200, Christophe Fergeau wrote:
> On Wed, Jun 10, 2015 at 02:53:05PM +0100, Frediano Ziglio wrote:
> > Do not just check and give warning before crashing the program
> > accessing a NULL pointer but use spice_malloc which exits with a
> > proper message.
> > 
> > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > ---
> >  server/red_worker.c | 6 ++----
> >  1 file changed, 2 insertions(+), 4 deletions(-)
> > 
> > Change from v1:
> > - fix type in patch comment
> > 
> > diff --git a/server/red_worker.c b/server/red_worker.c
> > index 1871e13..03d07e2 100644
> > --- a/server/red_worker.c
> > +++ b/server/red_worker.c
> > @@ -1664,8 +1664,7 @@ static SurfaceDestroyItem *get_surface_destroy_item(RedChannel *channel,
> >  {
> >      SurfaceDestroyItem *destroy;
> >  
> > -    destroy = (SurfaceDestroyItem *)malloc(sizeof(SurfaceDestroyItem));
> > -    spice_warn_if(!destroy);
> > +    destroy = (SurfaceDestroyItem *)spice_malloc(sizeof(SurfaceDestroyItem));
> >  
> >      destroy->surface_destroy.surface_id = surface_id;
> >  
> > @@ -9399,8 +9398,7 @@ static SurfaceCreateItem *get_surface_create_item(
> >  {
> >      SurfaceCreateItem *create;
> >  
> > -    create = (SurfaceCreateItem *)malloc(sizeof(SurfaceCreateItem));
> > -    spice_warn_if(!create);
> > +    create = (SurfaceCreateItem *)spice_malloc(sizeof(SurfaceCreateItem));
> 
> Actually the cast is unnecessary, I'll drop it too (I've already made the
> change locally, no need to resend a v3 for that).

I've now pushed this.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20150616/84293d7f/attachment.sig>


More information about the Spice-devel mailing list