After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 133518 - possible double free
possible double free
Status: RESOLVED FIXED
Product: gtkhtml2
Classification: Deprecated
Component: General
2.5.x
Other Linux
: Normal normal
: ---
Assigned To: padraig.obriain
padraig.obriain
Depends on:
Blocks:
 
 
Reported: 2004-02-05 15:42 UTC by Pawel Salek
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.5/2.6

Attachments
test file (12.86 KB, text/html)
2004-02-05 15:43 UTC, Pawel Salek 		  Details
Proposed patch (1.35 KB, patch)
2004-02-06 14:30 UTC, padraig.obriain 		none Details | Review

Description Pawel Salek 2004-02-05 15:42:51 UTC 
When displaying attached file with testgtkhtml and malloc checking enabled,
one gets a warning about invalid pointer passed to free(). It leads to
crashes in more involved programs.

$ MALLOC_CHECK_=1 tests/testgtkhtml
malloc: using debugging hooks
malloc: using debugging hooks
malloc: using debugging hooks
malloc: using debugging hooks
malloc: using debugging hooks
Initializing gnome-vfs...
gtk_window: 456 browser_window: 456
signal "link_clicked" url = "file:/home/pawsa/html.html"
free(): invalid pointer 0x8193c38!
 
(lt-testgtkhtml:9519): HtmlCss-WARNING **: Unhandled property: 323
text-underline
Comment 1 Pawel Salek 2004-02-05 15:43:31 UTC 
Created attachment 24104 [details]
test file
Comment 2 Pawel Salek 2004-02-05 15:45:09 UTC 
This happens on current CVS, I should add.
Comment 3 padraig.obriain 2004-02-06 14:30:32 UTC 
Created attachment 24143 [details] [review]
Proposed patch
Comment 4 padraig.obriain 2004-02-06 14:31:35 UTC 
Does the proposed patch fix the problem?
Comment 5 Pawel Salek 2004-02-06 14:58:56 UTC 
It appears so! Thanks!
Comment 6 padraig.obriain 2004-02-06 15:10:39 UTC 
Committed patch to CVS HEAD.