CEF/ergo
1
0
Fork 0
forked from External/ergo
Code Pull requests Activity

fix #1898

Browse source 
NS SAREGISTER would fail due to a nil dereference of `client`;
add two safeguards against this.
This commit is contained in:
Shivaram Lingamneni 2022-01-10 01:58:05 -05:00
parent 8605cd2295
commit 3162c8a1c8
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
irc/accounts.go
View file

@ -474,8 +474,10 @@ func (am *AccountManager) Register(client *Client, account string, callbackNames
am.Unregister(casefoldedAccount, true)
return &registrationCallbackError{underlying: err}
} else {
if client != nil && code != "" {
am.server.logger.Info("accounts",
fmt.Sprintf("nickname %s registered account %s, pending verification", client.Nick(), account))
}
return am.server.store.Update(func(tx *buntdb.Tx) error {
_, _, err = tx.Set(verificationCodeKey, code, setOptions)
return err