Return FALSE if non-hex char detected

duzun · duzun · commit 3b3f61946d2c · 2014-10-25T15:00:02.000+03:00
Return FALSE if non-hex char detected - same behaviour as PHP version of hex2bin().
diff --git a/functions/strings/hex2bin.js b/functions/strings/hex2bin.js
@@ -5,13 +5,18 @@ function hex2bin(s) {
   //   returns 1: 'Dima'
   //   example 2: bin2hex('00');
   //   returns 2: '\x00'
+  //   example 3: bin2hex('2f1q')
+  //   returns 3: false
 
   var ret = [], i = 0, l;
 
   s += '';
 
   for ( l = s.length ; i < l; i+=2 ) {
-    ret.push(parseInt(s.substr(i, 2), 16));
+    var c = parseInt(s.substr(i, 1), 16);
+    var k = parseInt(s.substr(i+1, 1), 16);
+    if(isNaN(c) || isNaN(k)) return false;
+    ret.push( (c << 4) | k );
   }
 
   return String.fromCharCode.apply(String, ret);
