mirror/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2026-01-11 16:32:14 -06:00
Code Issues Packages Projects Releases Wiki Activity

jsoncpp: Add missing assert before strcmp in json_value.cpp

Browse Source 
The strcmp function does not allow NULL pointers, so add an
assert to tell Clang scan-build that the code does not expect
a NULL pointer.
This commit is contained in:
Brad King
2015-01-13 15:08:32 -05:00
parent ad94b0521e
commit 50032bc847
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
Utilities/cmjsoncpp/src/lib_json/json_value.cpp
View File

@@ -195,14 +195,18 @@ Value::CZString& Value::CZString::operator=(CZString other) {
}
bool Value::CZString::operator<(const CZString& other) const {
if (cstr_)
if (cstr_) {
assert(other.cstr_);
return strcmp(cstr_, other.cstr_) < 0;
}
return index_ < other.index_;
}
bool Value::CZString::operator==(const CZString& other) const {
if (cstr_)
if (cstr_) {
assert(other.cstr_);
return strcmp(cstr_, other.cstr_) == 0;
}
return index_ == other.index_;
}