fix: wrong assumption that the timeoutId argument to clearTimeout is always a JS::Value of type JSVAL_TYPE_INT32

Author: Xmader

src/internalBinding/timers.cc

@@ -44,13 +44,13 @@ static bool cancelByTimeoutId(JSContext *cx, unsigned argc, JS::Value *vp) {
 
   args.rval().setUndefined();
 
-  // silently does nothing when an invalid timeoutID is passed in
-  if (!timeoutIdArg.isInt32()) {
+  // silently does nothing when an invalid timeoutID (should be an int32 value) is passed in
+  if (!timeoutIdArg.isNumber()) {
     return true;
   }
 
   // Retrieve the AsyncHandle by `timeoutID`
-  int32_t timeoutID = timeoutIdArg.toInt32();
+  double timeoutID = timeoutIdArg.toNumber();
   AsyncHandle *handle = AsyncHandle::fromId((uint32_t)timeoutID);
   if (!handle) return true; // does nothing on invalid timeoutID