Safer to check sub should be an object

moscajs · Aug 11, 2019 · 2df107b · 2df107b
1 parent 5bc48bd
commit 2df107b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/handlers/subscribe.js b/lib/handlers/subscribe.js
@@ -90,7 +90,7 @@ function blockDollarSignTopics (func) {
 }
 
 function storeSubscriptions (sub, done) {
-  if (!sub) {
+  if (!sub || typeof sub !== 'object') {
     return done(null, null)
   }
 
@@ -112,7 +112,7 @@ function storeSubscriptions (sub, done) {
 }
 
 function subTopic (sub, done) {
-  if (!sub) {
+  if (!sub || typeof sub !== 'object') {
     this.granted = 128
     return done()
   }