#52 Buffer no longer performs length check for hex strings in node v8.x

Reviewed by: Alex Wilson <alex.wilson@joyent.com>
Approved by: Alex Wilson <alex.wilson@joyent.com>

Author: melloc

lib/fingerprint.js

@@ -102,7 +102,7 @@ Fingerprint.parse = function (fp, options) {
 		parts = parts.join('');
 		/*JSSTYLED*/
 		var md5RE = /^[a-fA-F0-9]+$/;
-		if (!md5RE.test(parts))
+		if (!md5RE.test(parts) || md5RE.length % 2 !== 0)
 			throw (new FingerprintFormatError(fp));
 		try {
 			hash = Buffer.from(parts, 'hex');

test/openssl-cmd.js

@@ -212,7 +212,12 @@ function genTests() {
 		var bufs = [];
 		kid.stdout.on('data', bufs.push.bind(bufs));
 		kid.on('close', function (rc) {
-			t.equal(rc, 0);
+			t.equal(rc, 0, 'openssl exited 0');
+			if (bufs.length === 0) {
+				t.fail('"openssl dgst" wrote no output');
+				t.end();
+				return;
+			}
 			var output = Buffer.concat(bufs);
 			var sig = sshpk.parseSignature(output, algo, 'asn1');
 			t.ok(sig);