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);
 
   /**
@@ -97,7 +106,9 @@ _ml_tizen_get_feature_enabled (ml_feature_e ml_feature)
   int ret;
   int feature_enabled;
 
-  ml_tizen_initialize_feature_state ();
+  ret = ml_tizen_initialize_feature_state ();
+  if (ret != ML_ERROR_NONE)
+    return ret;
 
   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 (!GST_IS_PIPELINE (pipeline) || err) {
     _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",

java/android/nnstreamer/src/main/jni/nnstreamer-native-api.c

@@ -22,7 +22,11 @@ nns_attach_current_thread (pipeline_info_s * pipe_info)
   JavaVM *jvm;
   JavaVMAttachArgs args;
 
-  g_assert (pipe_info);
+  if (!pipe_info) {
+    _ml_loge ("pipe_info is NULL.");
+    return NULL;
+  }
+
   jvm = pipe_info->jvm;
 
   args.version = pipe_info->version;
@@ -48,7 +52,10 @@ nns_get_jni_env (pipeline_info_s * pipe_info)
 {
   JNIEnv *env;
 
-  g_assert (pipe_info);
+  if (!pipe_info) {
+    _ml_loge ("pipe_info is NULL.");
+    return NULL;
+  }
 
   if ((env = pthread_getspecific (pipe_info->jni_env)) == NULL) {
     env = nns_attach_current_thread (pipe_info);

java/android/nnstreamer/src/main/jni/nnstreamer-native-common.c

@@ -554,7 +554,13 @@ nnstreamer_native_get_data_path (void)
   char *data_path = NULL;
 
   G_LOCK (nns_native_lock);
-  g_assert (g_nns_is_initialized);
+
+  if (!g_nns_is_initialized) {
+    _ml_loge ("NNStreamer native library is not initialized.");
+    G_UNLOCK (nns_native_lock);
+    return NULL;
+  }
+
   data_path = g_files_dir;
   G_UNLOCK (nns_native_lock);