[Spice-devel] [spice-gtk] main: assign variable after check for null

Pavel Grunt pgrunt at redhat.com
Tue Jun 14 12:30:23 UTC 2016


On Tue, 2016-06-14 at 14:28 +0200, Victor Toso wrote:
Acked-by: Pavel Grunt <pgrunt at redhat.com>
> ---
>  src/channel-main.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/channel-main.c b/src/channel-main.c
> index 89675d5..e7171c1 100644
> --- a/src/channel-main.c
> +++ b/src/channel-main.c
> @@ -3181,12 +3181,13 @@ void spice_main_file_copy_async(SpiceMainChannel
> *channel,
>                                  GAsyncReadyCallback callback,
>                                  gpointer user_data)
>  {
> -    SpiceMainChannelPrivate *c = channel->priv;
> +    SpiceMainChannelPrivate *c;
>  
>      g_return_if_fail(channel != NULL);
>      g_return_if_fail(SPICE_IS_MAIN_CHANNEL(channel));
>      g_return_if_fail(sources != NULL);
>  
> +    c = channel->priv;
>      if (!c->agent_connected) {
>          g_task_report_new_error(channel,
>                                  callback,


More information about the Spice-devel mailing list