[Spice-devel] [PATCH spice-server 4/9] main-channel: Remove useless check

Frediano Ziglio fziglio at redhat.com
Tue Dec 5 08:41:07 UTC 2017


If there is a channel client there's surely a related channel.

Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
---
 server/main-channel.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/server/main-channel.c b/server/main-channel.c
index 43ab39ae..f5c079f3 100644
--- a/server/main-channel.c
+++ b/server/main-channel.c
@@ -165,7 +165,6 @@ static bool main_channel_handle_message(RedChannelClient *rcc, uint16_t type,
                                         uint32_t size, void *message)
 {
     RedChannel *channel = red_channel_client_get_channel(rcc);
-    MainChannel *main_chan = MAIN_CHANNEL(channel);
     MainChannelClient *mcc = MAIN_CHANNEL_CLIENT(rcc);
     RedsState *reds = red_channel_get_server(channel);
 
@@ -174,9 +173,6 @@ static bool main_channel_handle_message(RedChannelClient *rcc, uint16_t type,
         SpiceMsgcMainAgentStart *tokens;
 
         spice_printerr("agent start");
-        if (!main_chan) {
-            return FALSE;
-        }
         tokens = (SpiceMsgcMainAgentStart *)message;
         reds_on_main_agent_start(reds, mcc, tokens->num_tokens);
         break;
-- 
2.14.3



More information about the Spice-devel mailing list