Menu

#108 memory leak on 'dump master' command.

0.60
closed
true-bug (70)
3
2016-12-11
2004-11-01
Gary Setter
No

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.

Discussion

  • Kevin Atkinson

    Kevin Atkinson - 2004-11-01

    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<Convert> 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?

     
  • Kevin Atkinson

    Kevin Atkinson - 2004-11-01
    • labels: --> true-bug
    • priority: 5 --> 3
    • assigned_to: nobody --> kevina
     
  • Kevin Atkinson

    Kevin Atkinson - 2006-11-24

    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.

     
  • Kevin Atkinson

    Kevin Atkinson - 2010-03-04

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

     
  • Kevin Atkinson

    Kevin Atkinson - 2010-03-04
    • milestone: --> 0.60
     
  • Kevin Atkinson

    Kevin Atkinson - 2016-12-11

    This issue has moved to GitHub: https://github.com/GNUAspell/aspell/issues/167

     
  • Kevin Atkinson

    Kevin Atkinson - 2016-12-11
    • Status: open --> closed