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

VS2019 crashes when an F# source contains this string: let xxx x y = (# "divvvv" x y : float #) #7164

Closed
Arshia001 opened this issue Jul 9, 2019 · 7 comments

Comments

@Arshia001
Copy link

Arshia001 commented Jul 9, 2019

My installation of Visual Studio 2019 (version 16.1.5) crashed after I typed in this code and refuses to open the project again:

let xxx x y = (# "divvvv" x y : float #)

Repro steps

  1. Open an F# project in VS2019
  2. Type in the above code
  3. Mouse over the code.

Expected behavior

It should probably show an error that reads "Don't mess with stuff you don't understand, kid."

Actual behavior

Crashes completely.

Known workarounds

Open the file in another editor, remove that code.

  • Operating system Windows 10
  • .NET Runtime kind dotnet core 3.0 preview 6
  • Editing Tools VS 2019
@cartermp
Copy link
Contributor

cartermp commented Jul 9, 2019

Repro'd. This is critical to fix. cc @TIHan @brettfo @KevinRansom

@KevinRansom
Copy link
Member

Is this a regression?

@cartermp
Copy link
Contributor

cartermp commented Jul 9, 2019

I'm assuming VS didn't hard crash whenever IL literals were specified in source at some point, yes.

@TIHan
Copy link
Contributor

TIHan commented Jul 9, 2019

Yea, this should not take down VS.

@TIHan
Copy link
Contributor

TIHan commented Jul 9, 2019

It seems to crash when the IL in the string is invalid.

@TIHan
Copy link
Contributor

TIHan commented Jul 9, 2019

So, I don't think this is a regression, or at least a recent one.

Basically, the parser for the IL string will throw an exception by failwith "parse error" which will cause it to not recover and will re-raise that exception in which case, nothing handles it and takes down VS.

@TIHan
Copy link
Contributor

TIHan commented Jul 10, 2019

Closing as we merged the fix.

@TIHan TIHan closed this as completed Jul 10, 2019
@cartermp cartermp modified the milestones: Backlog, 16.3 Jul 24, 2019
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

4 participants