Crash if password entered as empty string

Bug #802788 reported by Andy Collins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
IRC.NET
Fix Released
Low
Alexander Regueiro

Bug Description

When calling the Connect() method in the IrcClient class, a parameter that implements IrcRegistrationInfo must be supplied. Using the IrcUserRegistrationInfo and supplying Password as an empty string will cause an unhandled ArgumentException in the CheckTrailingParameter method in the IrcClient class.

The exception is quite obscure and doesn't actually reference the password property of that particular class however, but once I changed it to a non empty string the issue dissappeared.

Tags: irc protocol

Related branches

Revision history for this message
Alexander Regueiro (alexreg) wrote :

This is due to the fact that the code handling the sending of raw IRC messages insures that no parameters are null-length. I've decided it's probably best to allow zero-length trailing parameters, thus this exception will no longer be thrown. (Specifying a zero-length password does however make the server complain by replying with an ERROR message.)

Changed in ircdotnet:
assignee: nobody → Alex Regueiro (noldorin)
importance: Undecided → Low
status: New → Confirmed
tags: added: irc protocol
Changed in ircdotnet:
status: Confirmed → Fix Committed
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.