Skip to content
This repository has been archived by the owner on Oct 28, 2023. It is now read-only.

--m-rand flag out-of-bounds access panic instead of error message #45

Closed
super-continent opened this issue Sep 23, 2019 · 0 comments · Fixed by #46
Closed

--m-rand flag out-of-bounds access panic instead of error message #45

super-continent opened this issue Sep 23, 2019 · 0 comments · Fixed by #46
Assignees
@Jake-Shadle
Labels
bug Something isn't working

Comments

@super-continent
Copy link

To reproduce the bug, enter any usually-valid valid cli command with the --m-rand flag set to 0

Output I got:
image
@super-continent super-continent added the bug Something isn't working label Sep 23, 2019
@Jake-Shadle Jake-Shadle self-assigned this Sep 25, 2019
Jake-Shadle added a commit that referenced this issue Sep 25, 2019
@Jake-Shadle
Validate m_rand is > 0
4495f49 
Resolves: #45
@Jake-Shadle Jake-Shadle mentioned this issue Sep 25, 2019
Merged
@mergify mergify bot closed this as completed in #46 Sep 25, 2019
mergify bot pushed a commit that referenced this issue Sep 25, 2019
@Jake-Shadle @mergify
Fix --m-rand (#46)
cd3ca52 
* Validate m_rand is > 0

Resolves: #45

* Update CHANGELOG
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Jake-Shadle Jake-Shadle

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix --m-rand EmbarkStudios/texture-synthesis
2 participants
@Jake-Shadle @super-continent