On 2014/11/20 17:07, Gonglei (Arei) wrote: > From: Gonglei <arei.gong...@huawei.com> > > Arguments in wrong order (SWAPPED_ARGUMENTS) > The positions of arguments in the call to > tight_fill_palette do not match the ordering of the parameters: > &fg is passed to bg > &bg is passed to fg > > Cc: Gerd Hoffmann <kra...@redhat.com> > Signed-off-by: Gonglei <arei.gong...@huawei.com> > --- > ui/vnc-enc-tight.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c > index 3d1b5cd..9a9ddf2 100644 > --- a/ui/vnc-enc-tight.c > +++ b/ui/vnc-enc-tight.c > @@ -1489,7 +1489,7 @@ static int send_sub_rect(VncState *vs, int x, int y, > int w, int h) > } > #endif > > - colors = tight_fill_palette(vs, x, y, w * h, &fg, &bg, &palette); > + colors = tight_fill_palette(vs, x, y, w * h, &bg, &fg, &palette); > > #ifdef CONFIG_VNC_JPEG > if (allow_jpeg && vs->tight.quality != (uint8_t)-1) {
Ping... Regards, -Gonglei