fix(lora): address CodeRabbit OOM and cancellation issues

- LoraViewModel: add ensureActive() in download loop so coroutine
  cancellation is respected during blocking stream reads
- lora_registry: check all rac_strdup results in deep_copy_lora_entry
  to prevent silent partial copies on OOM; use calloc for
  compatible_model_ids array and set count before fill loop so
  free_lora_entry can safely clean up on partial failure

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Siddhesh2377

examples/android/RunAnywhereAI/app/src/main/java/com/runanywhere/runanywhereai/presentation/lora/LoraViewModel.kt

@@ -22,6 +22,7 @@ import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.asStateFlow
 import kotlinx.coroutines.flow.update
+import kotlinx.coroutines.ensureActive
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
 import java.io.File
@@ -211,6 +212,7 @@ class LoraViewModel(application: Application) : AndroidViewModel(application) {
                             val buffer = ByteArray(8192)
                             var bytesRead: Int
                             while (input.read(buffer).also { bytesRead = it } != -1) {
+                                ensureActive()
                                 output.write(buffer, 0, bytesRead)
                                 downloaded += bytesRead
                                 if (totalSize > 0) {

sdk/runanywhere-commons/src/infrastructure/model_management/lora_registry.cpp

@@ -31,22 +31,32 @@ static rac_lora_entry_t* deep_copy_lora_entry(const rac_lora_entry_t* src) {
     copy->download_url = rac_strdup(src->download_url);
     copy->filename = rac_strdup(src->filename);
 
-    // id is required — if it failed to copy, the entry is unusable
-    if (src->id && !copy->id) {
+    // Any non-null source string that failed to copy means OOM — entry is unusable
+    if ((src->id && !copy->id) ||
+        (src->name && !copy->name) ||
+        (src->description && !copy->description) ||
+        (src->download_url && !copy->download_url) ||
+        (src->filename && !copy->filename)) {
         free_lora_entry(copy);
         return nullptr;
     }
 
     if (src->compatible_model_ids && src->compatible_model_count > 0) {
-        copy->compatible_model_ids = static_cast<char**>(malloc(sizeof(char*) * src->compatible_model_count));
+        // Use calloc so unwritten slots are null-safe for free_lora_entry on partial failure
+        copy->compatible_model_ids = static_cast<char**>(calloc(src->compatible_model_count, sizeof(char*)));
         if (!copy->compatible_model_ids) {
             free_lora_entry(copy);
             return nullptr;
         }
+        // Set count before filling so free_lora_entry can clean up on partial failure
+        copy->compatible_model_count = src->compatible_model_count;
         for (size_t i = 0; i < src->compatible_model_count; ++i) {
             copy->compatible_model_ids[i] = rac_strdup(src->compatible_model_ids[i]);
+            if (src->compatible_model_ids[i] && !copy->compatible_model_ids[i]) {
+                free_lora_entry(copy);
+                return nullptr;
+            }
         }
-        copy->compatible_model_count = src->compatible_model_count;
     }
     copy->file_size = src->file_size;
     copy->default_scale = src->default_scale;