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

Apps crash occasionally when writing output to integrated terminal if Win 10 Fall Creators Update is installed #38068

Closed
Wosi opened this issue Nov 10, 2017 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code) windows VS Code on Windows issues
Milestone

Comments

@Wosi
Copy link

Wosi commented Nov 10, 2017

This one is quite tough.
Today it was almost impossible for me to use the integrated terminal and build tasks in VSCode. Some tools occasionally crashed when they were run inside the integrated terminal. The tools randomly stopped writing their output while some others aborted with a "Disk full" error. This happened everytime when the tools were writing a lot of output to the integrated terminal.
I tested them intensively in standalone powershell and cmd windows: They worked as expected. They only crashed when running inside the integrated terminal.

My first thought was that it had something to do with the latest update of VSCode so I downgraded to the September release but it showed the same odd behaviour.

Then I uninstalled the Windows 10 Fall Creators Update which I installed last night. Now everything works fine again.

I guess this is more an issue in Windows than in VSCode but it's quite strange that different applications crash only when they run in the integrated terminal. That's why I post it here. Maybe you have an idea what could have changed in the Windows API that causes this effect. And I guess you might be interested in having applications behave identically regardless the terminal in which they are being run.

I've managed to create a test project that (almost) always crashed when running in the integrated terminal. It will print 100k lines of output to the terminal. You can find it here: https://github.com/Wosi/Print100kLines

Steps to reproduce (hopefully):

  • Install VSCode to a Windows 10 machine with the Fall Creator's Update
  • Run Print100kLines.exe in the integrated terminal -> It crashed on my system
  • Run the same application in any other terminal and it will work fine
@vscodebot vscodebot bot added the terminal Integrated terminal issues label Nov 10, 2017
@Tyriar
Copy link
Member

Tyriar commented Dec 15, 2017

@bitcrazed possible problem with FCU?

@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug upstream Issue identified as 'upstream' component related (exists outside of VS Code) windows VS Code on Windows issues labels Dec 15, 2017
@Tyriar Tyriar added this to the Backlog milestone Dec 15, 2017
@bitcrazed
Copy link

Thanks for the ping. @zadjii_msft is giving it a quick look.

@zadjii-msft
Copy link
Member

Hey all,

Took a look at this. I can confirm that it's happening with both VsCode and Hyper on 16299. Fortunately however, there's a change that's already been checked in that has a fix for this. I don't have the link to a particular GH issue that describes the original issue handy, though if I recall correctly it had something to do with WriteConsoleA getting re-written slightly.

It should be going out to Insiders soon, but with the holidays, it'll probably be closer to the end of janurary before this makes it all the way up the tree.

@Tyriar
Copy link
Member

Tyriar commented Dec 18, 2017

@zadjii-msft is this the same? microsoft/terminal#40

@zadjii-msft
Copy link
Member

Yep that looks like the one!

@Tyriar Tyriar closed this as completed Dec 28, 2017
@Tyriar Tyriar added the *duplicate Issue identified as a duplicate of another issue(s) label Dec 28, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Feb 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code) windows VS Code on Windows issues
Projects
None yet
Development

No branches or pull requests

4 participants