diff --git a/.travis.yml b/.travis.yml index 58649b7..5828bfb 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,10 +1,14 @@ before_script: - - "export DISPLAY=:99.0" - - "/sbin/start-stop-daemon --start --quiet --pidfile /tmp/custom_xvfb_99.pid --make-pidfile --background --exec /usr/bin/Xvfb -- :99 -nolisten $NOLISTEN" + - "export XAUTHORITY=/tmp/.Xauthority-Xvfb" + - "xauth add :99 . $(mcookie)" + - "xauth add 127.0.0.2:99 . $(mcookie)" + - "/sbin/start-stop-daemon --start --quiet --pidfile /tmp/custom_xvfb_99.pid --make-pidfile --background --exec /usr/bin/Xvfb -- :99 -nolisten $NOLISTEN -auth $XAUTHORITY" + - "sleep 1" env: - - NOLISTEN=tcp - - NOLISTEN=unix + - NOLISTEN=tcp DISPLAY=:99.0 + - NOLISTEN=unix DISPLAY=:99.0 + - NOLISTEN=unix DISPLAY=127.0.0.2:99.0 language: node_js node_js: diff --git a/lib/auth.js b/lib/auth.js index 264a3f2..1546e5d 100644 --- a/lib/auth.js +++ b/lib/auth.js @@ -1,4 +1,5 @@ -// TODO: http://en.wikipedia.org/wiki/X_Window_authorization +// TODO: differentiate between auth types (i.e., MIT-MAGIC-COOKIE-1 and XDM-AUTHORIZATION-1) +// and choose the best based on the algorithm in libXau's XauGetBestAuthByAddr var fs = require('fs'); var Buffer = require('buffer').Buffer; @@ -15,7 +16,7 @@ var typeToName = { 1: 'DECnet', 2: 'Chaos', 5: 'ServerInterpreted', - 6: 'InternetV6' + 6: 'Internet6' }; function parseXauth( buf ) @@ -27,7 +28,7 @@ function parseXauth( buf ) while (offset < buf.length) { var cookie = {}; - cookie.type = buf.readUInt16LE(offset); + cookie.type = buf.readUInt16BE(offset); if (!typeToName[cookie.type]) { console.warn('Unknown address type'); } @@ -73,8 +74,16 @@ function readXauthority(cb) { }); } -module.exports = function( display, host, cb ) +module.exports = function( display, host, socketFamily, cb ) { + var family; + if (socketFamily === 'IPv4') { + family = 0; // Internet + } else if (socketFamily === 'IPv6') { + family = 6; // Internet6 + } else { + family = 256; // Local + } readXauthority(function(err, data) { if(err) return cb(err); @@ -84,15 +93,18 @@ module.exports = function( display, host, cb ) authData: '' }); } - var auth = parseXauth(data); for (var cookieNum in auth) { var cookie = auth[cookieNum]; - if ((typeToName[cookie.family] === 'Wild' || cookie.address === host) && + if ((typeToName[cookie.family] === 'Wild' || (cookie.type === family && cookie.address === host)) && (cookie.display.length === 0 || cookie.display === display)) return cb( null, cookie ); } - cb(new Error('No auth cookie matching display=' + display + ' and host=' + host)); + // If no cookie is found, proceed without authentication + cb(null, { + authName: '', + authData: '' + }); }); }; diff --git a/lib/handshake.js b/lib/handshake.js index 45f08b0..254837b 100644 --- a/lib/handshake.js +++ b/lib/handshake.js @@ -90,7 +90,7 @@ function readScreens(bl, display, cbDisplayReady) if (display.screen.length == display.screen_num) { delete display.screen_num; - cbDisplayReady(display); + cbDisplayReady(null, display); return; } else { readScreens(bl, display, cbDisplayReady); @@ -189,9 +189,9 @@ function getByteOrder() { } } -function writeClientHello(stream, displayNum, authHost) +function writeClientHello(stream, displayNum, authHost, authFamily) { - getAuthString( displayNum, authHost, function( err, cookie ) { + getAuthString( displayNum, authHost, authFamily, function( err, cookie ) { if (err) { throw err; } diff --git a/lib/xcore.js b/lib/xcore.js index f4744a7..cf073e4 100644 --- a/lib/xcore.js +++ b/lib/xcore.js @@ -31,7 +31,6 @@ function XClient(displayNum, screenNum, options) this.displayNum = displayNum; this.screenNum = screenNum; - this.authHost = os.hostname(); } util.inherits(XClient, EventEmitter); @@ -39,6 +38,14 @@ XClient.prototype.init = function(stream) { this.stream = stream; + this.authHost = stream.remoteAddress; + // Node v0.10.x does not have stream.remoteFamily, so dig in to find it + this.authFamily = stream._getpeername ? stream._getpeername().family : stream.remoteFamily; + if (!this.authHost || this.authHost === '127.0.0.1' || this.authHost === '::1') { + this.authHost = os.hostname(); + this.authFamily = null; + } + var pack_stream = new PackStream(); // data received from stream is dispached to @@ -524,11 +531,13 @@ XClient.prototype.expectReplyHeader = function() XClient.prototype.startHandshake = function() { var client = this; - handshake.writeClientHello(this.pack_stream, this.displayNum, this.authHost); - handshake.readServerHello(this.pack_stream, function(display) + handshake.writeClientHello(this.pack_stream, this.displayNum, this.authHost, this.authFamily); + handshake.readServerHello(this.pack_stream, function(err, display) { - // TODO: readServerHello can set error state in display - // emit error in that case + if (err) { + client.emit('error', err); + return; + } client.expectReplyHeader(); client.display = display; display.client = client; @@ -560,8 +569,6 @@ module.exports.createClient = function(options, initCb) throw new Error("Cannot parse display"); var host = displayMatch[1]; - if (!host) - host = '127.0.0.1'; var displayNum = displayMatch[2]; if (!displayNum) @@ -586,10 +593,9 @@ module.exports.createClient = function(options, initCb) { socketPath = display; } - } else if(host == '127.0.0.1') //TODO check if it's consistent with xlib (DISPLAY=127.0.0.1:0 -> local unix socket or port 6000?) + } else if(!host) socketPath = '/tmp/.X11-unix/X' + displayNum; } - //socketPath = '/tmp/.X11-unix/X' + displayNum; var client = new XClient(displayNum, screenNum, options); var connectStream = function() {