GNUAspell / aspell

http://aspell.net
GNU Lesser General Public License v2.1
243 stars 53 forks source link

memory leak on 'dump master' command. #167

Open aspell-helper opened 20 years ago

aspell-helper commented 20 years ago

Gary Setter setterg\@sf created a bug report on 2004-11-01 13:16:27 UTC (Orig. from https://sourceforge.net/p/aspell/bugs/108)

The program leaks memory when the 'dump master' command is given.

Note that the Dict (aka Dictionary) object is not released after it is used.

aspell-helper commented 20 years ago

Kevin Atkinson kevina\@sf updated the issue on 2004-11-01 20:25:44 UTC

aspell-helper commented 20 years ago

Kevin Atkinson kevina\@sf commented on 2004-11-01 20:25:44 UTC

Logged In: YES user_id=6591

More specific info:

This is good. I had a leak, found it and I don't know why you didn't have one. It centers on how your global cache works.

This is my new dump master code, note the new like with todays date.

} else if (action == do_dump) {

EXIT_ON_ERR_SET(add_data_set(config->retrieve("master-path"), *config), Dict *, d); StackPtr conv(setup_conv(d->lang(), config)); dump(d, conv); d->release(); //31-Oct-04 }

What you may have intended, was to keep the Dict (aka Dictionary) object in the global cache and let the destructor of the cache delete it it. But your ~GlobalCacheBase does not release/delete anything. How come you are not getting a memory leak?

aspell-helper commented 17 years ago

Kevin Atkinson kevina\@sf commented on 2006-11-24 05:00:44 UTC

Logged In: YES user_id=6591 Originator: NO

Yes there is a leak.

However, d->release() does not fix it.

I need to study how dictionaries are normally deleted before I can determine the correct way to fix this.

aspell-helper commented 14 years ago

Kevin Atkinson kevina\@sf commented on 2010-03-04 01:46:02 UTC

What version of Aspell is this. Use "aspell -v" to find out.

aspell-helper commented 14 years ago

Kevin Atkinson kevina\@sf updated the issue on 2010-03-04 01:46:03 UTC