fixed issue TooTallNate#2 addMembership() should come after bind()

SamDecrock · May 31, 2012 · 363b924 · 363b924
1 parent e6ba669
commit 363b924
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/upnp.js b/lib/upnp.js
@@ -31,11 +31,11 @@ if (process.env.NODE_DEBUG && /upnp/.test(process.env.NODE_DEBUG)) {
 function ControlPoint() {
   events.EventEmitter.call(this);
   this.server = dgram.createSocket('udp4');
-  this.server.addMembership(BROADCAST_ADDR);
   var self = this;
   this.server.on('message', function(msg, rinfo) {self.onRequestMessage(msg, rinfo);});
   this._initParsers();
   this.server.bind(SSDP_PORT);
+  this.server.addMembership(BROADCAST_ADDR); //fixed issue #2
 }
 util.inherits(ControlPoint, events.EventEmitter);
 exports.ControlPoint = ControlPoint;