[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 4/8] spice: reset cursor on resize
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PULL 4/8] spice: reset cursor on resize |
Date: |
Fri, 26 Feb 2016 00:23:53 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 |
On 26/02/2016 00:23, Marc-André Lureau wrote:
> Hi
>
> On Fri, Feb 26, 2016 at 12:10 AM, Paolo Bonzini <address@hidden> wrote:
>>
>>
>> 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.
>>
>
> Oops, I don't know why there is a if there, it's unnecessary indeed.
> ACK if you did the patch already ;)
No I didn't. :)
Paolo
- [Qemu-devel] [PULL 0/8] spice: initial opengl/virgl support, postcopy migration fix., Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 8/8] Postcopy+spice: Pass spice migration data earlier, Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 6/8] spice/gl: add unblock timer, Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 7/8] spice/gl: tweak debug messages., Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 1/8] spice: init dcl before registering qxl interface, Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 2/8] configure: add dma-buf support detection., Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 4/8] spice: reset cursor on resize, Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 5/8] spice: add opengl/virgl/dmabuf support, Gerd Hoffmann, 2016/02/23
- [Qemu-devel] [PULL 3/8] egl-helpers: add functions for render nodes and dma-buf passing, Gerd Hoffmann, 2016/02/23
- Re: [Qemu-devel] [PULL 0/8] spice: initial opengl/virgl support, postcopy migration fix., Peter Maydell, 2016/02/23