[xslt] Apparent bug in libxslt



Hi all,

I was running some unit tests through Valgrind today and ran across what appears to be a (small) bug in libxslt.

It occurs in libxslt/transform.c on line 922 and 930

http://svn.gnome.org/viewvc/libxslt/trunk/libxslt/transform.c?view=markup

Line 922 (potentially) frees the memory pointed to by copy, line 930 dereferences the pointer. It looks to me like the easiest solution is to change line 922 to

copy = xmlAddChild
(target, copy);

Does this seem like the correct fix? Is there an appropriate place to send a patch?

Hopefully, this is the correct address for this type of issue. Thanks for any help,

Daniel





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]