Bug #573

avatar

Use of number in network name make the server crash

Added by Charles Vinchon 4793 days ago. Updated 4502 days ago.

Status:Closed Start:03/05/2011
Priority:High Due date:
Assigned to:avatarMirco Bauer % Done:

100%
Category:Engine
Target version:0.8.9
Complexity:

Low

 Found in Version:
Votes: 0

Description

Use of number in network name make the server crash : I added a server irc.2600.net with network name 2600. Then all change with graphical GNOME frontend in config make the clint crash (GNOME client with remote SSH server with IRC engine)

Associated revisions

Revision 65932b4708423eb4fc18ae5b6d34963c650c5c08
Added by Mirco Bauer 4516 days ago

[Engine] Parse Network always as string (closes: #573)

History

Updated by Mirco Bauer 4792 days ago

avatar
  • Category set to Engine
  • Target version changed from 0.8 to 0.8.9
  • Complexity set to Low

Updated by Robert Sheets 4568 days ago

avatar
Steps to reproduce:
  1. Add a new IRC server, setting the network name to a string composed only of numbers
  2. Quit Smuxi Engine
    • If using a local engine, simply quit the program
    • If using a remote engine, kill the server
  3. Start Smuxi Engine. When the engine (either local or remote) starts up and reads the config file, it crashes.

Example exception text:

Exception Type:
System.InvalidCastException

Exception Message:
Unable to cast object of type 'System.Int32' to type 'System.String'.

Exception StackTrace:
   at Smuxi.Engine.ServerListController.GetServer(String protocol, String servername)
   at Smuxi.Engine.ServerListController.GetServerList()
   at Smuxi.Engine.Session.RegisterFrontendUI(IFrontendUI ui)
   at Smuxi.Frontend.Gnome.Frontend.ConnectEngineToGUI()
   at Smuxi.Frontend.Gnome.EngineManagerDialog._OnConnectButtonPressed()
   at Smuxi.Frontend.Gnome.EngineManagerDialog._OnResponse(Object sender, ResponseArgs e)

Updated by Mirco Bauer 4516 days ago

avatar
  • Assigned to set to Mirco Bauer

Updated by Mirco Bauer 4516 days ago

avatar
  • Status changed from New to Closed
  • % Done changed from 0 to 100

Updated by Mirco Bauer 4502 days ago

avatar

This change can be tested with a development build which can be obtained from here: http://www.smuxi.org/documentation/running-from-git/

Also available in: Atom PDF