Login
Username:

Password:

Remember me



Lost Password?

Register now!

Sections

Who's Online
113 user(s) are online (65 user(s) are browsing Forums)

Members: 0
Guests: 113

more...

Headlines

 
  Register To Post  

« 1 (2)
Re: How detect and fix heap trashing ?
Not too shy to talk
Not too shy to talk


See User information
@kas1e
You should probably put the full source and your analyzer logs somewhere so we can take a better look. E.g. we don't even know how the structs look like etc.

Go to top
Re: How detect and fix heap trashing ?
Home away from home
Home away from home


See User information
@Daniel

Source:
https://sourceforge.net/p/foobillardplus/code/HEAD/tree/src/

DrMemory logs file "results.txt" (120kb) :
http://kas1e.mikendezign.com/aos4/gl4 ... llardplusplus/results.txt

DrMemory logs file "potential errors" (660kb):
http://kas1e.mikendezign.com/aos4/gl4 ... plus/potential_errors.txt


Result.txt one is which is bugs for sure, search there for "menu.c" , "font.c" and "textojb.c" for those 3 bugs it found.
"Potentional errors" as usuall can be all kind of false alarms of course

Join us to improve dopus5!
AmigaOS4 on youtube
Go to top
Re: How detect and fix heap trashing ?
Just popping in
Just popping in


See User information
@kas1e

getStringPixmapFT()/my_draw_bitmap() likely trashes mem when blitting into chunky buffer (data) -> font glyph info/"coords" (bitmap_left) maybe can have values the code does not expect (like negative ones). Because of kerning or whatever. my_draw_bitmap() isn't clipping anything.





Go to top
Re: How detect and fix heap trashing ?
Just popping in
Just popping in


See User information
Try

int blitx pen_x face->glyph->bitmap_left;
int blity pen_y font_height*face->ascender/(face->ascender-face->descender) - face->glyph->bitmap_top;
int blitw face->glyph->bitmap.width;
int blith face->glyph->bitmap.rows;

if ((
blitw <= w) && (blith <= h) &&
    (
blitx >= 0) && (blity >= 0) &&
    (
blitx blitw <= w) && (blity blith <= h))
{
  
my_draw_bitmap( (char *)face->glyph->bitmap.bufferblitwblithblitxblity, *data w);
}
else
{
  
kprintf("=== ignoring my_draw_bitmap which would trash: %d,%d %dx%d (bm %dx%d)\n"blitxblityblitwblithwh,);
}


Go to top
Re: How detect and fix heap trashing ?
Not too shy to talk
Not too shy to talk


See User information
@kas1e
Ouch, I see, this is the code where you got that completely messed up sound-file-extension code from...

@Georg
yes, worth a try. Commenting out that draw-code seemed to help.

Go to top
Re: How detect and fix heap trashing ?
Home away from home
Home away from home


See User information
@Daniel
Yeah, is that one :) We don't have normal billiard games (there some other old one, pure "billard" on os4depot, but it suck heavy, no sound, no music, dated look, etc). This one (foobillard++), is looks good, plays good, and i almost have no needs to worry about endianes (just in one place, where they load raw sound via fread()).

So, want to deal with all this, as game at least works already over gl4es/ogles/warp3d.

@Daniel,Georg

Yeah, we very close ! With that function, when i come to those entries which crashes before, it didn't now, but that what it bring to console:

=== ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 256x32)
=== 
ignoring my_draw_bitmap which would trash: -1,2 25x23 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)
=== 
ignoring my_draw_bitmap which would trash: -1,0 14x15 (bm 512x32)


That for 4 different places where it crashes before.

And visually, it also start to miss first letters. In some places. See screesnhots:

http://kas1e.mikendezign.com/aos4/gl4 ... obillardplusplus_fix1.jpg
http://kas1e.mikendezign.com/aos4/gl4 ... obillardplusplus_fix2.jpg
http://kas1e.mikendezign.com/aos4/gl4 ... obillardplusplus_fix3.jpg

On first you can see at top of game (no the menu entries itself, but at left top side , pure help-message), there is words "hank you very much" , while should be "Thank you very much". On second one "ypes", but should be "Types". And on third screenshot, this time it miss from menu entry in the world "Yes" first "Y.

When its all buggy , and i just comment out free(textdata) to avoid crashes, all letters are in place (but that of course trash the memory).

Join us to improve dopus5!
AmigaOS4 on youtube
Go to top
Re: How detect and fix heap trashing ?
Just popping in
Just popping in


See User information
@kas1e

