fix: client tmq connector coverity scan issues (#17573)
This commit is contained in:
parent
d898e25f59
commit
7023e465cf
|
@ -73,8 +73,8 @@ JNIEXPORT void JNICALL Java_com_taosdata_jdbc_tmq_TMQConnector_tmqConfDestroyImp
|
||||||
if (conf == NULL) {
|
if (conf == NULL) {
|
||||||
jniDebug("jobj:%p, tmq config is already destroyed", jobj);
|
jniDebug("jobj:%p, tmq config is already destroyed", jobj);
|
||||||
} else {
|
} else {
|
||||||
tmq_conf_destroy(conf);
|
|
||||||
jniDebug("jobj:%p, config:%p, tmq successfully destroy config", jobj, conf);
|
jniDebug("jobj:%p, config:%p, tmq successfully destroy config", jobj, conf);
|
||||||
|
tmq_conf_destroy(conf);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -206,6 +206,7 @@ JNIEXPORT void JNICALL Java_com_taosdata_jdbc_tmq_TMQConnector_tmqCommitAsync(JN
|
||||||
tmq_t *tmq = (tmq_t *)jtmq;
|
tmq_t *tmq = (tmq_t *)jtmq;
|
||||||
if (tmq == NULL) {
|
if (tmq == NULL) {
|
||||||
jniError("jobj:%p, tmq is closed", jobj);
|
jniError("jobj:%p, tmq is closed", jobj);
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
TAOS_RES *res = (TAOS_RES *)jres;
|
TAOS_RES *res = (TAOS_RES *)jres;
|
||||||
consumer = (*env)->NewGlobalRef(env, consumer);
|
consumer = (*env)->NewGlobalRef(env, consumer);
|
||||||
|
@ -252,6 +253,7 @@ JNIEXPORT jstring JNICALL Java_com_taosdata_jdbc_tmq_TMQConnector_tmqGetTopicNam
|
||||||
TAOS_RES *res = (TAOS_RES *)jres;
|
TAOS_RES *res = (TAOS_RES *)jres;
|
||||||
if (res == NULL) {
|
if (res == NULL) {
|
||||||
jniDebug("jobj:%p, invalid res handle", jobj);
|
jniDebug("jobj:%p, invalid res handle", jobj);
|
||||||
|
return NULL;
|
||||||
}
|
}
|
||||||
return (*env)->NewStringUTF(env, tmq_get_topic_name(res));
|
return (*env)->NewStringUTF(env, tmq_get_topic_name(res));
|
||||||
}
|
}
|
||||||
|
@ -259,6 +261,7 @@ JNIEXPORT jstring JNICALL Java_com_taosdata_jdbc_tmq_TMQConnector_tmqGetDbName(J
|
||||||
TAOS_RES *res = (TAOS_RES *)jres;
|
TAOS_RES *res = (TAOS_RES *)jres;
|
||||||
if (res == NULL) {
|
if (res == NULL) {
|
||||||
jniDebug("jobj:%p, invalid res handle", jobj);
|
jniDebug("jobj:%p, invalid res handle", jobj);
|
||||||
|
return NULL;
|
||||||
}
|
}
|
||||||
return (*env)->NewStringUTF(env, tmq_get_db_name(res));
|
return (*env)->NewStringUTF(env, tmq_get_db_name(res));
|
||||||
}
|
}
|
||||||
|
@ -266,6 +269,7 @@ JNIEXPORT jint JNICALL Java_com_taosdata_jdbc_tmq_TMQConnector_tmqGetVgroupId(JN
|
||||||
TAOS_RES *res = (TAOS_RES *)jres;
|
TAOS_RES *res = (TAOS_RES *)jres;
|
||||||
if (res == NULL) {
|
if (res == NULL) {
|
||||||
jniDebug("jobj:%p, invalid res handle", jobj);
|
jniDebug("jobj:%p, invalid res handle", jobj);
|
||||||
|
return -1;
|
||||||
}
|
}
|
||||||
return tmq_get_vgroup_id(res);
|
return tmq_get_vgroup_id(res);
|
||||||
}
|
}
|
||||||
|
@ -275,6 +279,7 @@ JNIEXPORT jstring JNICALL Java_com_taosdata_jdbc_tmq_TMQConnector_tmqGetTableNam
|
||||||
TAOS_RES *res = (TAOS_RES *)jres;
|
TAOS_RES *res = (TAOS_RES *)jres;
|
||||||
if (res == NULL) {
|
if (res == NULL) {
|
||||||
jniDebug("jobj:%p, invalid res handle", jobj);
|
jniDebug("jobj:%p, invalid res handle", jobj);
|
||||||
|
return NULL;
|
||||||
}
|
}
|
||||||
return (*env)->NewStringUTF(env, tmq_get_table_name(res));
|
return (*env)->NewStringUTF(env, tmq_get_table_name(res));
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue