diff options
| author | Andrei Karas <akaras@inbox.ru> | 2016-02-08 18:31:37 +0300 |
|---|---|---|
| committer | Andrei Karas <akaras@inbox.ru> | 2016-02-08 18:31:37 +0300 |
| commit | 519d998fdec7feff3ec14876ab4c077759e374e4 (patch) | |
| tree | 257065b6012f1da3db8d15a7adf4c7864a1133bd | |
| parent | 6fa6509bff3e0f761d8dc8979a17b190c482ae72 (diff) | |
| download | manaplus-519d998fdec7feff3ec14876ab4c077759e374e4.tar.gz manaplus-519d998fdec7feff3ec14876ab4c077759e374e4.tar.bz2 manaplus-519d998fdec7feff3ec14876ab4c077759e374e4.tar.xz manaplus-519d998fdec7feff3ec14876ab4c077759e374e4.zip | |
Fix possible variables overflow.
| -rw-r--r-- | src/gui/windows/whoisonline.cpp | 2 | ||||
| -rw-r--r-- | src/utils/mkdir.cpp | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/gui/windows/whoisonline.cpp b/src/gui/windows/whoisonline.cpp index e5a803a29..3c68adac1 100644 --- a/src/gui/windows/whoisonline.cpp +++ b/src/gui/windows/whoisonline.cpp @@ -373,7 +373,7 @@ void WhoIsOnline::loadWebList() mBrowserBox->clearRows(); bool listStarted(false); std::string lineStr; - int numOnline(0); + size_t numOnline(0U); // Tokenize and add each line separately char *line = strtok(mMemoryBuffer, "\n"); diff --git a/src/utils/mkdir.cpp b/src/utils/mkdir.cpp index f7a2aba37..25a8ba521 100644 --- a/src/utils/mkdir.cpp +++ b/src/utils/mkdir.cpp @@ -54,7 +54,7 @@ int mkdir_r(const char *const pathname) const int len = CAST_S32(strlen(tmp)); - if (len < 1) + if (len < 1 || len >= INT_MAX) return -1; // terminate the pathname with '/' |