Merge pull request #297 from Dmitry-Me/betterAssertsInParsing
Better asserts in parsing
This commit is contained in:
@@ -397,8 +397,8 @@ const char* XMLUtil::GetCharacterRef( const char* p, char* value, int* length )
|
|||||||
else {
|
else {
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
TIXMLASSERT( digit == 0 || mult <= UINT_MAX / digit );
|
|
||||||
TIXMLASSERT( digit >= 0 && digit < 16);
|
TIXMLASSERT( digit >= 0 && digit < 16);
|
||||||
|
TIXMLASSERT( digit == 0 || mult <= UINT_MAX / digit );
|
||||||
const unsigned int digitScaled = mult * digit;
|
const unsigned int digitScaled = mult * digit;
|
||||||
TIXMLASSERT( ucs <= ULONG_MAX - digitScaled );
|
TIXMLASSERT( ucs <= ULONG_MAX - digitScaled );
|
||||||
ucs += digitScaled;
|
ucs += digitScaled;
|
||||||
@@ -427,6 +427,7 @@ const char* XMLUtil::GetCharacterRef( const char* p, char* value, int* length )
|
|||||||
while ( *q != '#' ) {
|
while ( *q != '#' ) {
|
||||||
if ( *q >= '0' && *q <= '9' ) {
|
if ( *q >= '0' && *q <= '9' ) {
|
||||||
const unsigned int digit = *q - '0';
|
const unsigned int digit = *q - '0';
|
||||||
|
TIXMLASSERT( digit >= 0 && digit < 10);
|
||||||
TIXMLASSERT( digit == 0 || mult <= UINT_MAX / digit );
|
TIXMLASSERT( digit == 0 || mult <= UINT_MAX / digit );
|
||||||
const unsigned int digitScaled = mult * digit;
|
const unsigned int digitScaled = mult * digit;
|
||||||
TIXMLASSERT( ucs <= ULONG_MAX - digitScaled );
|
TIXMLASSERT( ucs <= ULONG_MAX - digitScaled );
|
||||||
|
|||||||
Reference in New Issue
Block a user