bzr commit into mysql-5.1-bugteam branch (azundris:2826) Bug#40657

[Tatiana A. Nurnberg]

#At file:///misc/mysql/forest/40657_/51-40657_/ based on revid:azundris@stripped

 2826 Tatiana A. Nurnberg	2009-03-05
      Bug#40657: assertion with out of range variables and traditional sql_mode
      
      Some changes that make things more elegant in 6.0,
      backported so the code is mostly the same in 5.1,
      for easier maintainence.
     @ sql/set_var.cc
        Backporting some changes to 5.1 so the code is
        mostly the same in both versions. Less to learn!

    modified:
      sql/set_var.cc
=== modified file 'sql/set_var.cc'
--- a/sql/set_var.cc	2009-02-27 20:43:43 +0000
+++ b/sql/set_var.cc	2009-03-05 15:42:12 +0000
@@ -1455,13 +1455,12 @@ static void bound_unsigned(THD *thd, ulo
   @param thd             thread handle
   @param var             the system-variable to get
   @param user_max        a limit given with --maximum-variable-name=... or 0
-  @param bound2ulong     pass TRUE if size is ulong, not ulonglong.  function
-                         will then bound on systems where it's necessary.
+  @param var_type        function will bound on systems where necessary.
 
   @retval                TRUE on error, FALSE otherwise (warning or OK)
  */
 static bool get_unsigned(THD *thd, set_var *var, ulonglong user_max,
-                         my_bool bound2ulong)
+                         ulong var_type)
 {
   int                     warnings= 0;
   ulonglong               unadjusted;
@@ -1506,10 +1505,10 @@ static bool get_unsigned(THD *thd, set_v
     the usual suspects handle the actual limiting.
   */
 
-  if (!limits && bound2ulong)
+  if (!limits && var_type != GET_ULL)
   {
     bzero(&fallback, sizeof(fallback));
-    fallback.var_type= GET_ULONG;
+    fallback.var_type= var_type;
     limits= &fallback;
   }
 
@@ -1541,7 +1540,7 @@ sys_var_long_ptr(sys_var_chain *chain, c
 
 bool sys_var_long_ptr_global::check(THD *thd, set_var *var)
 {
-  return get_unsigned(thd, var, 0, TRUE);
+  return get_unsigned(thd, var, 0, GET_ULONG);
 }
 
 bool sys_var_long_ptr_global::update(THD *thd, set_var *var)
@@ -1618,7 +1617,7 @@ uchar *sys_var_enum_const::value_ptr(THD
 
 bool sys_var_thd_ulong::check(THD *thd, set_var *var)
 {
-  if (get_unsigned(thd, var, max_system_variables.*offset, TRUE))
+  if (get_unsigned(thd, var, max_system_variables.*offset, GET_ULONG))
     return TRUE;
   DBUG_ASSERT(var->save_result.ulonglong_value <= ULONG_MAX);
   return ((check_func && (*check_func)(thd, var)));
@@ -1709,7 +1708,7 @@ uchar *sys_var_thd_ha_rows::value_ptr(TH
 
 bool sys_var_thd_ulonglong::check(THD *thd, set_var *var)
 {
-  return get_unsigned(thd, var, max_system_variables.*offset, FALSE);
+  return get_unsigned(thd, var, max_system_variables.*offset, GET_ULL);
 }
 
 bool sys_var_thd_ulonglong::update(THD *thd,  set_var *var)
@@ -2384,7 +2383,7 @@ end:
 */
 bool sys_var_key_cache_long::update(THD *thd, set_var *var)
 {
-  ulonglong tmp= (ulonglong) (ulong) var->value->val_int();
+  ulonglong tmp= var->value->val_int();
   LEX_STRING *base_name= &var->base;
   bool error= 0;
 


Attachment: [text/bzr-bundle] bzr/[e-mail]