Fix for bug #8430 (CEILING returns incorrect result)

1 files changed, 19 insertions, 7 deletions

diff --git a/sql/item_func.cc b/sql/item_func.cc
index 37e59fa89d3..2f00c06f68a 100644
--- a/sql/item_func.cc
+++ b/sql/item_func.cc
@@ -1629,13 +1629,25 @@ void Item_func_int_val::find_num_type()
 
 longlong Item_func_ceiling::int_op()
 {
-  /*
-    the volatile's for BUG #3051 to calm optimizer down (because of gcc's
-    bug)
-  */
-  volatile double value= args[0]->val_real();
-  null_value= args[0]->null_value;
-  return (longlong) ceil(value);
+  longlong result;
+  switch (args[0]->result_type()) {
+  case INT_RESULT:
+    result= args[0]->val_int();
+    null_value= args[0]->null_value;
+    break;
+  case DECIMAL_RESULT:
+  {
+    my_decimal dec_buf, *dec;
+    if ((dec= decimal_op(&dec_buf)))
+      my_decimal2int(E_DEC_FATAL_ERROR, dec, unsigned_flag, &result);
+    else
+      result= 0;
+    break;
+  }
+  default:
+    result= (longlong)real_op();
+  };
+  return result;
 }