[XERCESC-1131] DOMElement::setIdAttribute*() memory leak

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: 2.4.0
• Fix Version/s: 2.5.0
• Component/s: DOM
• Environment: Operating System: All; Platform: All
• Bugzilla Id: 26607
• Assignee: Unassigned
• Reporter: merlin
• Created: 03/Feb/04 00:46
• Updated: 12/Mar/18 03:55
• Resolved: 09/Apr/04 16:50

Description

DOMElement::setIdAttribute*() leaks when repeatedly called for an attribute that
is already set as an ID.

More specifically, DOMAttrImpl::addAttrToIDNodeMap() adds the attribute to the
DOMNodeIDMap whether or not it is already there.

The fix is:
File: src/xercesc/dom/impl/DOMAttrImpl.hpp
Function: inline void DOMAttrImpl::addAttrToIDNodeMap()
Insert at the start of the function:
if (fNode.isIdAttr())
return;

– merlin