mike/biboumi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Properly set the default values for invalid integer conf options

Browse Source 
fix #3372
This commit is contained in:
louiz’
2019-11-06 17:23:35 +01:00
parent 752cb86d8a
commit 34b4caeccf
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/xmpp/biboumi_adhoc_commands.cpp
+8 -2
View File
@@ -192,7 +192,13 @@ void ConfigureGlobalStep2(XmppComponent& xmpp_component, AdhocSession& session,
if (field->get_tag("var") == "max_history_length" &&
value && !value->get_inner().empty())
options.col<Database::MaxHistoryLength>() = atoi(value->get_inner().data());
{
try {
options.col<Database::MaxHistoryLength>() = std::stol(value->get_inner().data());
} catch (const std::logic_error&) {
options.col<Database::MaxHistoryLength>() = 20;
}
}
else if (field->get_tag("var") == "record_history" &&
value && !value->get_inner().empty())
{
@@ -497,7 +503,7 @@ void ConfigureIrcServerStep2(XmppComponent& xmpp_component, AdhocSession& sessio
try {
options.col<Database::ThrottleLimit>() = std::stol(value->get_inner());
} catch (const std::logic_error&) {
options.col<Database::ThrottleLimit>() = -1;
options.col<Database::ThrottleLimit>() = 10;
}
Bridge* bridge = biboumi_component.find_user_bridge(session.get_owner_jid());
if (bridge)