diff --git a/src/sql/engine/expr/ob_expr_char_length.cpp b/src/sql/engine/expr/ob_expr_char_length.cpp
index bd0474e8443c49e98b3f83680499b25316cbb1b1..62c40a3e7772fa0488dae6ce4b5555e97366bc9a 100644
--- a/src/sql/engine/expr/ob_expr_char_length.cpp
+++ b/src/sql/engine/expr/ob_expr_char_length.cpp
@@ -72,7 +72,7 @@ int ObExprCharLength::eval_char_length(const ObExpr& expr, ObEvalCtx& ctx, ObDat
   ObObjTypeClass in_tc = ob_obj_type_class(expr.args_[0]->datum_meta_.type_);
   if (!ob_is_castable_type_class(in_tc)) {
     res.set_null();
-  } else if (expr.eval_param_value(ctx, arg)) {
+  } else if (OB_FAIL(expr.eval_param_value(ctx, arg))) {
     LOG_WARN("eval arg failed", K(ret));
   } else if (arg->is_null()) {
     res.set_null();