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

fix bug missing a error_code

Browse Source
This commit is contained in:
jingtaoye35
2023-04-17 05:11:33 +00:00
committed by ob-robot
parent 2ef15964cb
commit a84e9d9634
2 changed files with 18 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/sql/resolver/dcl/ob_alter_user_profile_resolver.cpp
View File

@ -321,7 +321,11 @@ int ObAlterUserProfileResolver::resolve_default_role(const ParseNode &parse_tree
} }
OZ (params_.schema_checker_->get_user_info(tenant_id, user_name, host_name, user_info), OZ (params_.schema_checker_->get_user_info(tenant_id, user_name, host_name, user_info),
tenant_id, user_name, host_name); tenant_id, user_name, host_name);
if (ret == OB_USER_NOT_EXIST || OB_ISNULL(user_info)) { if (ret == OB_USER_NOT_EXIST) {
LOG_USER_ERROR(OB_USER_NOT_EXIST, user_name.length(), user_name.ptr());
}
if (OB_SUCC(ret)) {
if (user_info == NULL) {
ret = OB_USER_NOT_EXIST; ret = OB_USER_NOT_EXIST;
LOG_USER_ERROR(OB_USER_NOT_EXIST, user_name.length(), user_name.ptr()); LOG_USER_ERROR(OB_USER_NOT_EXIST, user_name.length(), user_name.ptr());
} else if (OB_FAIL(check_dcl_on_inner_user(parse_tree.type_, } else if (OB_FAIL(check_dcl_on_inner_user(parse_tree.type_,
@ -332,6 +336,7 @@ int ObAlterUserProfileResolver::resolve_default_role(const ParseNode &parse_tree
} else { } else {
arg.user_id_ = user_info->get_user_id(); arg.user_id_ = user_info->get_user_id();
} }
}
/* 2. resolve default role */ /* 2. resolve default role */
OZ (resolve_default_role_clause(parse_tree.children_[1], arg, OZ (resolve_default_role_clause(parse_tree.children_[1], arg,

5
src/sql/resolver/dcl/ob_revoke_resolver.cpp
View File

@ -191,8 +191,9 @@ int ObRevokeResolver::resolve_revoke_sysprivs_inner(
const ObString &user_name = user_name_array.at(i); const ObString &user_name = user_name_array.at(i);
const ObString &host_name = host_name_array.at(i); const ObString &host_name = host_name_array.at(i);
OZ (revoke_stmt->add_grantee(user_name)); OZ (revoke_stmt->add_grantee(user_name));
if (OB_SUCC(ret) && OB_FAIL(params_.schema_checker_->get_user_id( if (OB_FAIL(ret)) {
revoke_stmt->get_tenant_id(), user_name, host_name, user_id))) { } else if (OB_FAIL(params_.schema_checker_->get_user_id(revoke_stmt->get_tenant_id(), user_name,
host_name, user_id))) {
if (OB_USER_NOT_EXIST == ret) { if (OB_USER_NOT_EXIST == ret) {
ret = OB_ERR_USER_OR_ROLE_DOES_NOT_EXIST; ret = OB_ERR_USER_OR_ROLE_DOES_NOT_EXIST;
LOG_USER_ERROR(OB_ERR_USER_OR_ROLE_DOES_NOT_EXIST, user_name.length(), user_name.ptr()); LOG_USER_ERROR(OB_ERR_USER_OR_ROLE_DOES_NOT_EXIST, user_name.length(), user_name.ptr());