Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NRE happened during cold-storage workflow #242

Closed
knocte opened this issue Dec 8, 2023 · 0 comments
Closed

NRE happened during cold-storage workflow #242

knocte opened this issue Dec 8, 2023 · 0 comments

Comments

@knocte
Copy link
Member

knocte commented Dec 8, 2023

Created unsigned.json from version 0.5.405.0 (master - 0e07773) on macOS, and it starts with:

{
  "Version": "0.5.405.0",
  "TypeName": "GWallet.Backend.UnsignedTransaction`1[[GWallet.Backend.UtxoCoin.TransactionMetadata, GWallet.Backend, Version=0.5.405.0, Culture=neutral, PublicKeyToken=null]]",
  "Value": {
...

Then signed with version 0.4.403.0 (stable - b72581e) on Windows, and signed.json starts with:


  "Version": "1.0.0.0",
  "TypeName": "GWallet.Backend.SignedTransaction`1[[GWallet.Backend.UtxoCoin.TransactionMetadata, GWallet.Backend, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null]]",
  "Value": {
...

Then when trying to broadcast it in 0.5.405.0 again, it crashed with:

Introduce a file name to load the signed transaction: rebS.json
System.NullReferenceException: Object reference not set to an instance of an object.
   at GWallet.Backend.Account.ImportSignedTransactionFromJson(String jsonOrCompressedJson) in /Users/knocte/Documents/Code/geewalletMASTERclean/src/GWallet.Backend/Account.fs:line 724
   at GWallet.Backend.Account.LoadSignedTransactionFromFile(String filePath) in /Users/knocte/Documents/Code/geewalletMASTERclean/src/GWallet.Backend/Account.fs:line 764
   at Program.BroadcastPayment() in /Users/knocte/Documents/Code/geewalletMASTERclean/src/GWallet.Frontend.Console/Program.fs:line 62
   at Program.PerformOperation(UInt32 numActiveAccounts, UInt32 numHotAccounts) in /Users/knocte/Documents/Code/geewalletMASTERclean/src/GWallet.Frontend.Console/Program.fs:line 398
   at Program.ProgramMainLoop[a]() in /Users/knocte/Documents/Code/geewalletMASTERclean/src/GWallet.Frontend.Console/Program.fs:line 478
   at Program.NormalStartWithNoParameters() in /Users/knocte/Documents/Code/geewalletMASTERclean/src/GWallet.Frontend.Console/Program.fs:line 490

However, trying broadcast with version 0.4.407.0 (stable - 2abf306) it worked.

@knocte knocte closed this as completed in 9058df1 Jan 15, 2024
knocte added a commit that referenced this issue Jan 15, 2024
To bring the fix for bug 242 and the new LTC explorer.

Fixes #242
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant