Skip to content

Commit

Permalink
vnc: drop unused depth arg for set_pixel_format
Browse files Browse the repository at this point in the history
Spotted by Coverity.

Cc: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Message-id: 1465204725-31562-1-git-send-email-kraxel@redhat.com
  • Loading branch information
kraxel committed Jun 10, 2016
1 parent 0c33682 commit ec9fb41
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions ui/vnc.c
Expand Up @@ -2115,16 +2115,14 @@ static void send_color_map(VncState *vs)
}
}

static void set_pixel_format(VncState *vs,
int bits_per_pixel, int depth,
static void set_pixel_format(VncState *vs, int bits_per_pixel,
int big_endian_flag, int true_color_flag,
int red_max, int green_max, int blue_max,
int red_shift, int green_shift, int blue_shift)
{
if (!true_color_flag) {
/* Expose a reasonable default 256 color map */
bits_per_pixel = 8;
depth = 8;
red_max = 7;
green_max = 7;
blue_max = 3;
Expand Down Expand Up @@ -2231,7 +2229,7 @@ static int protocol_client_msg(VncState *vs, uint8_t *data, size_t len)
if (len == 1)
return 20;

set_pixel_format(vs, read_u8(data, 4), read_u8(data, 5),
set_pixel_format(vs, read_u8(data, 4),
read_u8(data, 6), read_u8(data, 7),
read_u16(data, 8), read_u16(data, 10),
read_u16(data, 12), read_u8(data, 14),
Expand Down

0 comments on commit ec9fb41

Please sign in to comment.