diff options
| author | Andrei Karas <akaras@inbox.ru> | 2013-03-02 17:00:07 +0300 |
|---|---|---|
| committer | Andrei Karas <akaras@inbox.ru> | 2013-03-02 21:16:56 +0300 |
| commit | fb1709d774139b2173da6ee95770abaad23f726e (patch) | |
| tree | a6e6530dd623de4f6719a190eb6d9f19da20f027 /src/net | |
| parent | f06505b08060ad0720653fdf21fcc306ca93eb13 (diff) | |
| download | manaplus-fb1709d774139b2173da6ee95770abaad23f726e.tar.gz manaplus-fb1709d774139b2173da6ee95770abaad23f726e.tar.bz2 manaplus-fb1709d774139b2173da6ee95770abaad23f726e.tar.xz manaplus-fb1709d774139b2173da6ee95770abaad23f726e.zip | |
Some fixes after coverity checks.
Diffstat (limited to 'src/net')
| -rw-r--r-- | src/net/download.cpp | 9 | ||||
| -rw-r--r-- | src/net/ea/guildhandler.cpp | 5 |
2 files changed, 10 insertions, 4 deletions
diff --git a/src/net/download.cpp b/src/net/download.cpp index 838038f7c..32625b181 100644 --- a/src/net/download.cpp +++ b/src/net/download.cpp @@ -68,6 +68,8 @@ Download::Download(void *ptr, const std::string &url, mError[0] = 0; mOptions.cancel = 0; + mOptions.memoryWrite = 0; + mOptions.checkAdler = true; } Download::~Download() @@ -88,9 +90,16 @@ Download::~Download() */ unsigned long Download::fadler32(FILE *file) { + if (!file) + return 0; + // Obtain file size fseek(file, 0, SEEK_END); long fileSize = ftell(file); + if (fileSize < 0) + { // file size error + return 0; + } rewind(file); // Calculate Adler-32 checksum diff --git a/src/net/ea/guildhandler.cpp b/src/net/ea/guildhandler.cpp index e9ec8c97d..129ba931d 100644 --- a/src/net/ea/guildhandler.cpp +++ b/src/net/ea/guildhandler.cpp @@ -552,11 +552,8 @@ void GuildHandler::processGuildMessage(Net::MessageIn &msg) const size_t pos = chatMsg.find(" : ", 0); if (pos != std::string::npos) { - std::string sender_name = ((pos == std::string::npos) - ? "" : chatMsg.substr(0, pos)); - + std::string sender_name = chatMsg.substr(0, pos); chatMsg.erase(0, pos + 3); - trim(chatMsg); guildTab->chatLog(sender_name, chatMsg); } |