I have reviewed every use of checkSdkBinaryPresent() and I think that there are no problems:

* In adb.js:120, checkAdbPresent() calls to checkSdkBinaryPresent() and saves its response in this.adb and this.adbCmd. 
ADB.prototype.exec uses this.adbCmd with quotes and ADB.prototype.spawn removes quotes before use this.adb. Every use of this.adb and this.adbCmd assumes that they have quotes. 

* In adb.js:748, launchAVD() calls to checkSdkBinaryPresent() and the response is used in a spawn(). Quotes must be removed here: emulatorBinaryPath.substr(1, emulatorBinaryPath.length - 2)
Perhaps it would be better to use emulatorBinaryPath.replace(/"/g, ''), like in other methods.

* checkSdkBinaryPresent() is also used to find 'aapt' and 'zipalign' paths, but, in both cases, they are used in exec() and work well with quotes.