OpenPunk/OpenFusion
1
0
Fork 0
mirror of https://github.com/OpenFusionProject/OpenFusion.git synced 2024-11-22 13:30:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

Do not truncate emails to 127 characters

Browse Source 
And assert that we never supply a string long enough for
sanitizeText() to truncate.
This commit is contained in:
dongresource 2020-12-19 05:46:38 +01:00
parent 2f44243abb
commit f8a359dfe9
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/ChatManager.cpp
View File

@ -873,11 +873,14 @@ void ChatManager::announcementHandler(CNSocket* sock, CNPacketData* data) {
// we only allow plain ascii, at least for now // we only allow plain ascii, at least for now
std::string ChatManager::sanitizeText(std::string text, bool allowNewlines) { std::string ChatManager::sanitizeText(std::string text, bool allowNewlines) {
int i; int i;
char buf[128]; const int BUFSIZE = 512;
char buf[BUFSIZE];
assert(text.size() < BUFSIZE);
i = 0; i = 0;
for (char c : text) { for (char c : text) {
if (i >= 127) if (i >= BUFSIZE-1)
break; break;
if (!allowNewlines && c == '\n') if (!allowNewlines && c == '\n')