Update WalletManager_findWallets

Do not free if the string returned is empty (it crashes the library)
MrCyjaneK · Sep 3, 2024 · 448dd55 · 448dd55
1 parent 811ade0
commit 448dd55
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/impls/monero.dart/lib/monero.dart b/impls/monero.dart/lib/monero.dart
@@ -3595,7 +3595,9 @@ String WalletManager_findWallets(WalletManager wm_ptr, {required String path}) {
  .cast<Utf8>();
  final str = strPtr.toDartString();
  calloc.free(path_);
- MONERO_free(strPtr.cast());
+ if (str.isNotEmpty) {
+ MONERO_free(strPtr.cast());
+ }
  debugEnd?.call('MONERO_WalletManager_findWallets');
  return str;
  } catch (e) {

diff --git a/impls/monero.dart/lib/wownero.dart b/impls/monero.dart/lib/wownero.dart
@@ -3230,7 +3230,9 @@ String WalletManager_findWallets(WalletManager wm_ptr, {required String path}) {
  .cast<Utf8>();
  final str = strPtr.toDartString();
  calloc.free(path_);
- WOWNERO_free(strPtr.cast());
+ if (str.isNotEmpty) {
+ WOWNERO_free(strPtr.cast());
+ }
  debugEnd?.call('WOWNERO_WalletManager_findWallets');
  return str;
  } catch (e) {