Fix a logic error in deciding whether to accept SessionGroup=

Fixes bug 22619; bugfix on 0.2.7.2-alpha

diff --git a/changes/bug22619 b/changes/bug22619
new file mode 100644 (file)
index 0000000..9c71996
--- /dev/null
+++ b/changes/bug22619
@@ -0,0 +1,3 @@
+  o Minor bugfixes (circuit isolation):
+    - Fix a logic error that prevented the SessionGroup sub-option from
+      being accepted. Fixes bug 22619; bugfix on 0.2.7.2-alpha.

diff --git a/src/app/config/config.c b/src/app/config/config.c
index 2a504d3065f1aed084e95726cc47fff9a467b12c..352559759115c1004113294053aeb9726b62ddff 100644 (file)
--- a/src/app/config/config.c
+++ b/src/app/config/config.c
@@ -7080,7 +7080,7 @@ parse_port_config(smartlist_t *out,
         if (!strcasecmpstart(elt, "SessionGroup=")) {
           int group = (int)tor_parse_long(elt+strlen("SessionGroup="),
                                           10, 0, INT_MAX, &ok, NULL);
-          if (!ok || !allow_no_stream_options) {
+          if (!ok || allow_no_stream_options) {
             log_warn(LD_CONFIG, "Invalid %sPort option '%s'",
                      portname, escaped(elt));
             goto err;