DO NOT REPLY TO THIS MESSAGE.  INSTEAD, POST ANY RESPONSES TO THE LINK BELOW.

[STR Pending]

Link: http://www.fltk.org/str.php?L2836
Version: 1.3-current
Fix Version: 1.3.1 (r9443)


I could reproduce the bug on Windows and Linux, using the posted demo
program. Valgrind (Linux) showed "Invalid read" errors.

The fix doesn't call strdup() if the old and new labels are the same (i.e.
copy_label() is called with the old label() value, if and only if it is
already a copied label). This modification has been done, so that it is
compatible with Fl_Widget::label() and to avoid unnecessary strdup()'s. I
also did some code cleanup to avoid duplicate code.

Tested with valgrind before and after the change, there are no "Invalid
read" errors anymore, and there are no (new) memory leaks.


Link: http://www.fltk.org/str.php?L2836
Version: 1.3-current
Fix Version: 1.3.1 (r9443)

_______________________________________________
fltk-bugs mailing list
fltk-bugs@easysw.com
http://lists.easysw.com/mailman/listinfo/fltk-bugs

Reply via email to