DOS IDE BUG: Drunk GPF after 50'000 bottles of BEER

Index > IDE Development > DOS IDE BUG: Drunk GPF after 50'000 bottles of BEER

Author

Thread

DOS386

Joined: 08 Dec 2006
Posts: 1900

DOS386 18 Jun 2009, 12:12

Quote:

Who drunk all the beer!

No idea, sorry Sad

I tested revolution's BEER algorithm , as-is works very well in DOS IDE and can copy the drunk garbage into the clipboard and from there to the main text buffer Smile

But I felt somewhat challenged so I wanted to test what happens if you drink 1'000'000 bottles instead of just 100: Out of memory. Well, so I reduced and reduced, and with 50'000 bottles I got a GPF (see shot). Crashing instruction is REP MOVSB in update_screen, however the screen is damaged (see top line in shot), and the ES selector is faulty Sad

Code:

  display_block:
       push    es
  mov     es,[low_memory_selector]
    mov     edi,[display_length]
        mov     eax,[display_length]
        add     eax,ecx
     cmp     eax,[low_memory_size]
       ja      not_enough_memory ; HERE: ES = ???  
 mov     [display_length],eax
        rep     movsb
       pop     es
  ret

; Error handling

  not_enough_memory:
 call    update_screen ; HERE: ES = ??? 
      mov     esi,_memory_error
   mov     ebx,_error
  movzx   eax,[error_box_colors]

What's the point of hogging the low memory and maintaining this low_memory_selector at all ? Confused

Can this get fixed ? Silent update ? 1.68.01 ? Wink

Description:	Drunk GPF (7'564 Bytes)
Filesize:	7.39 KB
Viewed:	7598 Time(s)

_________________
Bug Nr.: 12345

Title: Hello World program compiles to 100 KB !!!

Status: Closed: NOT a Bug

18 Jun 2009, 12:12

Tomasz Grysztar

Joined: 16 Jun 2003
Posts: 8351
Location: Kraków, Poland

Tomasz Grysztar 18 Jun 2009, 12:47

DOS386 wrote:

Can this get fixed ? Silent update ? 1.68.01 ?

You're still making the same mistake. 1.68 is version of core, not IDE. FASMD is currently versioned only by the date of last update. Wink

18 Jun 2009, 12:47

DOS386

Joined: 08 Dec 2006
Posts: 1900

DOS386 18 Jun 2009, 13:07

Tomasz Grysztar wrote:

You're still making the same mistake.

NO.

Quote:

1.68 is version of core, not IDE. FASMD is currently versioned only by the date of last update.

I do know this ... so this bug can be fixed in 1.68.01 (or 1.69.01) if you also fix something in the core, or "silently" in 1.68, or not at all, I can fix it myself by a very trivial rearrangement of instructions, but I would prefer this fix to be included in next "official" version also whenever it comes with whatever number Wink

18 Jun 2009, 13:07

Tomasz Grysztar

Joined: 16 Jun 2003
Posts: 8351
Location: Kraków, Poland

Tomasz Grysztar 18 Jun 2009, 13:10

DOS386 wrote:

I do know this ... so this bug can be fixed in 1.68.01 (or 1.69.01) if you also fix something in the core, or "silently" in 1.68

It won't be silent, the date will get updated. And there won't be any 1.68.01 version, because 1.68 is not a development line.

18 Jun 2009, 13:10

rCX

Joined: 29 Jul 2007
Posts: 172
Location: Maryland, USA

rCX 22 Jun 2009, 03:49

Kinda off topic but DOS386, what program are you using to make screenshots?

22 Jun 2009, 03:49

DOS386

Joined: 08 Dec 2006
Posts: 1900

DOS386 19 Dec 2009, 01:35

> what program are you using to make screenshots?

I don't know: http://www.unet.univie.ac.at/~a0503736/php/drdoswiki/index.php?n=Main.Gallery

Well, the BUG still isn't fixed:

Quote:

flat assembler 1.68 for DOS size: 309 kilobytes
last update: 06 Jul 2009 13:46:04 UTC

flat assembler 1.69.10 for DOS size: 308 kilobytes
last update: 28 Oct 2009 16:18:52 UTC

Less luck with a trivial bug than with the calculator Laughing

EDIT : reposted since still not fixed

19 Dec 2009, 01:35

< Last Thread | Next Thread >

Forum Rules:

You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
You cannot attach files in this forum
You can download files in this forum