Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-12-04 19:33:18 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix leak by making currNode an nsCOMPtr (bug 407818). r+sr=dbaron, a=schrep.

Browse Source
This commit is contained in:
jruderman@hmc.edu 2007-12-11 17:37:51 -08:00
parent bc82216410
commit 5da8357591
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
content/base/src/nsHTMLContentSerializer.cpp
View File

@ -1264,9 +1264,8 @@ nsHTMLContentSerializer::SerializeLIValueAttribute(nsIDOMElement* aElement,
// We are copying and we are at the "first" LI node of OL in selected range.
// It may not be the first LI child of OL but it's first in the selected range.
// Note that we get into this condition only once per a OL.
nsCOMPtr<nsIDOMNode> node = do_QueryInterface(aElement);
PRBool found = PR_FALSE;
nsIDOMNode* currNode = node;
nsCOMPtr<nsIDOMNode> currNode = do_QueryInterface(aElement);
nsAutoString valueStr;
PRInt32 offset = 0;
olState defaultOLState(0, PR_FALSE);
@ -1299,7 +1298,9 @@ nsHTMLContentSerializer::SerializeLIValueAttribute(nsIDOMElement* aElement,
}
}
}
currNode->GetPreviousSibling(&currNode);
nsCOMPtr<nsIDOMNode> tmp;
currNode->GetPreviousSibling(getter_AddRefs(tmp));
currNode.swap(tmp);
}
// If LI was not having "value", Set the "value" attribute for it.
// Note that We are at the first LI in the selected range of OL.