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

if the module heap is >0 and <0x800 set it to 0x800 #617

Merged
merged 1 commit into from
Feb 10, 2020

Conversation

cracyc
Copy link
Contributor

@cracyc cracyc commented Feb 9, 2020

ntvdm seems to work this way
fixes #615

@otya128 otya128 merged commit 32d9df5 into otya128:master Feb 10, 2020
@cracyc cracyc deleted the minheap branch February 10, 2020 14:39
@otya128
Copy link
Owner

otya128 commented Mar 8, 2021

It seems to behave differently when pSeg->minsize + pModule->ne_stack + pModule->ne_heap=0x800 exceeds 0x10000.

minsize:656 + stack:64622 + heap:19
---------------------------
Can't run 16-bit Windows program
---------------------------
Insufficient memory to run this application. Quit one or more Windows applications and then try again.
---------------------------
OK   
---------------------------
minsize:656 + stack:64621 + heap:19
ne_heap=256
minsize:656 + stack:62830 + heap:19
---------------------------
Can't run 16-bit Windows program
---------------------------
Insufficient memory to run this application. Quit one or more Windows applications and then try again.
---------------------------
OK   
---------------------------
minsize:656 + stack:62829 + heap:19
ne_heap=2048
minsize:656 + stack:62839 + heap:2039
---------------------------
Can't run 16-bit Windows program
---------------------------
Insufficient memory to run this application. Quit one or more Windows applications and then try again.
---------------------------
OK   
---------------------------
minsize:656 + stack:62838 + heap:2039
ne_heap=2039

otya128 added a commit that referenced this pull request Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash with "integer divide by 0" error when accessing to COM port
2 participants