Re: [Spice-devel] [spice-common 4/8] coverity: avoid dereference after null check

2016-04-04 Thread Christophe Fergeau
On Mon, Apr 04, 2016 at 10:03:35AM +0200, Fabiano FidĂȘncio wrote: > All decompress functions used after this check take into account that > encoder->palette is not NULL. So, if we already detected that the > palette is NULL, let's just return early. > --- > common/lz.c | 1 + > 1 file changed, 1 i

Re: [Spice-devel] [spice-common 4/8] coverity: avoid dereference after null check

2016-04-04 Thread Victor Toso
Hi, On Mon, Apr 04, 2016 at 10:03:35AM +0200, Fabiano FidĂȘncio wrote: > All decompress functions used after this check take into account that > encoder->palette is not NULL. So, if we already detected that the > palette is NULL, let's just return early. Acked-by: Victor Toso > --- > common/lz.

[Spice-devel] [spice-common 4/8] coverity: avoid dereference after null check

2016-04-04 Thread Fabiano FidĂȘncio
All decompress functions used after this check take into account that encoder->palette is not NULL. So, if we already detected that the palette is NULL, let's just return early. --- common/lz.c | 1 + 1 file changed, 1 insertion(+) diff --git a/common/lz.c b/common/lz.c index d1c4033..2589223 100