[Spice-devel] [spice v2 1/3] server: Fix a pointer to uint64_t cast in spice_replay_next_cmd()

Frediano Ziglio fziglio at redhat.com
Tue Jan 12 01:11:20 PST 2016


> 
> This avoids a compilation error with -Werror on 32 bit systems as the
> pointer size differs from that of an uint64_t.
> 
> Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
> ---
>  server/red-replay-qxl.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/server/red-replay-qxl.c b/server/red-replay-qxl.c
> index 66acf1e..7a146b6 100644
> --- a/server/red-replay-qxl.c
> +++ b/server/red-replay-qxl.c
> @@ -1156,7 +1156,7 @@ SPICE_GNUC_VISIBLE QXLCommandExt*
> spice_replay_next_cmd(SpiceReplay *replay,
>      case QXL_CMD_UPDATE:
>      case QXL_CMD_SURFACE:
>          info = (QXLReleaseInfo *)cmd->cmd.data;
> -        info->id = (uint64_t)cmd;
> +        info->id = (uint64_t)(uintptr_t)cmd;
>      }
>  
>      replay->counter++;
> --
> 2.6.4
> 

I was suggesting just

  info->id = (uintptr_t)cmd;

Frediano


More information about the Spice-devel mailing list