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

Vanity Address Tab addition #3

Closed
wants to merge 1 commit into from
Closed

Vanity Address Tab addition #3

wants to merge 1 commit into from

Conversation

n1bor
Copy link

@n1bor n1bor commented Sep 27, 2012

When calc a vanity address there is a need for the user to
multiply 2 private addresses together. This needs to be done
offline.

When calc a vanity address there is a need for the user to
multiply 2 private addresses together. This needs to be done
offline.
@ghost
Copy link

ghost commented Sep 27, 2012

I'm glad to see someone went ahead and did this. I reviewed the code but have not yet tested it. I'm curious if there's a reason why you chose multiply method over addition? I know keyconv uses the addition method but since you've no doubt tested this it would be apparent that either method works with vanitygen output.

@n1bor
Copy link
Author

n1bor commented Sep 28, 2012

I did addition originally, and this is what the vanitygen.exe -P does (I know this as tested it!). But oclvanityminer.exe needs multiplication (I am told - https://bitcointalk.org/index.php?topic=84569.msg1226994#msg1226994). But there is some confusion in my mind as when I tested normal generation, -P and oclvanityminer.exe all seemed to search at the same speed.

I will add 2 additional outputs so that it displayed the added and multiplied so users can take their pick. Will update patch latter, or tomorrow.

@n1bor n1bor closed this Sep 28, 2012
@ghost
Copy link

ghost commented Sep 29, 2012

I hadn't thought of it but it's a good idea to support both methods. That way if someone wants to create a vanity address for a friend or other third party they work it out using bitaddress as a trusted tool. I didn't know vanityminer used multiply but I haven't looked into it.

@ghost
Copy link

ghost commented Sep 29, 2012

I had an idea for using this as split key paper wallets as discussed here,
https://bitcointalk.org/index.php?topic=114074.0
I think it's a good idea and so I thought I'd mention it would be good if the arrangement you use would be suitable for printing with QR codes just like other tabs. I could see this helping people solve the issue of securing systems for paper wallet creation.

UdjinM6 pushed a commit to UdjinM6/paper.dash.org that referenced this pull request Aug 14, 2016
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

Successfully merging this pull request may close these issues.

1 participant