[wasm] Check the input of grow-memory before calling the runtime.

test/mjsunit/wasm/grow-memory.js

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --expose-wasm --expose-gc --stress-compaction
 
 load("test/mjsunit/wasm/wasm-constants.js");
 load("test/mjsunit/wasm/wasm-module-builder.js");
 
 var kPageSize = 0x10000;
@@ skipping 99 matching lines @@
 function testGrowMemoryTrapMaxPages() {
   var builder = genGrowMemoryBuilder();
   builder.addMemory(1, 1, false);
   var module = builder.instantiate();
   var maxPages = 16384;
   function growMem(pages) { return module.exports.grow_memory(pages); }
   assertEquals(-1, growMem(maxPages));
 }
 
 testGrowMemoryTrapMaxPages();
+
+(function testGrowMemoryTrapsWithNonSmiInput() {
+  var builder = genGrowMemoryBuilder();
+  var module = builder.instantiate();
+  function growMem(pages) { return module.exports.grow_memory(pages); }
+  // The parameter of grow_memory is unsigned. Therefore -1 stands for
+  // UINT32_MIN, which cannot be represented as SMI.
+  assertEquals(-1, growMem(-1));
+})();

[gdeepti, 2016/08/30 02:19:56]

Nit: Test not consistent with the format of other tests in this file. Fix to be
consistent? 

[ahaas, 2016/08/30 06:49:53]

You are right, it should be consistent.

Done.