Re: [PATCH spice-streaming-agent 2/2] Do not redefine "msg" field

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2018-03-09 at 07:35 +0000, Frediano Ziglio wrote:
> msg.msg was redefining msg.StreamMsgNotifyError::msg.
> This cause some confusion.
> 
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  src/spice-streaming-agent.cpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/spice-streaming-agent.cpp b/src/spice-streaming-agent.cpp
> index 37addf4..777e330 100644
> --- a/src/spice-streaming-agent.cpp
> +++ b/src/spice-streaming-agent.cpp
> @@ -146,13 +146,13 @@ static void handle_stream_error(size_t len)
>      }
>  
>      struct NotifyError : StreamMsgNotifyError {
> -        uint8_t msg[1024];
> +        uint8_t msg_buffer[1024];

Again, something like msg_1k could be better?

>      } msg;
>  
>      size_t len_to_read = std::min(len, sizeof(msg) - 1);
>  
>      read_all(&msg, len_to_read);
> -    msg.msg[len_to_read - sizeof(StreamMsgNotifyError)] = '\0';
> +    ((uint8_t *) &msg)[len_to_read] = '\0';
>  
>      syslog(LOG_ERR, "Received NotifyError message from the server: %d - %s\n",
>          msg.error_code, msg.msg);
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]