Raise a TypeError if user tries to push an integer from JS side to C/C++ function exported with embind and the integer (char/short/int/long) is out of bounds of the data type expected by the C++ function.

1 files changed, 8 insertions, 1 deletions

diff --git a/src/embind/embind.js b/src/embind/embind.js
index c7b1633a..ccce9f19 100755
--- a/src/embind/embind.js
+++ b/src/embind/embind.js
@@ -221,10 +221,14 @@ function __embind_register_bool(rawType, name, trueValue, falseValue) {
     });
 }
 
-function __embind_register_integer(rawType, name) {
+// When converting a number from JS to C++ side, the valid range of the number is
+// [minRange, maxRange], inclusive.
+function __embind_register_integer(primitiveType, name, minRange, maxRange) {
     name = Pointer_stringify(name);
     registerType(rawType, {
         name: name,
+        minRange: minRange,
+        maxRange: maxRange,
         fromWireType: function(value) {
             return value;
         },
@@ -232,6 +236,9 @@ function __embind_register_integer(rawType, name) {
             if (typeof value !== "number") {
                 throw new TypeError('Cannot convert "' + _embind_repr(value) + '" to ' + this.name);
             }
+            if (value < minRange || value > maxRange) {
+                throw new TypeError('Passing a number "' + _embind_repr(value) + '" from JS side to C/C++ side to an argument of type "' + name + '", which is outside the valid range [' + minRange + ', ' + maxRange + ']!');
+            }
             return value | 0;
         },
     });