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.

Sign up for GitHub

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

Jump to bottom

Race condition caused segfault #3

Closed
halturin opened this issue Apr 9, 2018 · 1 comment
Closed

Race condition caused segfault #3

halturin opened this issue Apr 9, 2018 · 1 comment
Assignees
@halturin
Labels
bug

Comments

@halturin
Copy link
Collaborator

halturin commented Apr 9, 2018

Here is details:

[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x5639bf]

goroutine 39 [running]:
example/vendor/github.com/halturin/ergonode/dist.(*NodeDesc).GetRemoteName(...)
    /home/taras/devel/demo/example/vendor/github.com/halturin/ergonode/dist/dist.go:313
example/vendor/github.com/halturin/ergonode.(*Node).run.func2(0xc000104660, 0x5ee6e0, 0xc000114040, 0xc0000ce000, 0xc000104600)
    /home/taras/devel/demo/example/vendor/github.com/halturin/ergonode/ergonode.go:277 +0x17f
created by example/vendor/github.com/halturin/ergonode.(*Node).run
    /home/taras/devel/demo/example/vendor/github.com/halturin/ergonode/ergonode.go:266 +0x10c

these two goroutines in race condition after the link has been closed.

image

getting remote name has no checking for the 'nil' value

func (nd *NodeDesc) GetRemoteName() etf.Atom {
	return etf.Atom(nd.remote.Name)
}
@halturin halturin self-assigned this Apr 9, 2018
@halturin halturin added the bug label Apr 9, 2018
@halturin
Copy link
Collaborator Author

is not valid anymore

halturin added a commit that referenced this issue Apr 6, 2021
@halturin
# This is a combination of 66 commits.
3f05edb 
# This is the 1st commit message:

osdep

# This is the commit message #2:

Fix Windows support (#30)

* Fix Windows support

* Updates for the tests related to the timing conditions due to specifics of the testing environment
# This is the commit message #3:

Update README.md
# This is the commit message #4:

sync

# This is the commit message #5:

sync

# This is the commit message #6:

fix TermIntoStruct. new TermMapIntoStruct

# This is the commit message #7:

sync

# This is the commit message #8:

sync

# This is the commit message #9:

sync

# This is the commit message #10:

sync

# This is the commit message #11:

sync

# This is the commit message #12:

sync

# This is the commit message #13:

sync

# This is the commit message #14:

sync

# This is the commit message #15:

sync

# This is the commit message #16:

sync

# This is the commit message #17:

sync

# This is the commit message #18:

sync

# This is the commit message #19:

sync

# This is the commit message #20:

sync

# This is the commit message #21:

sync

# This is the commit message #22:

sync

# This is the commit message #23:

sync

# This is the commit message #24:

sync

# This is the commit message #25:

sync

# This is the commit message #26:

sync

# This is the commit message #27:

sync

# This is the commit message #28:

sync

# This is the commit message #29:

sync

# This is the commit message #30:

sync

# This is the commit message #31:

sync

# This is the commit message #32:

sync

# This is the commit message #33:

sync

# This is the commit message #34:

sync

# This is the commit message #35:

sync

# This is the commit message #36:

sync

# This is the commit message #37:

sync

# This is the commit message #38:

sync

# This is the commit message #39:

sync

# This is the commit message #40:

sync

# This is the commit message #41:

sync

# This is the commit message #42:

sync

# This is the commit message #43:

sync

# This is the commit message #44:

sync

# This is the commit message #45:

sync

# This is the commit message #46:

sync

# This is the commit message #47:

sync

# This is the commit message #48:

sync

# This is the commit message #49:

sync

# This is the commit message #50:

sync

# This is the commit message #51:

sync

# This is the commit message #52:

sync

# This is the commit message #53:

sync

# This is the commit message #54:

sync

# This is the commit message #55:

sync

# This is the commit message #56:

sync

# This is the commit message #57:

sync

# This is the commit message #58:

sync

# This is the commit message #59:

sync

# This is the commit message #60:

sync

# This is the commit message #61:

sync

# This is the commit message #62:

sync

# This is the commit message #63:

sync

# This is the commit message #64:

sync

# This is the commit message #65:

sync

# This is the commit message #66:

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

@halturin halturin

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@halturin