Add checks for MissingValue in memset

Cellule · Cellule · commit 26e3e74d6ae5 · 2016-05-03T17:31:12.000-07:00
diff --git a/lib/Runtime/Language/JavascriptOperators.cpp b/lib/Runtime/Language/JavascriptOperators.cpp
@@ -4685,7 +4685,13 @@ namespace Js
                 }
                 else if (instanceType == TypeIds_NativeIntArray)
                 {
-                    returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<int32>(start, length, JavascriptConversion::ToInt32(value, scriptContext));
+                    int32 intValue = JavascriptConversion::ToInt32(value, scriptContext);
+                    // Special case for missing item
+                    if (SparseArraySegment<int32>::IsMissingItem(&intValue))
+                    {
+                        return false;
+                    }
+                    returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<int32>(start, length, intValue);
                 }
                 else
                 {
@@ -4694,7 +4700,14 @@ namespace Js
                     {
                         return false;
                     }
-                    returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<double>(start, length, JavascriptConversion::ToNumber(value, scriptContext));
+
+                    double doubleValue = JavascriptConversion::ToNumber(value, scriptContext);
+                    // Special case for missing item
+                    if (SparseArraySegment<double>::IsMissingItem(&doubleValue))
+                    {
+                        return false;
+                    }
+                    returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<double>(start, length, doubleValue);
                 }
                 returnValue &= vt == VirtualTableInfoBase::GetVirtualTable(instance);
             }

Original file line number	Diff line number	Diff line change
`@@ -4685,7 +4685,13 @@ namespace Js`
`4685`	`4685`	`}`
`4686`	`4686`	`else if (instanceType == TypeIds_NativeIntArray)`
`4687`	`4687`	`{`
`4688`		`- returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<int32>(start, length, JavascriptConversion::ToInt32(value, scriptContext));`
	`4688`	`+ int32 intValue = JavascriptConversion::ToInt32(value, scriptContext);`
	`4689`	`+ // Special case for missing item`
	`4690`	`+ if (SparseArraySegment<int32>::IsMissingItem(&intValue))`
	`4691`	`+ {`
	`4692`	`+ return false;`
	`4693`	`+ }`
	`4694`	`+ returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<int32>(start, length, intValue);`
`4689`	`4695`	`}`
`4690`	`4696`	`else`
`4691`	`4697`	`{`
`@@ -4694,7 +4700,14 @@ namespace Js`
`4694`	`4700`	`{`
`4695`	`4701`	`return false;`
`4696`	`4702`	`}`
`4697`		`- returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<double>(start, length, JavascriptConversion::ToNumber(value, scriptContext));`
	`4703`	`+`
	`4704`	`+ double doubleValue = JavascriptConversion::ToNumber(value, scriptContext);`
	`4705`	`+ // Special case for missing item`
	`4706`	`+ if (SparseArraySegment<double>::IsMissingItem(&doubleValue))`
	`4707`	`+ {`
	`4708`	`+ return false;`
	`4709`	`+ }`
	`4710`	`+ returnValue = JavascriptArray::FromVar(instance)->DirectSetItemAtRange<double>(start, length, doubleValue);`
`4698`	`4711`	`}`
`4699`	`4712`	`returnValue &= vt == VirtualTableInfoBase::GetVirtualTable(instance);`
`4700`	`4713`	`}`