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

divide by zero in instance creation #2238

Closed
Zekken01 opened this Issue Apr 9, 2018 · 3 comments

Comments

Projects
None yet
2 participants
@Zekken01
Copy link

Zekken01 commented Apr 9, 2018

pasting

int main()
{
    int i = 1 / 0;
}

into the create an instance causes weird stuff

@peterix peterix added the multimc: bug label Apr 9, 2018

@peterix

This comment has been minimized.

Copy link
Member

peterix commented Apr 9, 2018

Instance naming needs some filtering:

  • normalize whitespace to single spaces
  • remove leading and trailing whitespace
  • strip newlines and weird characters
  • remove invisible unicode characters?
  • forbid usage of !

For the on-disk name:

  • strip/ASCII-ize unicode
  • replace whitespace with _
@peterix

This comment has been minimized.

Copy link
Member

peterix commented Apr 9, 2018

Also see:
#847
#1105
#1674

peterix added a commit that referenced this issue Oct 31, 2018

@peterix

This comment has been minimized.

Copy link
Member

peterix commented Oct 31, 2018

This particular instance of this problem has been fixed in develop.

@peterix peterix closed this Oct 31, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.