Uses insecure library for generating Bitcoin addresses #2

Closed
qirtaiba opened this Issue Jun 14, 2017 · 6 comments

Comments

Projects
None yet
3 participants
@qirtaiba

A couple of people have experienced their Bitcoins vanishing when using addresses generated by the PHPCoinAddress.php library. See https://github.com/zamgo/PHPCoinAddress/issues/3. This doesn't inspire confidence. Possible that someone is generating the same private keys and sweeping up the coins?

@coinables

This comment has been minimized.

Show comment
Hide comment
@coinables

coinables Jun 15, 2017

Owner

ACK - Need to find a stand alone CPRNG PHP lib that can generate key pairs.

Owner

coinables commented Jun 15, 2017

ACK - Need to find a stand alone CPRNG PHP lib that can generate key pairs.

@nelsonmurciia

This comment has been minimized.

Show comment
Hide comment
@nelsonmurciia

nelsonmurciia Jul 7, 2017

Deberías borrar este proyecto de Github, todo el dinero se va a esta dirección https://blockchain.info/es/address/1JCuJXsP6PaVrGBk3uv7DecRC27GGkwFwE

nelsonmurciia commented Jul 7, 2017

Deberías borrar este proyecto de Github, todo el dinero se va a esta dirección https://blockchain.info/es/address/1JCuJXsP6PaVrGBk3uv7DecRC27GGkwFwE

@qirtaiba

This comment has been minimized.

Show comment
Hide comment
@qirtaiba

qirtaiba Jul 7, 2017

I'm going to post a pull request for the patch that fixed it for me.

qirtaiba commented Jul 7, 2017

I'm going to post a pull request for the patch that fixed it for me.

@coinables

This comment has been minimized.

Show comment
Hide comment
@coinables

coinables Jul 7, 2017

Owner

@qirtaiba did you find a decent php library? I'm not working on this project anymore, but if someone is stealing funds based on PHPCoinAddress library then I'm going to delete the repo like @nelsonmurciia suggests.

Owner

coinables commented Jul 7, 2017

@qirtaiba did you find a decent php library? I'm not working on this project anymore, but if someone is stealing funds based on PHPCoinAddress library then I'm going to delete the repo like @nelsonmurciia suggests.

@qirtaiba

This comment has been minimized.

Show comment
Hide comment

qirtaiba commented Jul 7, 2017

See #4.

@coinables

This comment has been minimized.

Show comment
Hide comment
@coinables

coinables Jul 7, 2017

Owner

This repo is now retired. Thank you for your contributions @qirtaiba @nelsonmurciia

Owner

coinables commented Jul 7, 2017

This repo is now retired. Thank you for your contributions @qirtaiba @nelsonmurciia

@coinables coinables closed this Jul 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment