-
gingeropolous
take me out to the black, tell em I aint comin back
-
gingeropolous
so we're getting Oxygen Orion?
-
nioc
yep
-
nioc
boring
-
gingeropolous
wonder what thunderosa_ might come up with for that one
-
gingeropolous
for some reason im seeing a bull
-
gingeropolous
is orion a bull?
-
nioc
nope
-
moneromooo
A hunter.
-
nioc
a human hunter
-
nioc
he has a dog
-
moneromooo
A bull dog ?
-
nioc
hunting the bull
-
moneromooo
:o
-
gingeropolous
y am i seeing bull. ah well.
-
» moneromooo does not like
-
nioc
why did we choose to hunt the bull o_O
-
gingeropolous
well, you shoulda sock puppeted the shit outta that poll
-
nioc
my choice was not even in the poll
-
gingeropolous
Oxygen Rasmataz
-
nioc
just realized now about the bull lol
-
gingeropolous
little known constellation
-
nioc
Oumuamua
-
gingeropolous
yeah, monero';s the only cryptocurrency that would name their release after a bull hunter
-
nioc
pronounced Oh mooo ah mooo ah
-
gingeropolous
yeahi liked that one
-
nioc
next time I will pay 1000 moooneros to name the release
-
gingeropolous
ah well, there's always whatevers after oxygen
-
nioc
Flourine
-
gingeropolous
Fluorine Fettuccine
-
nioc
I'll start working on that one
-
gingeropolous
another little known constellation
-
gingeropolous
ah but this friggin tx_extra. what a bowl of fettucine
-
nioc
fetid fettucine
-
nioc
cc
-
gingeropolous
so, we either effectively ensure that all txs are uniform, protecting privacy
-
gingeropolous
or we say "but, we dunno whats gonna happen" and leave it as a fingerprinting mechanism
-
gingeropolous
that deadbeef was funny
-
gingeropolous
36 kb of deadbeef
-
gingeropolous
there's gotta be a way to have both
-
yanmaani
gingeropolous: by being an idiot who leaks info you threaten not only your own privacy but that of others
-
» Inge- mirrors the number of transactions for a week, and publishes the view key
-
Inge-
-
s3rb31
Hello, I am trying to make a transaction with monero-wallet-gui/cli but I consistently get the error: "Error: internal error: Transaction sanity check failed". I tried a few things like reimporting wallet, re-syncing but to no avail. Using Win10 x64 with remote node.
-
moneromooo
A quick way out for now is likely to remove the ~/.shared-ringdb directory, and run the wallet again.
-
moneromooo
HOWEVER:
-
moneromooo
This will mean that node will be able to tell the real output that's being spent.
-
moneromooo
I assume it's not yours. If it's yours (as it should be), it's fine.
-
moneromooo
There's some code to deal with this but it seems to not work :/
-
s3rb31
So you're saying: exit wallet, delete "C:\ProgramData\.shared-ringdb", open wallet and retry?
-
moneromooo
Yes.
-
moneromooo
That database remembers rings used for your outputs, so it can reuse them if necessary.
-
moneromooo
Here, when you send again, it reuses th esame ring. But this ring ain't passing the sanity check, so you get stuck.
-
moneromooo
If the db is removed, it forgets that ring, and will create a new one, which will most likely pass the sanity check.
-
moneromooo
But it means you will have sent two requests to the node: both will contain your real output, and a set of fake outputs.
-
moneromooo
Most likely, the two sets' intersection will be just your output, so the node knows which output you're sending.
-
moneromooo
That's why rings are remembered/reused.
-
s3rb31
Thanks. I will try that in a few minutes. I am not really concerned about privacy with this transaction
-
s3rb31
moneromooo: I tried the steps you explained but I still get the same error.
-
moneromooo
Alright. Now that's unexpected...
-
moneromooo
If you run with --log-level 2, you should see what exactly fails in the log.
-
moneromooo
Actually --log-level 1 is enough for this.
-
moneromooo
Or it could be the node you're using is sending dud data. I'll assume not for now.
-
s3rb31
I am using node.moneroworld.com:18089
-
selsta
s3rb31: try node.xmr.to:18081 or node.supportxmr.com:18081
-
selsta
node.moneroworld.com is a node aggregator so bad nodes might be included
-
s3rb31
I got the log with log-level 1 and it shows some errors but I don't know what that means to be honest.
-
s3rb31
-
s3rb31
I'll try the other nodes now
-
moneromooo
The interesting part: median offset index is too low (median is 128421 out of total 20690220offsets)
-
moneromooo
That's *way* too low..
-
moneromooo
Is your wallet synced ?
-
moneromooo
ie, type "status" in it.
-
s3rb31
yes it was
-
moneromooo
Oooh, I think I might know.
-
» moneromooo goes read code
-
moneromooo
It's assuming rct here, but you're asking for pre-rct outputs. So it's comparing apples and oranges.
-
moneromooo
So it's a bug in the monero wallet. It needs to skip the rct sanity check for pre-rct outputs.
-
moneromooo
Foir now, you can find the line in wallet2.cpp which has:
-
moneromooo
if (tx_sanity_check(unique.first, unique.second, rct_offsets.empty() ? 0 : rct_offsets.back()))
-
moneromooo
And comment it out.
-
moneromooo
So it reads:
-
moneromooo
// if (tx_sanity_check(unique.first, unique.second, rct_offsets.empty() ? 0 : rct_offsets.back()))
-
moneromooo
Then you should be able to send thoes old outputs.
-
moneromooo
selsta: a fix for this is something that we'd want in the release.
-
selsta
sounds good
-
s3rb31
Okay, I'll give it a try. Very nice, thanks
-
moneromooo
Alternatively, I might have a real patch soon.
-
moneromooo
-
moneromooo
That should fix it properly. It would be nice if you could confirm whether if fixes your partiuclar case.
-
moneromooo
s3rb31: ^
-
s3rb31
Yeah thanks, will do.
-
monerod[m]
Hm new qiesrd thing happening to me. THe XMR address that im inputing in the GUI is not the same that is shown on the ledger nano S when it is asking to confirm the TX
-
monerod[m]
never seen that before.
-
monerod[m]
Hmm.. but just found this site. and when i insert the address and "check address"
-
monerod[m]
the one shown on the ledger is in the "Standard XMR" field..
-
monerod[m]
-
moneromooo
Is it mostly the same ? ie, most of the middle of it.
-
moneromooo
If so, it's probably an integrated address, which is the address with a payment id appended, and a different prefix.
-
monerod[m]
yeah true they are
-
moneromooo
Would be nice for the ledger to show the integrated address instead. Maybe in the future.
-
monerod[m]
indeed
-
monerod[m]
thanks bro
-
Inge-
<3 subaddresses
-
Inge-
they show up correctly
-
s3rb31
moneromooo: Got it compiling after a little fight with the makefile xD, will report about the patch once it's done
-
moneromooo
Cool, thanks.
-
s3rb31
moneromooo: Okay, I tried with your patch applied but it gives me the same error.
-
s3rb31
-
s3rb31
I even added some log output to make sure the binary reflects the code changes
-
s3rb31
-
moneromooo
Hmm. Thanks. /me looks at the code again
-
moneromooo
Ah, I think I know. rct here is we're making a rct tx, but it can include pre rct outputs...
-
moneromooo
And we don't know which outputs are rct or not at this point.
-
moneromooo
I guess I can disable the check if we have at least one non rct output.
-
moneromooo
Apply
paste.debian.net/hidden/4147fe0a on top of the existing patch.
-
moneromooo
This skips the rct check even on rct txes if they include at least one pre-rct output.
-
s3rb31
Alright, will try that
-
s3rb31
Okay with the new patch the tx_sanity_check errors disappeared but it still runs into the exception
-
s3rb31
-
Mochi101
because you're exceptional s3rb31
-
s3rb31
why?
-
moneromooo
Oh right, it defaults to the exception.
-
moneromooo
-
s3rb31
Alright, thanks
-
s3rb31
Nice, it worked.
-
s3rb31
moneromooo: Big thanks :D
-
moneromooo
Thanks for testing :)
-
s3rb31
no problem, cya