Commit 676f9868 authored by unknown's avatar unknown

Produce an error when arguments are not compatible for CONCAT()

parent 80630f12
...@@ -182,10 +182,7 @@ bool Item::set_charset(CHARSET_INFO *cs1, enum coercion co1, ...@@ -182,10 +182,7 @@ bool Item::set_charset(CHARSET_INFO *cs1, enum coercion co1,
} }
if (!my_charset_same(cs1,cs2)) if (!my_charset_same(cs1,cs2))
{ return 1;
set_charset(&my_charset_bin, COER_NOCOLL);
return 0;
}
if (co1 < co2) if (co1 < co2)
{ {
...@@ -198,7 +195,12 @@ bool Item::set_charset(CHARSET_INFO *cs1, enum coercion co1, ...@@ -198,7 +195,12 @@ bool Item::set_charset(CHARSET_INFO *cs1, enum coercion co1,
else // co2 == co1 else // co2 == co1
{ {
if (cs1 != cs2) if (cs1 != cs2)
set_charset(&my_charset_bin, COER_NOCOLL); {
CHARSET_INFO *bin= get_charset_by_csname(cs1->csname, MY_CS_BINSORT,MYF(0));
if (!bin)
return 1;
set_charset(bin, COER_NOCOLL);
}
else else
set_charset(cs2, co2); set_charset(cs2, co2);
} }
......
...@@ -328,8 +328,12 @@ void Item_func_concat::fix_length_and_dec() ...@@ -328,8 +328,12 @@ void Item_func_concat::fix_length_and_dec()
for (uint i=0 ; i < arg_count ; i++) for (uint i=0 ; i < arg_count ; i++)
{ {
max_length+=args[i]->max_length; max_length+=args[i]->max_length;
set_charset(charset(), coercibility, if (set_charset(charset(), coercibility,
args[i]->charset(), args[i]->coercibility); args[i]->charset(), args[i]->coercibility))
{
my_error(ER_WRONG_ARGUMENTS,MYF(0),func_name());
break;
}
} }
if (max_length > MAX_BLOB_WIDTH) if (max_length > MAX_BLOB_WIDTH)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment