Server|Fixed: Catch exception when receiving invalid packet from shel…

…l user
skyjake · Feb 1, 2013 · 3601488 · 3601488
1 parent db12b81
commit 3601488
Showing 1 changed file with 13 additions and 6 deletions.
diff --git a/doomsday/server/src/shelluser.cpp b/doomsday/server/src/shelluser.cpp
@@ -86,14 +86,21 @@ void ShellUser::handleIncomingPackets()
         QScopedPointer<Packet> packet(nextPacket());
         if(packet.isNull()) break;
 
-        switch(protocol().recognize(packet.data()))
+        try
         {
-        case shell::Protocol::Command:
-            Con_Execute(CMDS_CONSOLE, protocol().command(*packet).toUtf8().constData(), false, true);
-            break;
+            switch(protocol().recognize(packet.data()))
+            {
+            case shell::Protocol::Command:
+                Con_Execute(CMDS_CONSOLE, protocol().command(*packet).toUtf8().constData(), false, true);
+                break;
 
-        default:
-            break;
+            default:
+                break;
+            }
+        }
+        catch(Error const &er)
+        {
+            LOG_WARNING("Error while processing packet from %s:\n%s") << packet->from() << er.asText();
         }
     }
 }