diff options
| author | linuxdaemon <linuxdaemon@users.noreply.github.com> | 2018-12-21 12:37:22 -0600 |
|---|---|---|
| committer | Peter Powell <petpow@saberuk.com> | 2018-12-21 19:37:22 +0100 |
| commit | 7530285740c2db1bc4457859b0d5f2436e0ed113 (patch) | |
| tree | 89635613725a1b7ad6e262d7e3035a518d42e8d2 /src/modules/m_timedbans.cpp | |
| parent | 60ae49b62a371791ebbd9e4b131e33b857873f06 (diff) | |
Only parse valid durations, don't treat invalid multipliers as seconds (#1538)
Diffstat (limited to 'src/modules/m_timedbans.cpp')
| -rw-r--r-- | src/modules/m_timedbans.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/modules/m_timedbans.cpp b/src/modules/m_timedbans.cpp index 058028f61..32b376c79 100644 --- a/src/modules/m_timedbans.cpp +++ b/src/modules/m_timedbans.cpp @@ -84,13 +84,13 @@ class CommandTban : public Command } TimedBan T; - unsigned long duration = InspIRCd::Duration(parameters[1]); - unsigned long expire = duration + ServerInstance->Time(); - if (duration < 1) + unsigned long duration; + if (!InspIRCd::Duration(parameters[1], duration)) { user->WriteNotice("Invalid ban time"); return CMD_FAILURE; } + unsigned long expire = duration + ServerInstance->Time(); std::string mask = parameters[2]; bool isextban = ((mask.size() > 2) && (mask[1] == ':')); if (!isextban && !InspIRCd::IsValidMask(mask)) |