-
needbrrrrrrr90
So from memory to confirm, when a user sends to a particular address, they generate a random nonce that gets combined with the address to determine where to send their funds
-
needbrrrrrrr90
And choosing the same number and sending to it twice will generate outs with the same key image, rendering only one spend able
-
needbrrrrrrr90
If I'm missing a piece please fill it in, my memory says that there's an infinitesimal chance that sending to a user will cause an unredeemable spend
-
UkoeHB_
that's right, reusing transaction public keys can cause problems but is very very unlikely if you generate them randomly each new tx
-
hyc
I would have assumed the chance of a single account generating duplicate key images was zero
-
hyc
and the only chance of collision was across multiple accounts
-
needbrrrrrrr90
-
needbrrrrrrr90
There's the context
-
hyc
ah yes, all-zero spendkey
-
hyc
some crypto mechs just error-out on an all-zero input
-
gingeropolous
precomputed hashes aren't working on master
-
dengjiong
Hello, I want to generate keyimage, but I can't find the Java implementation
-
UkoeHB_
needbrrrrrrr90: I saw the reddit thread. The problem is the private spend key is all zeros, which doesn't work mathematically (can't multiply an elliptic curve point by zero, it just doesn't make sense), and I guess the code is built to handle that kind of issue.
-
UkoeHB_
isn't* built
-
gingeropolous
yeah, confirmed on second box and build "2020-05-09 13:11:28.894 E Failed to load hashes - unexpected data size
-
gingeropolous
"
-
gingeropolous
gimme mah hashes!
-
moneromooo
Is that with selsta's recent patch for 0.15.1.0 ?
-
gingeropolous
its master
-
moneromooo
Fine, I get it, I'll look.
-
selsta
yes the hashes haven’t been updated in a while, they work with my v0.15.1.0 patch
-
gingeropolous
so if its not pulled in yet, then i didn't compile it
-
selsta
(that I have to update to v0.16...)
-
gingeropolous
lemme hunt down that patch
-
moneromooo
wtf, git pull trying (and failing) to pull some ethereum stuff
-
selsta
yes trezor submodules pulls some ethereum stuff
-
selsta
has for a while now
-
moneromooo
Succeeded now. That is dodgy.
-
moneromooo
So it looks like it does not have selsta's patch yet. Does it work if you apply it ? :)
-
iDunk
That's why 0.15.0.1 was released, wasn't it ?
-
iDunk
Wrong hash step (256 -> 512) in 0.15.0.0, IIRC.
-
moneromooo
You mean master has had a bad hoh file all this time, but gingeropolous only spotted it now ?
-
iDunk
Could be. Any way to tell if the hashes in master are 256-step or 512 ?
-
moneromooo
The header has the number of hashes.
-
moneromooo
They're not logged though. Or check the date of the commit that modified it last.
-
moneromooo
Says 0.15.0.0.
-
moneromooo
So you're likely correct and we can just ignore this, since selsta's commit probably uses the right version.
-
iDunk
DE1D = 1DDE = 7646 x 256 = 1957376 ?
-
iDunk
Sounds plausible ?
-
selsta
yes hashes master have been broken for a while
-
iDunk
release-v0.15 has 030F = 0F03 = 3843 x 512 = 1967616.
-
moneromooo
Sounds very plausible.
-
selsta
UkoeHB_: there is this now
monero-project/monero #6296
-
moneromooo
Is that causing trouble ? It's just a "tell the user to stop being an idiot", really to placate some spammy dude.
-
selsta
Someone entered `abbey` 25 times as their seed and asked if/why things are broken.
-
moneromooo
Was that given by some software, or is that someone being an idiot on purpose ?
-
selsta
> It was at this point he told me it wouldn't be the first time the seed had changed hands, and that someone had given him the seed at a Monero Meetup.
-
selsta
maybe someone did this as a joke to confuse people?
-
selsta
-
moneromooo
Using a seed from someone else, clearly trolling.
-
moneromooo
Anyway, from time to time people pop up thinking it's a good idea to use their own words and not understanding that not veru word string maps to a valid seed.
-
moneromooo
AFAIK nothing in monero tells people this is a good idea.
-
moneromooo
*every
-
selsta
maybe they are conusing it with a brainwallet?
-
moneromooo
I *think* this is why some people report a changed seed (software reduces the scalar).
-
moneromooo
Brainwallet tells you to come up with your own words ?
-
selsta
yes
-
moneromooo
Unfortunate. Then maybe it's not idiotic, just bad assumptions.
-
selsta
there is this correct horse battery staple xkcd and a lot of people used it as their brainwallet
-
selsta
so now there are bots that instantly transfer out transactions of this brainwallet
-
moneromooo
Come to think of it, if people want their own woeds, we could just hash this with a KDF to a seed.
-
moneromooo
OK, back to idiotic then.
-
moneromooo
Actually, my address generator does that already. I wonder if that helped confuse people :/
-
selsta
I think the idea can be neat if you can memorize your seed.
-
selsta
you mean the custom entropy thing?
-
luigi1111w
abbey is word 1
-
luigi1111w
aka zero
-
luigi1111w
someone trolled that dude I guess
-
selsta
lol
-
rbrunner
I think it's about the 4th time this "abbey abbey abbey ..." wallet and its strange behaviour comes up on Reddit. I guess it's possible to come with this for curious people all on their own and then check that seed
-
rbrunner
moneromooo, will your patch now make this wallet un-restorable?
-
moneromooo
No.
-
moneromooo
(just for the json restore path)
-
moneromooo
Guess we could warn when we get an invalid words list.
-
moneromooo
Doesn't hlep people who use a seed someone else made though.
-
rbrunner
By the way, is this wallet unique? Or are there other ones where every out transaction produces the same key image, say because some modulo operation makes it work like these zero keys?
-
rbrunner
(Somebody asked this on Reddit, and I got interested. My guess is "yes, it's unique", but I am not sure.)
-
luigi1111w
I don't really get it
-
rbrunner
Get what, my question?
-
luigi1111w
seems like a bug
-
luigi1111w
a zeroed private key shouldn't result in identity key images
-
luigi1111w
only a zeroed output private key should
-
rbrunner
Well, maybe I assume this to be much too simple, but isn't there some op where "times 0" (from the keys) gives 0, and from there on it goes wrong?
-
rbrunner
Or does EC multiplication not work this way?
-
luigi1111w
it does, yes
-
luigi1111w
but the private key is only used by addition
-
luigi1111w
anyway plenty of copies
-
luigi1111w
large hemlock royal beware woozy gather slackens vain nanny tumbling gained identity abbey abbey abbey abbey abbey abbey abbey abbey abbey justice yearbook annoyed nanny
-
luigi1111w
wade saga knuckle dolphin website pegs ethics angled galaxy seeded pause space abbey abbey abbey abbey abbey abbey abbey abbey abbey upwards withdrawn below saga
-
luigi1111w
I assume if you import them the wallet will reduce to abbey abbey....
-
rbrunner
How did you come up with those seeds?
-
rbrunner
Replacing some words with "abbey" to get a zero key?
-
selsta
more seeds to troll at meetups
-
luigi1111w
the wallet does reduce to abbey, just tested
-
rbrunner
Because the one key derives from the other, right?
-
luigi1111w
rbrunner just double l
-
luigi1111w
there should be like 15 of them
-
rbrunner
"yearbook annoyed nanny" lol
-
luigi1111w
unfortunately all of them will ahve a lot of abbey's later on, so anyone in the know can detect them
-
rbrunner
So it's unique, in a way? There is no key, say "halfway up to the maximum" key that has the same behaviour because some mod operation makes it zero.
-
rbrunner
(Sometimes stupid questions lead to something, so please forgive :)
-
luigi1111w
no of course not, the modulus is a prime number
-
luigi1111w
keys are 0, modulus, 2*modulus, and so on
-
luigi1111w
when the wallet sees a key higher than modulus, it reduces it to the equivalent key in the set
-
rbrunner
I see. So only limited fun.
-
luigi1111w
but I still don't get it
-
luigi1111w
I guess I'll scan that wallet and see
-
rbrunner
See what? It has 7 transactions incoming, one is from me when I tried whether the transactions were doctored, or the wallet is "broken"
-
luigi1111w
yeah I wanna see them
-
luigi1111w
can we tell everyone to send their burn coins there
-
iDunk
There should be an "abbey" command in the wallet for that.
-
rbrunner
iDunk, you made my day
-
iDunk
:)
-
hyc
abbey normal?
-
UkoeHB_
idk if the abbey wallet qualifies as a burn address, be very careful
-
UkoeHB_
I expect someone could patch the code to make it useable
-
luigi1111w
figured it out
-
luigi1111w
wallet thinks it's view only
-
luigi1111w
UkoeHB_ it's definitely not a burn address
-
luigi1111w
thus <luigi1111w> can we tell everyone to send their burn coins there
-
iDunk
lol
-
luigi1111w
-
luigi1111w
go get your free monero
-
UkoeHB_
lol
-
hyc
abbey is a permanent watch-only wallet?
-
UkoeHB_
not permanent, you just have to patch it up
-
UkoeHB_
custom build
-
luigi1111w
yeah, even a non dev (if you can figure out how to compile) could do it
-
hyc
the spendkey is still zero tho
-
luigi1111w
very easy change
-
UkoeHB_
one-time addresses aren't just the spend key, so it works out
-
hyc
ah
-
luigi1111w
the spendkey is only used in an addition context for transactions, that's why it works
-
rbrunner
Almost an anti-climax
-
hyc
yeah, much better as a cursed address
-
rbrunner
Hmmm, Monero Core Team member with "cloud10again" as their Reddit handle? They just made a post explaining the thing with the treatment as output-only wallet ...
-
selsta
luigi
-
iDunk
cloudluigi
-
rbrunner
Oh, ok.
-
dEBRUYNE
<luigi1111w> a zeroed private key shouldn't result in identity key images <= Perhaps someone trolled by sending burnt outputs to the wallet
-
dEBRUYNE
By keeping r constant for numerous transactions
-
luigi1111w
no
-
selsta
loogi pls branch
-
selsta
v0.16 or whatever you like
-
» iDunk votes for release-v0.16
-
luigi1111w
releasing v.16.1111
-
luigi1111w
-
selsta
nice
-
selsta
luigi = monero leader
-
selsta
confirmed by wikipedia^tm