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

`show_transfers out` reports incorrect values #3983

Closed
moneroexamples opened this Issue Jun 11, 2018 · 5 comments

Comments

3 participants
@moneroexamples
Contributor

moneroexamples commented Jun 11, 2018

I'm using stagenet now and monero version commit 25e7a7d96f2e46821d6613e6d2012b367dd70f38.

seed for the stagenet wallet.

gels lair teeming cease nanny utility inexact leisure
civilian emerge zippers skew gasp enjoy fugitive nanny
candy nuance muppet scrub uneven yard ulcers unquoted yard

This corresponds to default wallet of open monero at stagenet (http://172.104.45.209:81/) and am pretty sure that outgoing values shown there are correct. But monero-wallet-cli shows very strange numbers of outgoing transfers (notice values of about 18446685.05933599319) for the same wallet.

[wallet 57Hx8Q]: show_transfers out
   91167    out       2018-06-06 18446657.97436341319 a783198e169e0fb8eee990cf41f40145497d0cee21abcaa0367c0dae5bc8279b 0000000000000000 0.063408800000  0 - 
   93138    out       2018-06-09      67.917661481583 bcc39dbc076818ee6e61252b7baeb83ed150678ba3eb93f020095ac0de368d2d 0000000000000000 0.017007620000  0 - 
   93643    out       2018-06-09 18446685.05933599319 1640236fe817f83b0bd2082c655d152be390e4f78e41db5b935bb8a53249fe8c 0000000000000000 0.067964960000  0 - 
   94433    out      07:49:44 PM 18445744.17672471161 87617d03aeab903262dacf764eb845b13a3775860acec23f2af595af43b8c295 0000000000000000 0.063015160000  0 - 

But the wallet balance is ok, so it seems that the values are incorrectly reported only for outgoing txs. These outgoing txs were not made using monero-wallet-cli.

@stoffu

This comment has been minimized.

Show comment
Hide comment
@stoffu

stoffu Jun 11, 2018

Contributor

Yes, this seems like a bug.

Contributor

stoffu commented Jun 11, 2018

Yes, this seems like a bug.

@moneromooo-monero

This comment has been minimized.

Show comment
Hide comment
@moneromooo-monero

moneromooo-monero Jun 11, 2018

Contributor

#3985

Whatever code is doing this is adding the pubkey twice, fingerprinting it in the process.

Contributor

moneromooo-monero commented Jun 11, 2018

#3985

Whatever code is doing this is adding the pubkey twice, fingerprinting it in the process.

@stoffu

This comment has been minimized.

Show comment
Hide comment
@stoffu

stoffu Jun 11, 2018

Contributor

@moneroexamples Oh, I noticed just now that some of these txes have additional tx keys, which I find strange because those txes have only two outputs (destination & change), so additional tx keys shouldn't be necessary. What exactly did you do to create these txes?

Contributor

stoffu commented Jun 11, 2018

@moneroexamples Oh, I noticed just now that some of these txes have additional tx keys, which I find strange because those txes have only two outputs (destination & change), so additional tx keys shouldn't be necessary. What exactly did you do to create these txes?

@moneromooo-monero

This comment has been minimized.

Show comment
Hide comment
@moneromooo-monero

moneromooo-monero Jun 11, 2018

Contributor

#3985 was off, fixed now. Building on master now to double check as I tested on my work branch where the wallet code has a lot of changes in that area. checked.

Contributor

moneromooo-monero commented Jun 11, 2018

#3985 was off, fixed now. Building on master now to double check as I tested on my work branch where the wallet code has a lot of changes in that area. checked.

@moneroexamples

This comment has been minimized.

Show comment
Hide comment
@moneroexamples

moneroexamples Jun 11, 2018

Contributor

@moneromooo-monero

Oh, havn't noticed extra tx keys. thanks. So this must be bug in open monero then.

edit. Just found it. It was open monero adding second key. Already fixed.

Thanks again.

Contributor

moneroexamples commented Jun 11, 2018

@moneromooo-monero

Oh, havn't noticed extra tx keys. thanks. So this must be bug in open monero then.

edit. Just found it. It was open monero adding second key. Already fixed.

Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment