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

fix session may be leaked when init mgr failed

Browse Source
This commit is contained in:
obdev
2023-08-03 07:18:13 +00:00
committed by ob-robot
parent 7d7b5f5e28
commit 09e015fb7b
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/sql/session/ob_sql_session_mgr.cpp
View File

@ -426,6 +426,14 @@ int ObSQLSessionMgr::create_session(const uint64_t tenant_id,
ObFLTControlInfoManager mgr(tenant_id); ObFLTControlInfoManager mgr(tenant_id);
if (OB_FAIL(mgr.init())) { if (OB_FAIL(mgr.init())) {
LOG_WARN("failed to init full link control info", K(ret)); LOG_WARN("failed to init full link control info", K(ret));
if (FALSE_IT(revert_session(tmp_sess))) {
LOG_ERROR("fail to free session", K(err), K(sessid), K(proxy_sessid));
} else if (OB_SUCCESS != (err = sessinfo_map_.del(Key(sessid)))) {
LOG_ERROR("fail to free session", K(err), K(sessid), K(proxy_sessid));
} else {
LOG_DEBUG("free session successfully in create session", K(err),
K(sessid), K(proxy_sessid));
}
} else if (mgr.is_valid_tenant_config()) { } else if (mgr.is_valid_tenant_config()) {
tmp_sess->set_flt_control_info(mgr.get_control_info()); tmp_sess->set_flt_control_info(mgr.get_control_info());
} }