[tbb-commits] [torbutton/master] Bug 16990: Avoid matching '250 ' to the end of node name

[gk at torproject.org]

commit 55724056112b1fd6bc2a42046a42dc265e1e44ea
Author: Arthur Edelstein <arthuredelstein at gmail.com>
Date:   Thu Nov 5 08:15:16 2015 -0800

    Bug 16990: Avoid matching '250 ' to the end of node name
---
 src/modules/tor-control-port.js |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/modules/tor-control-port.js b/src/modules/tor-control-port.js
index de9f10a..2b41897 100644
--- a/src/modules/tor-control-port.js
+++ b/src/modules/tor-control-port.js
@@ -517,11 +517,11 @@ info.stringToValue = function (string) {
       key = matchForKey ? matchForKey[1] : null;
   if (key === null) return null;
   // matchResult finds a single-line result for `250-` or a multi-line one for `250+`.
-  let matchResult = string.match(/250[ -].+?=(.*?)$/mi) ||
-                    string.match(/250\+.+?=([\s\S]*?)^\.$/mi),
+  let matchResult = string.match(/^250[ -].+?=(.*?)$/mi) ||
+                    string.match(/^250\+.+?=([\s\S]*?)^\.$/mi),
       // Retrieve the captured group (the text of the value in the key-value pair)
       valueString = matchResult ? matchResult[1] : null,
-      // Get the parser functino for the key found.
+      // Get the parser function for the key found.
       parse = info.getParser(key.toLowerCase());
   if (parse === undefined) {
     throw new Error("No parser found for '" + key + "'");