From 9b74b00c8e7ab3ed584097f390e5432c18fe3305 Mon Sep 17 00:00:00 2001 From: Alexander Barkov Date: Fri, 4 Aug 2017 14:41:05 +0400 Subject: [PATCH] Adding the "const" qualified to the LEX_CSTRING parameter of a few check_xxx() functions Functions: check_routine_name() check_string_byte_length() check_string_char_length() check_ident_length() --- sql/sp_head.cc | 2 +- sql/sp_head.h | 2 +- sql/sql_parse.cc | 6 +++--- sql/sql_parse.h | 6 +++--- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/sql/sp_head.cc b/sql/sp_head.cc index 9737169ede6..1f71139b9af 100644 --- a/sql/sp_head.cc +++ b/sql/sp_head.cc @@ -480,7 +480,7 @@ sp_name::sp_name(const MDL_key *key, char *qname_buff) */ bool -check_routine_name(LEX_CSTRING *ident) +check_routine_name(const LEX_CSTRING *ident) { DBUG_ASSERT(ident); DBUG_ASSERT(ident->str); diff --git a/sql/sp_head.h b/sql/sp_head.h index f1d89a083ab..f9a021d535e 100644 --- a/sql/sp_head.h +++ b/sql/sp_head.h @@ -125,7 +125,7 @@ public: bool -check_routine_name(LEX_CSTRING *ident); +check_routine_name(const LEX_CSTRING *ident); class sp_head :private Query_arena, public Database_qualified_name diff --git a/sql/sql_parse.cc b/sql/sql_parse.cc index 056f4cefe16..8f403e0bef8 100644 --- a/sql/sql_parse.cc +++ b/sql/sql_parse.cc @@ -9677,7 +9677,7 @@ LEX_USER *create_definer(THD *thd, LEX_CSTRING *user_name, The function is not used in existing code but can be useful later? */ -bool check_string_byte_length(LEX_CSTRING *str, uint err_msg, +bool check_string_byte_length(const LEX_CSTRING *str, uint err_msg, uint max_byte_length) { if (str->length <= max_byte_length) @@ -9707,7 +9707,7 @@ bool check_string_byte_length(LEX_CSTRING *str, uint err_msg, */ -bool check_string_char_length(LEX_CSTRING *str, uint err_msg, +bool check_string_char_length(const LEX_CSTRING *str, uint err_msg, uint max_char_length, CHARSET_INFO *cs, bool no_error) { @@ -9726,7 +9726,7 @@ bool check_string_char_length(LEX_CSTRING *str, uint err_msg, } -bool check_ident_length(LEX_CSTRING *ident) +bool check_ident_length(const LEX_CSTRING *ident) { if (check_string_char_length(ident, 0, NAME_CHAR_LEN, system_charset_info, 1)) { diff --git a/sql/sql_parse.h b/sql/sql_parse.h index c97d824ffa1..b8d7bb46e08 100644 --- a/sql/sql_parse.h +++ b/sql/sql_parse.h @@ -73,12 +73,12 @@ LEX_USER *create_default_definer(THD *thd, bool role); LEX_USER *create_definer(THD *thd, LEX_CSTRING *user_name, LEX_CSTRING *host_name); LEX_USER *get_current_user(THD *thd, LEX_USER *user, bool lock=true); bool sp_process_definer(THD *thd); -bool check_string_byte_length(LEX_CSTRING *str, uint err_msg, +bool check_string_byte_length(const LEX_CSTRING *str, uint err_msg, uint max_byte_length); -bool check_string_char_length(LEX_CSTRING *str, uint err_msg, +bool check_string_char_length(const LEX_CSTRING *str, uint err_msg, uint max_char_length, CHARSET_INFO *cs, bool no_error); -bool check_ident_length(LEX_CSTRING *ident); +bool check_ident_length(const LEX_CSTRING *ident); CHARSET_INFO* merge_charset_and_collation(CHARSET_INFO *cs, CHARSET_INFO *cl); CHARSET_INFO *find_bin_collation(CHARSET_INFO *cs); bool check_host_name(LEX_CSTRING *str);