#42 configxml.cpp memory leak

closed-accepted
nobody
None
5
2012-11-09
2012-11-09
logzero
No

Another case of
configNodeFactory *domf = new configNodeFactory();
configNode *top = (configNode*) domf->LoadXML( configfile );

I had a look at the code, see no reason for allocating the node factory on the heap. Alternative would be to use explicit delete of course.

Discussion

  • logzero

    logzero - 2012-11-09

    patch

     
  • Klauss++

    Klauss++ - 2012-11-09

    I like auto variables... so it's more than OK the way this patch does it. If you find more, fix them all and send them in one patch.

     
  • Klauss++

    Klauss++ - 2012-11-09
    • status: open --> closed-accepted
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks