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

assertion failed "nNewWidth>0 && nNewHeight>0" in RealBuffer.cpp:1894 ConEmu 161022 #934

Closed
0anton opened this issue Nov 1, 2016 · 8 comments
Projects

Comments

@0anton
Copy link

0anton commented Nov 1, 2016

Hi,

I experience ConEmu crashes after resizing ConEmu window.

Assertion ConEmu 161022 [32] PID=7808, TID=14216
RealBuffer.cpp:1894
nNewWidth>0 && nNewHeight>0

Versions

ConEmu build: 161022 x64
OS version: Windows 10 x64
Used shell: cygwin

Problem description

ConEmu crashes after console resize (e.g. full-screen / part-screen toggle)

Steps to reproduce

  1. Start Conemu.
  2. Create couple of shells in the screen split mode.
  3. Open some files.
  4. Toggle full-screen for one of them.

Actual results

ConEmu crashes.

Expected results

ConEmu continue to work.

Additional files

[Traces]
https://www.dropbox.com/s/q77q57047exc4mx/Trap-161022-7808.zip?dl=0

[Settings]
https://www.dropbox.com/s/2jlceqaicbcoyhd/ConEmu.xml?dl=0

[screenshots]
https://www.dropbox.com/s/fhdeezyip86k2tz/ScreenShot031.jpg?dl=0

[logs]

(will follow with the next crash)

@Maximus5
Copy link
Owner

Maximus5 commented Nov 1, 2016

That usually means something bad happened with conhost.exe.
Can you reproduce? Than attach LogFiles please.

@0anton
Copy link
Author

0anton commented Nov 1, 2016

Sure Max,

I'll provide logs with the next crash.

Kind regards,
Anton

On Tue, Nov 1, 2016 at 3:27 PM, Maximus5 notifications@github.com wrote:

That usually means something bad happened with conhost.exe.
Can you reproduce? Than attach LogFiles please.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#934 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQgg5C44Tjbrxehh6I7AD2yh-ZZx4pBUks5q50xhgaJpZM4KmJPI
.

@0anton
Copy link
Author

0anton commented Nov 3, 2016

Hi Max,

As requested, please find the crash log here:

ConEmu-srv-19028.zip

Kind regards,
Anton

@Maximus5
Copy link
Owner

Maximus5 commented Dec 8, 2016

2 all

I need more information to reproduce the issue.

  1. What are you doing exactly?
  2. Show screenshots with status bar visible of course. I need at least these columns: Terminal modes, Console visible rectangle, Console buffer size, Cursor information, Console server PID.
  3. I need all LogFiles created by this instance of ConEmu. Not only one selected file without any information what were you doing.

The issue is hard to fix because it depens on some race conditions in Microsoft's conhost.exe.

@Shredder121
Copy link

At the time I encountered my issue, it was easily reproducible.
You would just generate output until you were at the end of the buffer (scroll handle at the bottom of the scroll bar), and then clear.
But now, with a fresh install (both Windows and conemu) it doesn't happen.

So something, somewhere must break it, but I think it's not directly conemu.

@0anton
Copy link
Author

0anton commented Dec 8, 2016 via email

@m0n
Copy link

m0n commented Feb 27, 2018

ConEmu build: 180206 x64
OS version: Windows 10 LTSB x64
Used shell: Far Manager, version 3.0 (build 5151) x64

I can reproduce this assertion 10/10 on my PC:

  1. ConEmu64 -basic -log
  2. FAR.exe - without this step everything is OK
  3. wmic.exe diskdrive list brief /format:list
  4. Command executed properly
  5. wmic.exe
  6. Assertion happens

The dump & logs of this sequence are here.

I noticed that starting wmic.exe modifies "Screen Buffer Size" to 1500*300 in a normal cmd.exe (with "Use legacy console")

Wow - I just realised that if You set "Use legacy console" to "on" then wmic.exe starts with an extra delay but no assertion happens! This is an answer to why this bug not appears on Windows 7...

@Maximus5 Maximus5 added this to To Do in ConEmu via automation Apr 24, 2018
Maximus5 added a commit that referenced this issue Apr 24, 2018
@Maximus5 Maximus5 moved this from To Do to Ready for Testing in ConEmu Apr 24, 2018
Maximus5 added a commit that referenced this issue Apr 28, 2018
@0anton
Copy link
Author

0anton commented Feb 28, 2021

I think this issue can be closed since much time passed by.

@0anton 0anton closed this as completed Feb 28, 2021
ConEmu automation moved this from Ready for Testing to Done Feb 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
ConEmu
  
Done
Development

No branches or pull requests

4 participants