>
> Using spicevmc_red_channel_client_get_state(rcc) (which already
> exists) instead of SPICE_CONTAINEROF(rcc->channel, SpiceVmcState,
> channel).
> ---
> server/spicevmc.c | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/server/spicevmc.c b/server/spicevmc.c
> index dffdd9a..54dfe4a 100644
> --- a/server/spicevmc.c
> +++ b/server/spicevmc.c
> @@ -152,9 +152,14 @@ static void
> spicevmc_chardev_send_msg_to_client(RedPipeItem *msg,
> red_channel_client_pipe_add_push(state->rcc, (RedPipeItem *)vmc_msg);
> }
>
> +static SpiceVmcState *spicevmc_red_channel_client_get_state(RedChannelClient
> *rcc)
> +{
> + return SPICE_CONTAINEROF(rcc->channel, SpiceVmcState, channel);
> +}
> +
> static void spicevmc_port_send_init(RedChannelClient *rcc)
> {
> - SpiceVmcState *state = SPICE_CONTAINEROF(rcc->channel, SpiceVmcState,
> channel);
> + SpiceVmcState *state = spicevmc_red_channel_client_get_state(rcc);
> SpiceCharDeviceInstance *sin = state->chardev_sin;
> RedPortInitPipeItem *item = spice_malloc(sizeof(RedPortInitPipeItem));
>
> @@ -217,7 +222,7 @@ static void
> spicevmc_red_channel_client_on_disconnect(RedChannelClient *rcc)
> return;
> }
>
> - state = SPICE_CONTAINEROF(rcc->channel, SpiceVmcState, channel);
> + state = spicevmc_red_channel_client_get_state(rcc);
>
> if (state->recv_from_client_buf) { /* partial message which wasn't
> pushed to device */
> red_char_device_write_buffer_release(state->chardev,
> state->recv_from_client_buf);
> @@ -245,11 +250,6 @@ static void
> spicevmc_red_channel_client_on_disconnect(RedChannelClient *rcc)
> }
> }
>
> -static SpiceVmcState *spicevmc_red_channel_client_get_state(RedChannelClient
> *rcc)
> -{
> - return SPICE_CONTAINEROF(rcc->channel, SpiceVmcState, channel);
> -}
> -
> static int
> spicevmc_channel_client_handle_migrate_flush_mark(RedChannelClient *rcc)
> {
> red_channel_client_pipe_add_type(rcc,
> RED_PIPE_ITEM_TYPE_SPICEVMC_MIGRATE_DATA);
> @@ -317,7 +317,7 @@ static uint8_t
> *spicevmc_red_channel_alloc_msg_rcv_buf(RedChannelClient *rcc,
> {
> SpiceVmcState *state;
>
> - state = SPICE_CONTAINEROF(rcc->channel, SpiceVmcState, channel);
> + state = spicevmc_red_channel_client_get_state(rcc);
>
> switch (type) {
> case SPICE_MSGC_SPICEVMC_DATA:
> @@ -345,7 +345,7 @@ static void
> spicevmc_red_channel_release_msg_rcv_buf(RedChannelClient *rcc,
> {
> SpiceVmcState *state;
>
> - state = SPICE_CONTAINEROF(rcc->channel, SpiceVmcState, channel);
> + state = spicevmc_red_channel_client_get_state(rcc);
>
> switch (type) {
> case SPICE_MSGC_SPICEVMC_DATA:
> @@ -381,7 +381,7 @@ static void
> spicevmc_red_channel_send_migrate_data(RedChannelClient *rcc,
> {
> SpiceVmcState *state;
>
> - state = SPICE_CONTAINEROF(rcc->channel, SpiceVmcState, channel);
> + state = spicevmc_red_channel_client_get_state(rcc);
> red_channel_client_init_send_data(rcc, SPICE_MSG_MIGRATE_DATA, item);
> spice_marshaller_add_uint32(m, SPICE_MIGRATE_DATA_SPICEVMC_MAGIC);
> spice_marshaller_add_uint32(m, SPICE_MIGRATE_DATA_SPICEVMC_VERSION);
Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel