View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001294 | Anope Development (1.9.x series) | Nickserv | public | 2011-07-29 17:45 | 2011-07-29 22:52 |
Reporter | Nita | Assigned To | LEthaLity | ||
Priority | urgent | Severity | crash | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | Linux x64 | OS | Debian | ||
Fixed in Version | 1.9.x-GIT | ||||
Summary | 0001294: Anope-1.9-f5e78d7 current devel crash: /msg NickServ SET HIDE EMAIL ON | ||||
Description | Anope-1.9-f5e78d7 current development crash: /msg NickServ SET HIDE EMAIL ON Crash (Tested with 3 different users) | ||||
Steps To Reproduce | /msg NickServ SET HIDE EMAIL ON | ||||
Additional Information | Inspircd 2.0.5 | ||||
Tags | No tags attached. | ||||
|
Fixed in f29c88bcd535e6dcf |
|
ns_set_hide.diff (490 bytes)
diff --git a/modules/core/ns_set_hide.cpp b/modules/core/ns_set_hide.cpp index e1a9ecb..de3a96a 100644 --- a/modules/core/ns_set_hide.cpp +++ b/modules/core/ns_set_hide.cpp @@ -25,7 +25,7 @@ class CommandNSSetHide : public Command void Run(CommandSource &source, const Anope::string &user, const Anope::string ¶m, const Anope::string &arg) { NickAlias *na = findnick(user); - if (user == NULL) + if (!na) { source.Reply(NICK_X_NOT_REGISTERED, user.c_str()); return; |
|
Confirmed, though I don't think it's limited to just HIDE EMAIL. http://pastebin.com/BYxSpra8 Attached patch. |
Date Modified | Username | Field | Change |
---|---|---|---|
2011-07-29 17:45 | Nita | New Issue | |
2011-07-29 19:00 | LEthaLity | Note Added: 0005882 | |
2011-07-29 19:00 | LEthaLity | Status | new => confirmed |
2011-07-29 19:37 | LEthaLity | File Added: ns_set_hide.diff | |
2011-07-29 19:41 | LEthaLity | Note Edited: 0005882 | View Revisions |
2011-07-29 22:52 | LEthaLity | Note Added: 0005883 | |
2011-07-29 22:52 | LEthaLity | Status | confirmed => resolved |
2011-07-29 22:52 | LEthaLity | Fixed in Version | => 1.9.x-GIT |
2011-07-29 22:52 | LEthaLity | Resolution | open => fixed |
2011-07-29 22:52 | LEthaLity | Assigned To | => LEthaLity |