[Expat-bugs] [ expat-Bugs-2680699 ] Memory leak: outName

[SourceForge.net]

Bugs item #2680699, was opened at 2009-03-10 16:46
Message generated for change (Comment added) made by kwaclaw
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110127&aid=2680699&group_id=10127

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
>Group: Not a Bug
>Status: Closed
>Resolution: Rejected
Priority: 5
Private: No
Submitted By: Bill Egert (boogachamp)
>Assigned to: Karl Waclawek (kwaclaw)
Summary: Memory leak: outName

Initial Comment:
Using cppcheck 1.30 against expat 2.0.1 yeilds:

cppcheck -j2 -a -s -q .
[./xmlwf/xmlwf.c:857]: (all) Memory leak: outName

.
.
.
    if (outputDir) {
      if (outputType == 'm')
        metaEndDocument(parser);
      fclose(fp);
      if (!result)
        tremove(outName);
      free(outName);
    }
    XML_ParserFree(parser);
  }
  return 0;
}

shouldn't outName be freed regardless of outputDir's value?



----------------------------------------------------------------------

>Comment By: Karl Waclawek (kwaclaw)
Date: 2012-03-05 11:00

Message:
I looked at the code, and outName is only allocated when outputDir !=
null.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110127&aid=2680699&group_id=10127