-
selsta
KnifeOfPi: I just tested the branch with `./monero-wallet-cli --log-level 0 --restore-deterministic-wallet --daemon-address node.xmr.to:18081`
-
selsta
and my logs are correct, so they are doing something wrong at some point (sends wrong logs maybe)
-
selsta
can you ask them to use a different file hoster?
-
KnifeOfPi_
he’s sent the file like 3 different ways
-
KnifeOfPi_
every time 73kb
-
moneromooo
Maybe his disk is shot ^_^
-
moneromooo
That wouldn't explain the seemingly nice extra logs though. Shrug.
-
iDunk
Or RAM.
-
moneromooo
The log timestamp seems to be earlier than the binary timestamp, assuming it's not a locale display thing.
-
selsta
timestamp seem to match, the wallet got created at minute 56 and the logs also begin at 56
-
selsta
the binary timestamp are the same as mine
-
moneromooo
nvm, I misread 8/6.
-
selsta
did you see this wallet2.cpp:2501 logging line before?
-
moneromooo
I don't remember.
-
selsta
KnifeOfPi: can the person move out his coins?
-
KnifeOfPi_
i will ask
-
KnifeOfPi_
no he can not move out his coins.
-
KnifeOfPi_
his wallet shows only 0.084 XMR balance.
-
iDunk
Which makes sense if you count the digits 84.,201,250,000
-
moneromooo
The std::to_string probably butchers the number (there's a broken patch on github to deal with that).
-
moneromooo
So the code to insert . sees , as another character.
-
selsta
KnifeOfPi: and what is his balance supposed to be?
-
iDunk
src/wallet/wallet2.cpp:2625Received money: 0.084201250000,
-
KnifeOfPi_
it should be 128 XMR I believe
-
KnifeOfPi_
none of the transactions he sent ever actually showed up on the chain
-
KnifeOfPi_
but they were still deducted from his balance
-
iDunk
I think someone is pulling your leg with fake txids.
-
moneromooo
I guess you could ask to run with windows' equivalent of all the LC_* unset.
-
KnifeOfPi_
this guy seems very non technical
-
moneromooo
Enough to add new logs :)
-
selsta
well something is weird here, the logs even say that he is running the correct commit according to the version number
-
selsta
moneromooo: would the logs show up if --log-level 0 is missing?
-
moneromooo
Only the first line showing log levels.
-
selsta
e.g. the logs say 2020-08-07 22:13:03.603 16984 ERROR wallet.wallet2 src/wallet/wallet2.cpp:1889
-
selsta
1889 is a line we added
-
selsta
but the logs don’t match what it is supposed to say
-
moneromooo
selsta: what platform are you running on ?
-
selsta
Mac, I was going by the logs the KnifeOfPi sent
-
selsta
that*
-
selsta
when I run the branch it looks correct on my mac, it says the `process_new_transaction: money received in` line
-
moneromooo
iDunk: if we can get the locale settings that person runs with, could you try running with those to see if you get weird things ?
-
KnifeOfPi_
he is from pakistan
-
moneromooo
(on windows)
-
moneromooo
(or anyone else who runs windows)
-
iDunk
Sure (but I still think someone in Pakistan is trying to get something for nothing).
-
moneromooo
Hmm. The code printing txids doesn't use std::to_string, it indexes direcrtly into a hex char string, so unlikely to be a locale thing.
-
moneromooo
is that enough info (from pakistan) for windows ?
-
moneromooo
They've got at least two languages there IIRC, pashtun and.. arabic I think ?
-
moneromooo
Wikipedia gives a few more.
-
iDunk
-
iDunk
Pashto (Pakistan) numbers and currency formats
i.imgur.com/8VQpPBh.png i.imgur.com/qeaFP4N.png
-
iDunk
^ default settings.
-
moneromooo
The , matches what we see. So, are txid mangled ? :)
-
selsta
the line numbers seem totally wrong in the logs
-
selsta
even for known things like the "Received money with tx" line
-
iDunk
L473 looks like an editing fail.
-
moneromooo
Heh, odd one indeed.
-
moneromooo
Could be two threads logging at once maybe.
-
moneromooo
Though I don't see a partial remainder of such a line.
-
selsta
iDunk: btw does the binary produce expected logs on your system?
-
iDunk
I might as well run it...
-
iDunk
(Not to mention that block heights in the last four lines are ~10 days into the future)
-
KnifeOfPi
lol what the hell
-
KnifeOfPi
why would he go through effort of faking logs
-
iDunk
Social atomic swaps :)
-
selsta
I don’t understand what’s going on lol
-
selsta
but the weird logs + changing file size is suspicious
-
selsta
though not sure what he is trying to achieve
-
KnifeOfPi
could it be caused by a localization issue...?
-
iDunk
Here's what line numbers belong to what category and what they log
paste.debian.net/hidden/b7971390
-
selsta
iDunk: that’s what it looks like on my system too
-
selsta
this was with my bin?
-
iDunk
With mine.
-
selsta
KnifeOfPi: one thing he can try is to restore a non broken wallet and then post the logs
-
selsta
if the resulting logs are also broken then I would guess some localization issue
-
iDunk
Or an attempted scam.
-
selsta
also possible :D
-
iDunk
-
iDunk
XMC->XMR swap :D
-
iDunk
Block heights in the future make sense now.
-
selsta
still don’t fully get it
-
selsta
tried to scam?
-
iDunk
Someone is trying to pass his XMC for XMR.
-
selsta
you think the logs are faked?
-
iDunk
Yes.
-
selsta
the whole story was suspicious anyway lol
-
iDunk
I linked the 128 XMR txid to Monero Classic.
-
selsta
yea I saw it
-
selsta
soo they used XMC to create the screenshot?
-
selsta
and then merged the logs
-
iDunk
Probably.
-
selsta
doctored logs would explain why they went from 100KB -> 70KB lol
-
selsta
-
iDunk
Mystery solved :)
-
KnifeOfPi
w a t
-
KnifeOfPi
XMC???
-
KnifeOfPi
its fucking monero classic
-
KnifeOfPi
is there any way it could have been done by mistake?
-
selsta
KnifeOfPi: no
-
iDunk
No, the logs were doctored. You only need to take a look at other lines in the wallet2.cpp selsta linked.
-
KnifeOfPi
fucking hell man
-
iDunk
"FUCKXMR" wallet name made me look at other forks' explorers :)
-
KnifeOfPi
i had expected he would just be mad
-
KnifeOfPi
because we had him use cli and he was acting like a very non technical user
-
KnifeOfPi
this idiot wasted many hours of our time
-
iDunk
I think he was trying to get a "refund". That's why I keep calling it a swap.
-
KnifeOfPi
yeah, that's what I expected
-
KnifeOfPi
he seemed suspicious from the beginning because he complained of 128xmr being 1 year of his income
-
KnifeOfPi
so how the hell did he buy it in the first place
-
KnifeOfPi
hey, iDunk, did he just use straight up XMC logs, or did he try to splice in some stuff from the proper wallet log?
-
selsta
he mixed them
-
vikrants
Wow
-
vikrants
So someone who knows how monero works well
-
KnifeOfPi
i almost want to make a post on /r/monero to expose this guy
-
KnifeOfPi
he wasted so many hours of time of everybody on the cake wallet team, and you guys as well
-
KnifeOfPi
> ok i wont be able to do it again please... i have tried thousand times and then my seed works
-
KnifeOfPi
he would usually take 30 minutes to one hour to respond with stuff
-
KnifeOfPi
guess because he was faking shit
-
endogenic
welcome to my world
-
vikrants
endogenic: this happened to u before?
-
endogenic
running a high traffic wallet and having to support lots of different user cases? being asked to do things for people on occasionally bad info who didnt want to contribute back? lol welcome to cryptocurrency
-
endogenic
risk
-
endogenic
finance
-
vikrants
Yeah
-
xmr-pr
moneromooo-monero opened pull request #6752: wallet2: fix setting tx keys when another is already set
-
xmr-pr
-
cohcho
moneromooo: I can't sync monerod from scratch. Using recent master build. I's beind stuck with calculating tx weight. 2020-08-08 17:23:16.881 E get_pruned_transaction_weight does not support older range proof types
-
cohcho
Is it interesting bug or something trivial to avoid?
-
cohcho
If you're going to release soon then this bug will pop up later. it's likely unexpected regression.
-
moneromooo
I thought I'd fixed that. Let me check github...
-
moneromooo
What height block is it breaking on ?
-
cohcho
2020-08-08 17:32:03.354 I transaction is too heavy: 18446744073709551615 bytes, maximum weight: 149400
-
cohcho
2020-08-08 17:32:03.354 E Transaction verification failed: <bf98829d8644e7a395a8972543ee7313290ecf801330ab88453fbda9f7b957ec>
-
cohcho
Thas' from block 1685619
-
cohcho
I can locally dig into it if it's important and not trivial problem.
-
moneromooo
I'd like to know where it's coming from (call stack).
-
cohcho
Ok. I'll try to debug it personally since it require this level of details.
-
moneromooo
-
moneromooo
I was asking for prunable from when BPs started, but should do so only when they become mandatory.
-
moneromooo
Alternatively, making get_pruned_transaction_weight support Borromean proofs would be better.
-
cohcho
Why this error didn't pop up earlier during your own testing?
-
moneromooo
I doubt I made a full sync.
-
cohcho
Are there no any relevant regression tests in monero codebase?
-
moneromooo
Not for this.
-
moneromooo
More tests are welcome if you feel like adding some. Though tests syncing through actual live data... not sure.
-
cohcho
Patch works: it's syncing. Hope my report was useful.
-
cohcho
Thanks for fast response.
-
moneromooo
It was. Thanks.
-
dEBRUYNE
UkoeHB_: Thanks, I may have conflated two different proofs
-
xmr-pr
moneromooo-monero opened pull request #6753: cryptonote_protocol: don't synced pruned blocks before v9
-
xmr-pr