Closed Bug 626758 Opened 13 years ago Closed 13 years ago

XMLObject::_toXMLString removes namespaces incorrectly

Categories

(Tamarin Graveyard :: Virtual Machine, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: stejohns, Assigned: stejohns)

Details

(Whiteboard: WE:2787930)

Attachments

(1 file)

quoth Werner:

this loop in XMLObject::_toXMLString is backwards:

  while (namespaceLength > newNamespaceArray.length())
     newNamespaceArray.removeLast();

it should be:

     while (newNamespaceArray.length() > namespaceLength)
         newNamespaceArray.removeLast();

prior to the NamespaceList change it was:

     newNamespaceArray->setLength (namespaceLength);
Whiteboard: WE:2787930
Attached patch PatchSplinter Review
Assignee: nobody → stejohns
Attachment #504831 - Flags: review?(rreitmai)
Attachment #504831 - Flags: feedback?(wsharp)
Attachment #504831 - Flags: feedback?(wsharp) → feedback+
Comment on attachment 504831 [details] [diff] [review]
Patch

r+ seems correct.
Attachment #504831 - Flags: review?(rreitmai) → review+
TR 5805:a191bb3a8ed3
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: