diff options
author | sangwan.kwon <sangwan.kwon@samsung.com> | 2016-08-01 17:17:54 +0900 |
---|---|---|
committer | sangwan.kwon <sangwan.kwon@samsung.com> | 2016-08-01 18:23:07 +0900 |
commit | 36eb4021277e375bdb179970bac24553cfc0cde9 (patch) | |
tree | c688fe61c115d3e6b82aabbcfbdddd35f32e103b /src/bn.c | |
parent | f732baaeb6fc2e9872696cb381b90b8c523d0c53 (diff) | |
download | xmlsec1-36eb4021277e375bdb179970bac24553cfc0cde9.tar.gz xmlsec1-36eb4021277e375bdb179970bac24553cfc0cde9.tar.bz2 xmlsec1-36eb4021277e375bdb179970bac24553cfc0cde9.zip |
Fix svace defects
* null check to return value
* overflow check on array
* fix dead code (it seem to typo, (++ptr))
Change-Id: I184b29a184812c745a8ee192632ea101e349b10d
Signed-off-by: sangwan.kwon <sangwan.kwon@samsung.com>
Diffstat (limited to 'src/bn.c')
-rw-r--r-- | src/bn.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -252,7 +252,7 @@ xmlSecBnFromString(xmlSecBnPtr bn, const xmlChar* str, xmlSecSize base) { continue; } - xmlSecAssert2(ch <= sizeof(xmlSecBnLookupTable), -1); + xmlSecAssert2(ch < sizeof(xmlSecBnLookupTable) / sizeof(xmlSecBnLookupTable[0]), -1); nn = xmlSecBnLookupTable[ch]; if((nn < 0) || ((xmlSecSize)nn > base)) { xmlSecError(XMLSEC_ERRORS_HERE, |