Replace g_assert with error handling

Release builds often define the compile option -DG_DISABLE_ASSERT
for performance optimization. When this option is enabled, the
g_assert(condition) macro is replaced by a no-op by the preprocessor.

Signed-off-by: hyunil park <[email protected]>

Author: songgot

c/src/ml-api-common-tizen-feature-check.c

@@ -45,19 +45,23 @@ static feature_info_s *feature_info = NULL;
 /**
  * @brief Internal function to initialize feature state.
  */
-static void
+static int
 ml_tizen_initialize_feature_state (void)
 {
   int i;
 
   if (feature_info == NULL) {
-    feature_info = g_new0 (feature_info_s, 1);
-    g_assert (feature_info);
+    feature_info = g_try_new0 (feature_info_s, 1);
+    if (feature_info == NULL) {
+      _ml_loge ("Failed to allocate memory for feature_info");
+      return ML_ERROR_OUT_OF_MEMORY;
+    }
 
     g_mutex_init (&feature_info->mutex);
     for (i = 0; i < ML_FEATURE_MAX; i++)
       feature_info->feature_state[i] = NOT_CHECKED_YET;
   }
+  return ML_ERROR_NONE;
 }
 
 /**
@@ -66,7 +70,12 @@ ml_tizen_initialize_feature_state (void)
 int
 _ml_tizen_set_feature_state (ml_feature_e ml_feature, int state)
 {
-  ml_tizen_initialize_feature_state ();
+  int status;
+
+  status = ml_tizen_initialize_feature_state ();
+  if (status != ML_ERROR_NONE)
+    return status;
+
   g_mutex_lock (&feature_info->mutex);
 
   /**
@@ -96,8 +105,11 @@ _ml_tizen_get_feature_enabled (ml_feature_e ml_feature)
 {
   int ret;
   int feature_enabled;
+  int status;
 
-  ml_tizen_initialize_feature_state ();
+  status = ml_tizen_initialize_feature_state ();
+  if (status != ML_ERROR_NONE)
+    return status;
 
   g_mutex_lock (&feature_info->mutex);
   feature_enabled = feature_info->feature_state[ml_feature];

c/src/ml-api-inference-pipeline.c

@@ -1031,7 +1031,7 @@ construct_pipeline_internal (const char *pipeline_description,
   pipeline = gst_parse_launch (description, &err);
   g_free (description);
 
-  if (pipeline == NULL || err) {
+  if (pipeline == NULL || err || !GST_IS_PIPELINE (pipeline)) {
     _ml_error_report
         ("ml_pipeline_construct error: gst_parse_launch cannot parse and launch the given pipeline = [%s]. The error message from gst_parse_launch is '%s'.",
         pipeline_description, (err) ? err->message : "unknown reason");
@@ -1044,12 +1044,16 @@ construct_pipeline_internal (const char *pipeline_description,
     goto failed;
   }
 
-  g_assert (GST_IS_PIPELINE (pipeline));
   pipe_h->element = pipeline;
 
   /* bus and message callback */
   pipe_h->bus = gst_element_get_bus (pipeline);
-  g_assert (pipe_h->bus);
+  if (pipe_h->bus == NULL) {
+    _ml_error_report
+        ("ml_pipeline_construct error: Failed to retrieve bus from the pipeline.");
+    status = ML_ERROR_STREAMS_PIPE;
+    goto failed;
+  }
 
   gst_bus_enable_sync_message_emission (pipe_h->bus);
   pipe_h->signal_msg = g_signal_connect (pipe_h->bus, "sync-message",