Add checking that arguments to be sent is actually an object

obs-websocket-community-projects · Mar 4, 2021 · ed726ff · ed726ff
1 parent 1703b84
commit ed726ff
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/lib/OBSWebSocket.js b/lib/OBSWebSocket.js
@@ -29,6 +29,10 @@ class OBSWebSocket extends Socket {
         rejectReason = Status.REQUEST_TYPE_NOT_SPECIFIED;
       }
 
+      if (args && (typeof args !== 'object' || args === null || Array.isArray(args))) {
+        rejectReason = Status.ARGS_NOT_OBJECT;
+      }
+
       if (!this._connected) {
         rejectReason = Status.NOT_CONNECTED;
       }

diff --git a/lib/Status.js b/lib/Status.js
@@ -19,6 +19,10 @@ module.exports = {
     status: 'error',
     description: 'A Request Type was not specified.'
   },
+  ARGS_NOT_OBJECT: {
+    status: 'error',
+    description: 'The supplied argments parameter is not an object.'
+  },
 
   init() {
     for (const key in this) {