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

Random Freeing #20

Open
ProgrammerDan opened this issue Jan 24, 2017 · 2 comments
Open

Random Freeing #20

ProgrammerDan opened this issue Jan 24, 2017 · 2 comments

Comments

@ProgrammerDan
Copy link
Contributor

Strange rare race condition

Where a pearl is in a crafting table grid, being recharged, at the same moment when the Decay task runs, the pearl will be released.
@ProgrammerDan ProgrammerDan assigned ghost Jan 24, 2017
@ghost
Copy link

ghost commented Jan 24, 2017

Interesting. Any more info than this? The decay task uses the same check that the locate command uses, so if a bug exists in the routine, then someone could likely force the condition by spamming /ep locate.

@ProgrammerDan
Copy link
Contributor Author

It's just an unchecked condition.

The pearl is sitting in the crafting grid alongside the obsidian. Not an inventory holder, so unrecognized. Fails all checks, and is freed due to default, default.

ProgrammerDan added a commit that referenced this issue May 26, 2017
@ProgrammerDan
Changes targetting #20 but no change during isolated testing. Issue r…
c3c3c29 
…emains unresolved.
@Lazersmoke Lazersmoke mentioned this issue Mar 4, 2018
Closed
Diet-Cola pushed a commit to Diet-Cola/ExilePearl that referenced this issue May 24, 2022
@wingzero54
Merge pull request DevotedMC#20 from Diet-Cola/formatting-fixes
844b86f 
Fix colour parsing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ProgrammerDan