hcq/oceanbase
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix SeArray memory leak when deserializing rcode_.warnings_

Browse Source
This commit is contained in:
liucc1997
2023-05-17 13:46:26 +00:00
committed by ob-robot
parent d07260f033
commit 29be7ce8ce
2 changed files with 26 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
deps/oblib/src/rpc/obrpc/ob_poc_rpc_proxy.cpp vendored
View File

@ -87,6 +87,9 @@ ObAsyncRespCallback* ObAsyncRespCallback::create(ObRpcMemPool& pool, UAsyncCB* u
RPC_LOG(WARN, "ucb.clone fail", K(ret)); RPC_LOG(WARN, "ucb.clone fail", K(ret));
} else { } else {
cb->low_level_cb_ = pcb; cb->low_level_cb_ = pcb;
if (cb != ucb) {
cb->set_cloned(true);
}
} }
} }
new(pcb)ObAsyncRespCallback(pool, cb); new(pcb)ObAsyncRespCallback(pool, cb);
@ -107,7 +110,9 @@ int ObAsyncRespCallback::handle_resp(int io_err, const char* buf, int64_t sz)
} }
if (ucb_ == NULL) { if (ucb_ == NULL) {
// do nothing // do nothing
} else if (0 != io_err) { } else {
bool cb_cloned = ucb_->get_cloned();
if (0 != io_err) {
ucb_->set_error(io_err); ucb_->set_error(io_err);
if (OB_SUCCESS != ucb_->on_error(io_err)) { if (OB_SUCCESS != ucb_->on_error(io_err)) {
ucb_->on_timeout(); ucb_->on_timeout();
@ -126,6 +131,10 @@ int ObAsyncRespCallback::handle_resp(int io_err, const char* buf, int64_t sz)
RPC_LOG(WARN, "ucb.process fail", K(tmp_ret)); RPC_LOG(WARN, "ucb.process fail", K(tmp_ret));
} }
} }
if (cb_cloned) {
ucb_->~AsyncCB();
}
}
pool_.destroy(); pool_.destroy();
return ret; return ret;
} }

2
deps/oblib/src/rpc/pnio/io/write_queue.c vendored
View File

@ -92,7 +92,7 @@ int wq_flush(sock_t* s, write_queue_t* wq, dlink_t** old_head) {
int err = 0; int err = 0;
int64_t wbytes = 0; int64_t wbytes = 0;
err = sk_flush_blist((sock_t*)s, &wq->queue.head, wq->pos, &wbytes); err = sk_flush_blist((sock_t*)s, &wq->queue.head, wq->pos, &wbytes);
if (0 == err) { if (0 == err && wbytes > 0) {
*old_head = wq_consume(wq, wbytes); *old_head = wq_consume(wq, wbytes);
} }
return err; return err;