CSS leaks: WAS Re: [Dillo-dev] <ol>'s leak the labels of their items

On Mon, Jan 26, 2009 at 11:33:43PM +0100, Hofmann Johannes wrote:

There is at least one leak I know of. It's font-family names that are parsed in CSS style data.

Aha! I looked at the Valgrind log for that one but didn't know the cause. Valgrind identifies the dStrdup() in val->strVal = dStrdup(parser->tval); in Css_parse_value() as the leak. What's the fix? I thought the problem was that CssPropertyList::set() does not free any existing value but I'm not sure I understand how these classes are meant to manage their memory. Regards, Jeremy Henty