Re: [PATCH 3/3] Fix "NULL_RETURNS" caught by coverity

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

 



On Mon, Jul 14, 2014 at 01:44:45PM +0200, Fabiano Fidêncio wrote:
> On Mon, Jul 14, 2014 at 1:30 PM, Marc-André Lureau <mlureau@xxxxxxxxxx>
> wrote:
> > Could malloc & free do not have annotations like the glib ones.
> >
> > What happens for example if you replace malloc() by g_malloc(), and free()
> > by g_free() in the function below?
> >
> 
> I don't think that would help as one of the Coverity's complaints came
> from: "data = g_malloc(d->area.width * d->area.height * 3);"

Did it complain about an unchecked return value? It could be complaining
about a potential integer overflow when computing how much to alloc.

Christophe

Attachment: pgpNzKqNaDi6Y.pgp
Description: PGP signature

_______________________________________________
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]