On 23/02/2016 15:24, Gerd Hoffmann wrote:
> qemu_mutex_lock(&ssd->lock);
> + if (c) {
> + cursor_get(c);
> + }
> + cursor_put(ssd->cursor);
> + ssd->cursor = c;
> ssd->hot_x = c->hot_x;
> ssd->hot_y = c->hot_y;Coverity complains that this would dereference a NULL c, and I think it's right; either an unlock+return is missing, or the "if" is unnecessary. Paolo
