Merge pull request #4359 from taosdata/bugfix/td-2182
[TD-2182]<fix>: coverity issues
This commit is contained in:
commit
f3aa7cff18
|
@ -442,7 +442,12 @@ TAOS_RES *taos_consume(TAOS_SUB *tsub) {
|
||||||
|
|
||||||
size_t size = taosArrayGetSize(pSub->progress) * sizeof(STableIdInfo);
|
size_t size = taosArrayGetSize(pSub->progress) * sizeof(STableIdInfo);
|
||||||
size += sizeof(SQueryTableMsg) + 4096;
|
size += sizeof(SQueryTableMsg) + 4096;
|
||||||
tscAllocPayload(&pSql->cmd, (int)size);
|
int code = tscAllocPayload(&pSql->cmd, (int)size);
|
||||||
|
if (code != TSDB_CODE_SUCCESS) {
|
||||||
|
tscError("failed to alloc payload");
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
for (int retry = 0; retry < 3; retry++) {
|
for (int retry = 0; retry < 3; retry++) {
|
||||||
tscRemoveFromSqlList(pSql);
|
tscRemoveFromSqlList(pSql);
|
||||||
|
|
||||||
|
|
|
@ -5836,9 +5836,9 @@ static int32_t convertQueryMsg(SQueryTableMsg *pQueryMsg, SArray **pTableIdList,
|
||||||
pQueryMsg->interval.interval = htobe64(pQueryMsg->interval.interval);
|
pQueryMsg->interval.interval = htobe64(pQueryMsg->interval.interval);
|
||||||
pQueryMsg->interval.sliding = htobe64(pQueryMsg->interval.sliding);
|
pQueryMsg->interval.sliding = htobe64(pQueryMsg->interval.sliding);
|
||||||
pQueryMsg->interval.offset = htobe64(pQueryMsg->interval.offset);
|
pQueryMsg->interval.offset = htobe64(pQueryMsg->interval.offset);
|
||||||
pQueryMsg->interval.intervalUnit = pQueryMsg->interval.intervalUnit;
|
// pQueryMsg->interval.intervalUnit = pQueryMsg->interval.intervalUnit;
|
||||||
pQueryMsg->interval.slidingUnit = pQueryMsg->interval.slidingUnit;
|
// pQueryMsg->interval.slidingUnit = pQueryMsg->interval.slidingUnit;
|
||||||
pQueryMsg->interval.offsetUnit = pQueryMsg->interval.offsetUnit;
|
// pQueryMsg->interval.offsetUnit = pQueryMsg->interval.offsetUnit;
|
||||||
pQueryMsg->limit = htobe64(pQueryMsg->limit);
|
pQueryMsg->limit = htobe64(pQueryMsg->limit);
|
||||||
pQueryMsg->offset = htobe64(pQueryMsg->offset);
|
pQueryMsg->offset = htobe64(pQueryMsg->offset);
|
||||||
|
|
||||||
|
|
|
@ -79,15 +79,12 @@ void taosCloseQueue(taos_queue param) {
|
||||||
|
|
||||||
if (queue->qset) taosRemoveFromQset(qset, queue);
|
if (queue->qset) taosRemoveFromQset(qset, queue);
|
||||||
|
|
||||||
pthread_mutex_lock(&queue->mutex);
|
|
||||||
|
|
||||||
while (pNode) {
|
while (pNode) {
|
||||||
pTemp = pNode;
|
pTemp = pNode;
|
||||||
pNode = pNode->next;
|
pNode = pNode->next;
|
||||||
free (pTemp);
|
free (pTemp);
|
||||||
}
|
}
|
||||||
|
|
||||||
pthread_mutex_unlock(&queue->mutex);
|
|
||||||
pthread_mutex_destroy(&queue->mutex);
|
pthread_mutex_destroy(&queue->mutex);
|
||||||
free(queue);
|
free(queue);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue