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

There is a small problem #36

Open
Panxhang opened this issue May 11, 2018 · 2 comments · May be fixed by #92
Open

There is a small problem #36

Panxhang opened this issue May 11, 2018 · 2 comments · May be fixed by #92

Comments

@Panxhang
Copy link

Hi!In the 4th part,If someone transfers money to himself,his account balance will be double。
if out.CanBeUnlockedWith(address) {
unspentTXs = append(unspentTXs, *tx)
}
There is a “break” missing here!This statement is in the blockchain.go。
Thank you very much for your tutorial and please note this little bug~

@stoneflying
Copy link

stoneflying commented May 19, 2019

Yes, I just found it

@mubarizkyc
Copy link

if from == to {

	log.Println("ERROR: Sender and Recipient cannot be same")
	os.Exit(1)
} you can add this in cli.send  function

@mubarizkyc mubarizkyc linked a pull request Jun 20, 2024 that will close this issue
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 a pull request may close this issue.

3 participants