Skip to content

Commit

Permalink
Fix the stupid untested off-by-two ident length check
Browse files Browse the repository at this point in the history
  • Loading branch information
Daniel De Graaf committed Aug 13, 2010
1 parent 340d0c0 commit e0ffef8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/users.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1724,7 +1724,7 @@ bool User::ChangeIdent(const char* newident)
if (this->ServerInstance->Config->CycleHosts && !MOD_RESULT)
this->WriteCommonExcept("%s","QUIT :Changing ident");

this->ident.assign(newident, 0, ServerInstance->Config->Limits.IdentMax + 1);
this->ident.assign(newident, 0, ServerInstance->Config->Limits.IdentMax - 1);

this->InvalidateCache();

Expand Down

0 comments on commit e0ffef8

Please sign in to comment.