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

"checksig" test always pass even when it should not #1

Open
binaryFate opened this issue Nov 27, 2015 · 3 comments
Open

"checksig" test always pass even when it should not #1

binaryFate opened this issue Nov 27, 2015 · 3 comments

Comments

@binaryFate
Copy link

The "checksig" test in mininero.py always pass, even if the private key is arbitrarily changed.

For instance, when changing the private key before signing, like this on line 700:
sir, sic = generate_signature(prefix, pub,
99999999999999999999999999999999999999999999999999999999999999999999999999999)

check_signature() still returns True.

@ghost
Copy link

ghost commented Nov 27, 2015

Dear binaryfate, I haven't updated this code in a while - recent version of my code (which should be more compatible with actual Monero) is at https://github.com/ShenNoether/MiniNero - sometime in the near future, I will most likely replace the code here with that, however I'm quite busy with the Ring CT stuff, as well as some unrelated projects, so it might be a month or two (iirc the original MiniNero code was just for the purpose of seeing how the ring sigs worked, and I didn't bother with the basic stuff)

@binaryFate
Copy link
Author

Yes, I had a look at your more recent version and it's indeed much cleaner. Very appreciated!
I must say I miss the tests there though, many tests are in old version and not in new one. They're very useful in learning how to do stuff.

@ghost
Copy link

ghost commented Nov 28, 2015

Sure, once I get the ringCT stuff finished, I can add those tests into
the newer version.

On 11/27/2015 12:40 PM, binaryFate wrote:

Yes, I had a look at your more recent version and it's indeed much
cleaner. Very appreciated!
I must say I miss the tests there though, many tests are in old
version and not in new one. They're very useful in learning how to do
stuff.


Reply to this email directly or view it on GitHub
#1 (comment).

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