Quick fix (assumes only blitx can be out of bounds (on left side)). Undo previous stuff (kprintf), first:

[..]

int left_most 0;

[..]

for(
i=0;i<2;i++)
{
 if(
i==1)
 {
   
-= left_most;
   
   [..]
 }
 
 [..]
 
 for( 
0; [..] ) {
 
 [..]
 
int blitx pen_x face->glyph->bitmap_left;
int blity pen_y font_height*face->ascender/(face->ascender-face->descender) - face->glyph->bitmap_top;
int blitw face->glyph->bitmap.width;
int blith face->glyph->bitmap.rows;

if (
blitx left_mostleft_most blitx;

if (
!= 0)
{
  
my_draw_bitmap( (char *)face->glyph->bitmap.bufferblitwblithblitx left_mostblity, *data w);
                                                                    ^^^^^^^^^^^^^^^^^
 [..]
}
else
{
 [..]
}

[..]


Go to top
Re: How detect and fix heap trashing ?
Home away from home
Home away from home


See User information
@Georg
Yeah, it did the trick !

Through as i understand we just avoid accessing to trashed memory, while some parts of memory still trashed and leaks still there ?

Join us to improve dopus5!
AmigaOS4 on youtube
Go to top
Re: How detect and fix heap trashing ?
Home away from home
Home away from home


See User information
@Georg
Dunno if it another bug, or related to that trashed memory, but
another bug arise in another place : not with free(), but with just accessing to some data via that line:

if( (*menu)->parent_entry != ){
            if( (*
menu)->parent_entry->show_subsetting ){
                
menu_entry_set_settingtext( (*menu)->parent_entry, (*menu)->entry[(*menu)->select_index].text );
            }
        }


Its from the same menu.c , just in the menu_choose() this time. Crashes exactly on that first line "if( (*menu)->parent_entry != 0 ){" :

https://sourceforge.net/p/foobillardpl ... HEAD/tree/src/menu.c#l406

Visually, it happens when i just choice one of the menu entries (few of them cause that crash). Dunno if it related through, or if it just another different issue with that shit-code ..

Probabaly as accessing is to "*menu" , maybe its again abou trashed heap ?

Join us to improve dopus5!
AmigaOS4 on youtube
Go to top
Re: How detect and fix heap trashing ?
Just popping in
Just popping in


See User information
@kas1e

Crash is more likely second line ("if( (*menu)->parent_entry->show_subsetting )") with parent_entry likely being uninitialized.

Add a line to menu_new() which sets menu->parent_entry to 0. Or change the malloc in there to calloc.


Go to top
Re: How detect and fix heap trashing ?
Just popping in
Just popping in


See User information
@kas1e

Quote:

Through as i understand we just avoid accessing to trashed memory, while some parts of memory still trashed and leaks still there ?


It avoids blitting outside left bounds (and thus trashing memory there) because that's what debug output shows it did (negative coords). So the change causes the bitmap to be allocated bigger (wider) and the blits to be offset to the right so that final blit X position of each glyph (char in string) is always >= 0.

Theoretically (weird fonts/languages/accents?) it could still blit outside right/bottom/top side. So to be 100 % safe you would need to check all sides. ~

[..]

int top_most 0;
int right_most 0;
int bottom_most 0;

[..]

int blitx2 blitx blitw 1;
int blity2 blity blith 1;

if (
blitx left_mostleft_most blitx;
if (
blity top_mosttop_most blity;
if (
blitx2 right_mostright_most blitx2;
if (
blity2 bottom_mostbottom_most blity2;

[..]

= (right_most left_most) + 1;
= (bottom_most top_Most) + 1;

[..]

my_draw_bitmap([..] blitx left_mostblity bottom_most [..]);



Go to top
Re: How detect and fix heap trashing ?
Home away from home
Home away from home


See User information
@Georg
Yeah, you was right ! Just setting "menu->parent_entry = 0;" in menu_new did the trick ! Thanks a bunch!

Will also add better check on possible negative values as you show.

Thanks again, hope there will be no new bugs of this kind :)

Join us to improve dopus5!
AmigaOS4 on youtube
Go to top
Re: How detect and fix heap trashing ?
Home away from home
Home away from home


See User information
.

Join us to improve dopus5!
AmigaOS4 on youtube
Go to top

  Register To Post
« 1 (2)

 




Currently Active Users Viewing This Thread: 1 ( 0 members and 1 Anonymous Users )




Powered by XOOPS 2.0 © 2001-2023 The XOOPS Project