fix(query): user/role name not support \b and \f (#17530)

Author: TCeason

src/query/sql/src/planner/binder/binder.rs

@@ -408,7 +408,7 @@ impl<'a> Binder {
             } => {
                 if illegal_ident_name(role_name) {
                     return Err(ErrorCode::IllegalRole(
-                        format!("Illegal Role Name: Illegal role name [{}], not support username contain ' or \"", role_name),
+                        format!("Illegal Role Name: Illegal role name [{}], not support username contain ' or \" or \\b or \\f", role_name),
                     ));
                 }
                 Plan::CreateRole(Box::new(CreateRolePlan {

src/query/sql/src/planner/binder/ddl/account.rs

@@ -265,7 +265,7 @@ impl Binder {
         } = stmt;
         if illegal_ident_name(&user.username) {
             return Err(ErrorCode::IllegalUser(format!(
-                "Illegal Username: Illegal user name [{}], not support username contain ' or \"",
+                "Illegal Username: Illegal user name [{}], not support username contain ' or \" \\b or \\f",
                 user.username
             )));
         }

src/query/sql/src/planner/binder/util.rs

@@ -33,7 +33,9 @@ use crate::NameResolutionSuggest;
 /// Ident name can not contain ' or "
 /// Forbidden ' or " in UserName and RoleName, to prevent Meta injection problem
 pub fn illegal_ident_name(ident_name: &str) -> bool {
-    ident_name.chars().any(|c| c == '\'' || c == '\"')
+    ident_name
+        .chars()
+        .any(|c| c == '\'' || c == '\"' || c == '\u{000C}' || c == '\u{0008}')
 }
 
 impl Binder {

tests/sqllogictests/suites/base/05_ddl/05_0004_ddl_create_user.test

@@ -90,3 +90,15 @@ create user `a"a` identified by '123'
 
 statement error 2218
 create user `a'a` identified by '123'
+
+statement error 2218
+CREATE user 'a\b' identified by '123'
+
+statement error 2218
+CREATE user 'a\f' identified by '123'
+
+statement ok
+drop user if exists 'a\b';
+
+statement ok
+drop user if exists 'a\f';

tests/sqllogictests/suites/base/05_ddl/05_0014_ddl_create_role.test

@@ -7,6 +7,18 @@ CREATE ROLE `test-a`
 statement error 2216
 CREATE ROLE `test-a`
 
+statement error 2217
+CREATE ROLE 'a\b'
+
+statement error 2217
+CREATE ROLE 'a\f'
+
+statement ok
+drop role if exists 'a\b';
+
+statement ok
+drop role if exists 'a\f';
+
 statement ok
 CREATE ROLE IF NOT EXISTS `test-a`
 
@@ -32,7 +44,7 @@ statement error 2217
 create role 'public'
 
 statement error 2217
-create role `a"a`
+create role 'a"a'
 
 statement error 2217
-create role `a'a`
+create role "a'a"