fix: data deleter memory leak issue
This commit is contained in:
parent
189dadeadb
commit
e51199d541
|
@ -273,10 +273,18 @@ static int32_t getCacheSize(struct SDataSinkHandle* pHandle, uint64_t* size) {
|
||||||
int32_t createDataDeleter(SDataSinkManager* pManager, const SDataSinkNode* pDataSink, DataSinkHandle* pHandle,
|
int32_t createDataDeleter(SDataSinkManager* pManager, const SDataSinkNode* pDataSink, DataSinkHandle* pHandle,
|
||||||
void* pParam) {
|
void* pParam) {
|
||||||
int32_t code = TSDB_CODE_SUCCESS;
|
int32_t code = TSDB_CODE_SUCCESS;
|
||||||
|
if (pParam == NULL) {
|
||||||
|
code = TSDB_CODE_QRY_INVALID_INPUT;
|
||||||
|
qError("invalid input param in creating data deleter, code%s", tstrerror(code));
|
||||||
|
goto _end;
|
||||||
|
}
|
||||||
|
|
||||||
|
SDeleterParam* pDeleterParam = (SDeleterParam*)pParam;
|
||||||
|
|
||||||
SDataDeleterHandle* deleter = taosMemoryCalloc(1, sizeof(SDataDeleterHandle));
|
SDataDeleterHandle* deleter = taosMemoryCalloc(1, sizeof(SDataDeleterHandle));
|
||||||
if (NULL == deleter) {
|
if (NULL == deleter) {
|
||||||
code = terrno;
|
code = terrno;
|
||||||
|
taosArrayDestroy(pDeleterParam->pUidList);
|
||||||
taosMemoryFree(pParam);
|
taosMemoryFree(pParam);
|
||||||
goto _end;
|
goto _end;
|
||||||
}
|
}
|
||||||
|
@ -292,12 +300,6 @@ int32_t createDataDeleter(SDataSinkManager* pManager, const SDataSinkNode* pData
|
||||||
deleter->pDeleter = pDeleterNode;
|
deleter->pDeleter = pDeleterNode;
|
||||||
deleter->pSchema = pDataSink->pInputDataBlockDesc;
|
deleter->pSchema = pDataSink->pInputDataBlockDesc;
|
||||||
|
|
||||||
if (pParam == NULL) {
|
|
||||||
code = TSDB_CODE_QRY_INVALID_INPUT;
|
|
||||||
qError("invalid input param in creating data deleter, code%s", tstrerror(code));
|
|
||||||
goto _end;
|
|
||||||
}
|
|
||||||
|
|
||||||
deleter->pParam = pParam;
|
deleter->pParam = pParam;
|
||||||
deleter->status = DS_BUF_EMPTY;
|
deleter->status = DS_BUF_EMPTY;
|
||||||
deleter->queryEnd = false;
|
deleter->queryEnd = false;
|
||||||
|
|
Loading…
Reference in New Issue