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

segfault on syntax error #421

Closed
pavelkveton opened this issue Jan 18, 2016 · 1 comment
Closed

segfault on syntax error #421

pavelkveton opened this issue Jan 18, 2016 · 1 comment
Milestone

Comments

@pavelkveton
Copy link
Contributor

IT-90-EXPORT_USIM_OPTIONS.zip

running the test file is fine, unless you change the line

    const REMOTE_TABLE = "ps_etlit_erp_codprod";

to

    const REMOTE_TABLE = "ps_etlit_erp_codprod",

(mark the last character change from semicolon to colon). The latter one produces a segfault, instead of some reasonable syntax error message.

Note that I had to zip the test file since github does not like .qtest suffix...

@tethal
Copy link
Contributor

tethal commented Jan 18, 2016

minimalistic example:

%new-style
class P {
  private { const X = string m; }
}

or with old style:

class P {
  private { const X = my string $m; }
}

Basically the issue is that we are declaring a local variable m which does not make sense.

davidnich added a commit that referenced this issue Jan 18, 2016
…s_decl

refs #421 fixed segfault when a local variable was used in const init…
@tethal tethal added the fixed label Jan 18, 2016
@tethal tethal closed this as completed Jan 18, 2016
@davidnich davidnich added this to the 0.8.12 milestone Feb 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants