Merge pull request #18154 from taosdata/fix/TD-20417-D
fix: null pointer check when invoke memcpy
This commit is contained in:
commit
be6ff9a46e
|
@ -93,7 +93,7 @@ static SHNode *doCreateHashNode(const void *key, size_t keyLen, const void *data
|
||||||
pNewNode->keyLen = keyLen;
|
pNewNode->keyLen = keyLen;
|
||||||
pNewNode->dataLen = dataLen;
|
pNewNode->dataLen = dataLen;
|
||||||
pNewNode->next = NULL;
|
pNewNode->next = NULL;
|
||||||
memcpy(GET_SHASH_NODE_DATA(pNewNode), data, dataLen);
|
if (data) memcpy(GET_SHASH_NODE_DATA(pNewNode), data, dataLen);
|
||||||
memcpy(GET_SHASH_NODE_KEY(pNewNode, dataLen), key, keyLen);
|
memcpy(GET_SHASH_NODE_KEY(pNewNode, dataLen), key, keyLen);
|
||||||
return pNewNode;
|
return pNewNode;
|
||||||
}
|
}
|
||||||
|
@ -203,7 +203,7 @@ int32_t tSimpleHashPut(SSHashObj *pHashObj, const void *key, size_t keyLen, cons
|
||||||
pNewNode->next = pHashObj->hashList[slot];
|
pNewNode->next = pHashObj->hashList[slot];
|
||||||
pHashObj->hashList[slot] = pNewNode;
|
pHashObj->hashList[slot] = pNewNode;
|
||||||
atomic_add_fetch_64(&pHashObj->size, 1);
|
atomic_add_fetch_64(&pHashObj->size, 1);
|
||||||
} else { // update data
|
} else if (data) { // update data
|
||||||
memcpy(GET_SHASH_NODE_DATA(pNode), data, dataLen);
|
memcpy(GET_SHASH_NODE_DATA(pNode), data, dataLen);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue