Clear elements to avoid leaking memory. Close jquerygh-837. Fixes #11809

gibson042 · rwaldron · commit b6d02735e955 · 2012-06-26T14:49:56.000-04:00
diff --git a/src/manipulation.js b/src/manipulation.js
@@ -341,6 +341,9 @@ jQuery.fn.extend({
 				}
 			}
 
+			// Fix #11809: Avoid leaking memory
+			fragment = first = null;
+
 			if ( scripts.length ) {
 				jQuery.each( scripts, function( i, elem ) {
 					if ( elem.src ) {
@@ -708,7 +711,7 @@ jQuery.extend({
 		// Fix #11356: Clear elements from safeFragment
 		if ( div ) {
 			safe.removeChild( div );
-			div = safe = null;
+			elem = div = safe = null;
 		}
 
 		// Reset defaultChecked for any radios and checkboxes

Original file line number	Diff line number	Diff line change
`@@ -341,6 +341,9 @@ jQuery.fn.extend({`
`341`	`341`	`}`
`342`	`342`	`}`
`343`	`343`
	`344`	`+ // Fix #11809: Avoid leaking memory`
	`345`	`+ fragment = first = null;`
	`346`	`+`
`344`	`347`	`if ( scripts.length ) {`
`345`	`348`	`jQuery.each( scripts, function( i, elem ) {`
`346`	`349`	`if ( elem.src ) {`
`@@ -708,7 +711,7 @@ jQuery.extend({`
`708`	`711`	`// Fix #11356: Clear elements from safeFragment`
`709`	`712`	`if ( div ) {`
`710`	`713`	`safe.removeChild( div );`
`711`		`- div = safe = null;`
	`714`	`+ elem = div = safe = null;`
`712`	`715`	`}`
`713`	`716`
`714`	`717`	`// Reset defaultChecked for any radios and checkboxes`