From 601c09e290c76a61f7e95e42da2c732b02574a73 Mon Sep 17 00:00:00 2001
From: Zlatan <578072836@qq.com>
Date: Tue, 13 Jul 2021 14:15:27 +0800
Subject: [PATCH] add a OB_FAIL for ret check (#142)

---
 src/sql/resolver/ddl/ob_ddl_resolver.cpp | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/sql/resolver/ddl/ob_ddl_resolver.cpp b/src/sql/resolver/ddl/ob_ddl_resolver.cpp
index edfbacc02..b75bb6ddc 100644
--- a/src/sql/resolver/ddl/ob_ddl_resolver.cpp
+++ b/src/sql/resolver/ddl/ob_ddl_resolver.cpp
@@ -428,9 +428,12 @@ int ObDDLResolver::resolve_default_value(ParseNode* def_node,
             default_value.set_param_meta();
           } else if (ObFloatType == old_obj.get_type()) {
             float value = 0.0f;
-            old_obj.get_float(value);
-            default_value.set_float(-value);
-            default_value.set_param_meta();
+            if (OB_FAIL(old_obj.get_float(value))) {
+              SQL_RESV_LOG(WARN, "failed to get float value from old_obj", K(ret), K(old_obj));
+            } else {
+              default_value.set_float(-value);
+              default_value.set_param_meta();
+            }
           } else if (ObDoubleType == old_obj.get_type()) {
             double value = 0.0;
             if (OB_FAIL(old_obj.get_double(value))) {