fix(query): check for null ptr.
This commit is contained in:
parent
cc6daa0855
commit
922b0681ea
|
@ -1246,11 +1246,17 @@ void destroyIntervalOperatorInfo(void* param) {
|
||||||
if (param == NULL) {
|
if (param == NULL) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
SIntervalAggOperatorInfo* pInfo = (SIntervalAggOperatorInfo*)param;
|
SIntervalAggOperatorInfo* pInfo = (SIntervalAggOperatorInfo*)param;
|
||||||
|
|
||||||
cleanupBasicInfo(&pInfo->binfo);
|
cleanupBasicInfo(&pInfo->binfo);
|
||||||
cleanupResultInfo(pInfo->pOperator->pTaskInfo, &pInfo->pOperator->exprSupp, pInfo->aggSup.pResultBuf,
|
|
||||||
&pInfo->groupResInfo, pInfo->aggSup.pResultRowHashTable);
|
if (pInfo->pOperator) {
|
||||||
pInfo->pOperator = NULL;
|
cleanupResultInfo(pInfo->pOperator->pTaskInfo, &pInfo->pOperator->exprSupp, pInfo->aggSup.pResultBuf,
|
||||||
|
&pInfo->groupResInfo, pInfo->aggSup.pResultRowHashTable);
|
||||||
|
pInfo->pOperator = NULL;
|
||||||
|
}
|
||||||
|
|
||||||
cleanupAggSup(&pInfo->aggSup);
|
cleanupAggSup(&pInfo->aggSup);
|
||||||
cleanupExprSupp(&pInfo->scalarSupp);
|
cleanupExprSupp(&pInfo->scalarSupp);
|
||||||
|
|
||||||
|
@ -1258,6 +1264,7 @@ void destroyIntervalOperatorInfo(void* param) {
|
||||||
|
|
||||||
taosArrayDestroy(pInfo->pInterpCols);
|
taosArrayDestroy(pInfo->pInterpCols);
|
||||||
pInfo->pInterpCols = NULL;
|
pInfo->pInterpCols = NULL;
|
||||||
|
|
||||||
taosArrayDestroyEx(pInfo->pPrevValues, freeItem);
|
taosArrayDestroyEx(pInfo->pPrevValues, freeItem);
|
||||||
pInfo->pPrevValues = NULL;
|
pInfo->pPrevValues = NULL;
|
||||||
|
|
||||||
|
@ -1351,6 +1358,7 @@ int32_t createIntervalOperatorInfo(SOperatorInfo* downstream, SIntervalPhysiNode
|
||||||
SOperatorInfo* pOperator = taosMemoryCalloc(1, sizeof(SOperatorInfo));
|
SOperatorInfo* pOperator = taosMemoryCalloc(1, sizeof(SOperatorInfo));
|
||||||
if (pInfo == NULL || pOperator == NULL) {
|
if (pInfo == NULL || pOperator == NULL) {
|
||||||
code = terrno;
|
code = terrno;
|
||||||
|
lino = __LINE__;
|
||||||
goto _error;
|
goto _error;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1458,8 +1466,10 @@ _error:
|
||||||
if (pInfo != NULL) {
|
if (pInfo != NULL) {
|
||||||
destroyIntervalOperatorInfo(pInfo);
|
destroyIntervalOperatorInfo(pInfo);
|
||||||
}
|
}
|
||||||
|
|
||||||
destroyOperatorAndDownstreams(pOperator, &downstream, 1);
|
destroyOperatorAndDownstreams(pOperator, &downstream, 1);
|
||||||
pTaskInfo->code = code;
|
pTaskInfo->code = code;
|
||||||
|
qError("error happens at %s %d, code:%s", __func__, lino, tstrerror(code));
|
||||||
return code;
|
return code;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue