From 02859be56d43bcece02aab350e02bc95ed1bf446 Mon Sep 17 00:00:00 2001 From: attilamolnar Date: Sun, 30 Sep 2012 01:10:57 +0200 Subject: Fix more undefined behavior caused by referencing the returned buffer by std::string::c_str() when the object is temporary See 83c7cc45daf6fb1f8c36f15297a4657e45a34e88 --- src/modules/m_opermotd.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/modules/m_opermotd.cpp') diff --git a/src/modules/m_opermotd.cpp b/src/modules/m_opermotd.cpp index 5c107b6b1..fcd248b90 100644 --- a/src/modules/m_opermotd.cpp +++ b/src/modules/m_opermotd.cpp @@ -62,7 +62,10 @@ class CommandOpermotd : public Command user->SendText(":%s 375 %s :- IRC Operators Message of the Day", servername.c_str(), user->nick.c_str()); for (int i=0; i != opermotd.FileSize(); i++) - user->SendText(":%s 372 %s :- %s", servername.c_str(), user->nick.c_str(), opermotd.GetLine(i).c_str()); + { + std::string line = opermotd.GetLine(i); + user->SendText(":%s 372 %s :- %s", servername.c_str(), user->nick.c_str(), line.c_str()); + } user->SendText(":%s 376 %s :- End of OPERMOTD", servername.c_str(), user->nick.c_str()); } -- cgit v1.3.1-10-gc9f91