July 2018 - tor-commits - lists.torproject.org

[tor/master] Merge remote-tracking branch 'dgoulet/bug26523_033_01' into maint-0.3.3
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit 9aeef05f8f56f16c9ff7e147ca313cc7bab88eda Merge: 72ee0b6f7 3f8a12a63 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun Jul 1 10:22:18 2018 -0400 Merge remote-tracking branch 'dgoulet/bug26523_033_01' into maint-0.3.3 changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-)

1 0

[tor/maint-0.3.4] Merge branch 'maint-0.3.3' into maint-0.3.4
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit cf8c3abff179b4550404bb9978a9d2b5ad18fc34 Merge: e042727cf 9aeef05f8 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun Jul 1 10:22:21 2018 -0400 Merge branch 'maint-0.3.3' into maint-0.3.4 changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-)

1 0

[tor/maint-0.3.4] control: Make HSPOST properly parse HSADDRESS= param
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit 3f8a12a63a5a1d8b4d40451586377423c95bda09 Author: David Goulet <dgoulet(a)torproject.org> Date: Wed Jun 27 12:20:39 2018 -0400 control: Make HSPOST properly parse HSADDRESS= param For HSv3, the HSADDRESS= wasn't properly parsed for the HSPOST command. It now correctly use it and furthermore sends back a "200 OK" in case the command is successful for a v3 descriptor. Fixes #26523 Signed-off-by: David Goulet <dgoulet(a)torproject.org> --- changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/changes/bug26523 b/changes/bug26523 new file mode 100644 index 000000000..a739d240e --- /dev/null +++ b/changes/bug26523 @@ -0,0 +1,5 @@ + o Minor bugfixes (hidden service, control port): + - The HSPOST command wasn't parsing properly the HSADDRESS= parameter and + thus not using it. It now handles it correctly. Fixes bug 26523; bugfix on + 0.3.3.1-alpha. Patch by "akwizgran". + diff --git a/src/or/control.c b/src/or/control.c index 028339f49..06360586e 100644 --- a/src/or/control.c +++ b/src/or/control.c @@ -4342,8 +4342,10 @@ handle_control_hspost(control_connection_t *conn, smartlist_t *args = smartlist_new(); - /* If any SERVER= options were specified, try parse the options line */ - if (!strcasecmpstart(argline, opt_server)) { + /* If any SERVER= or HSADDRESS= options were specified, try to parse + * the options line. */ + if (!strcasecmpstart(argline, opt_server) || + !strcasecmpstart(argline, opt_hsaddress)) { /* encoded_desc begins after a newline character */ cp = cp + 1; encoded_desc = cp; @@ -4366,11 +4368,12 @@ handle_control_hspost(control_connection_t *conn, hs_dirs = smartlist_new(); smartlist_add(hs_dirs, node->rs); } else if (!strcasecmpstart(arg, opt_hsaddress)) { - if (!hs_address_is_valid(arg)) { + const char *address = arg + strlen(opt_hsaddress); + if (!hs_address_is_valid(address)) { connection_printf_to_buf(conn, "512 Malformed onion address\r\n"); goto done; } - onion_address = arg; + onion_address = address; } else { connection_printf_to_buf(conn, "512 Unexpected argument \"%s\"\r\n", arg); @@ -4385,6 +4388,8 @@ handle_control_hspost(control_connection_t *conn, read_escaped_data(encoded_desc, encoded_desc_len, &desc_str); if (hs_control_hspost_command(desc_str, onion_address, hs_dirs) < 0) { connection_printf_to_buf(conn, "554 Invalid descriptor\r\n"); + } else { + send_control_done(conn); } tor_free(desc_str); goto done;

1 0

[tor/maint-0.3.4] Merge remote-tracking branch 'dgoulet/bug26523_033_01' into maint-0.3.3
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit 9aeef05f8f56f16c9ff7e147ca313cc7bab88eda Merge: 72ee0b6f7 3f8a12a63 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun Jul 1 10:22:18 2018 -0400 Merge remote-tracking branch 'dgoulet/bug26523_033_01' into maint-0.3.3 changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-)

1 0

[tor/master] Merge branch 'maint-0.3.4'
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit adcd1d8b9ac09f3abc11e2e3187fe363ad3df2fd Merge: c03958dbc cf8c3abff Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun Jul 1 10:22:21 2018 -0400 Merge branch 'maint-0.3.4' changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-)

1 0

[tor/master] control: Make HSPOST properly parse HSADDRESS= param
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit 3f8a12a63a5a1d8b4d40451586377423c95bda09 Author: David Goulet <dgoulet(a)torproject.org> Date: Wed Jun 27 12:20:39 2018 -0400 control: Make HSPOST properly parse HSADDRESS= param For HSv3, the HSADDRESS= wasn't properly parsed for the HSPOST command. It now correctly use it and furthermore sends back a "200 OK" in case the command is successful for a v3 descriptor. Fixes #26523 Signed-off-by: David Goulet <dgoulet(a)torproject.org> --- changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/changes/bug26523 b/changes/bug26523 new file mode 100644 index 000000000..a739d240e --- /dev/null +++ b/changes/bug26523 @@ -0,0 +1,5 @@ + o Minor bugfixes (hidden service, control port): + - The HSPOST command wasn't parsing properly the HSADDRESS= parameter and + thus not using it. It now handles it correctly. Fixes bug 26523; bugfix on + 0.3.3.1-alpha. Patch by "akwizgran". + diff --git a/src/or/control.c b/src/or/control.c index 028339f49..06360586e 100644 --- a/src/or/control.c +++ b/src/or/control.c @@ -4342,8 +4342,10 @@ handle_control_hspost(control_connection_t *conn, smartlist_t *args = smartlist_new(); - /* If any SERVER= options were specified, try parse the options line */ - if (!strcasecmpstart(argline, opt_server)) { + /* If any SERVER= or HSADDRESS= options were specified, try to parse + * the options line. */ + if (!strcasecmpstart(argline, opt_server) || + !strcasecmpstart(argline, opt_hsaddress)) { /* encoded_desc begins after a newline character */ cp = cp + 1; encoded_desc = cp; @@ -4366,11 +4368,12 @@ handle_control_hspost(control_connection_t *conn, hs_dirs = smartlist_new(); smartlist_add(hs_dirs, node->rs); } else if (!strcasecmpstart(arg, opt_hsaddress)) { - if (!hs_address_is_valid(arg)) { + const char *address = arg + strlen(opt_hsaddress); + if (!hs_address_is_valid(address)) { connection_printf_to_buf(conn, "512 Malformed onion address\r\n"); goto done; } - onion_address = arg; + onion_address = address; } else { connection_printf_to_buf(conn, "512 Unexpected argument \"%s\"\r\n", arg); @@ -4385,6 +4388,8 @@ handle_control_hspost(control_connection_t *conn, read_escaped_data(encoded_desc, encoded_desc_len, &desc_str); if (hs_control_hspost_command(desc_str, onion_address, hs_dirs) < 0) { connection_printf_to_buf(conn, "554 Invalid descriptor\r\n"); + } else { + send_control_done(conn); } tor_free(desc_str); goto done;

1 0

[tor/release-0.3.3] Merge branch 'maint-0.3.3' into release-0.3.3
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit b22b60365f2d7db2db31128fa733c67d6f52fb8c Merge: 103076a75 9aeef05f8 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun Jul 1 10:22:20 2018 -0400 Merge branch 'maint-0.3.3' into release-0.3.3 changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-)

1 0

[tor/release-0.3.4] Merge remote-tracking branch 'dgoulet/bug26523_033_01' into maint-0.3.3
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit 9aeef05f8f56f16c9ff7e147ca313cc7bab88eda Merge: 72ee0b6f7 3f8a12a63 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun Jul 1 10:22:18 2018 -0400 Merge remote-tracking branch 'dgoulet/bug26523_033_01' into maint-0.3.3 changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-)

1 0

[tor/master] Merge branch 'maint-0.3.3' into maint-0.3.4
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit cf8c3abff179b4550404bb9978a9d2b5ad18fc34 Merge: e042727cf 9aeef05f8 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun Jul 1 10:22:21 2018 -0400 Merge branch 'maint-0.3.3' into maint-0.3.4 changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-)

1 0

[tor/release-0.3.3] control: Make HSPOST properly parse HSADDRESS= param
by nickm＠torproject.org 01 Jul '18

01 Jul '18

commit 3f8a12a63a5a1d8b4d40451586377423c95bda09 Author: David Goulet <dgoulet(a)torproject.org> Date: Wed Jun 27 12:20:39 2018 -0400 control: Make HSPOST properly parse HSADDRESS= param For HSv3, the HSADDRESS= wasn't properly parsed for the HSPOST command. It now correctly use it and furthermore sends back a "200 OK" in case the command is successful for a v3 descriptor. Fixes #26523 Signed-off-by: David Goulet <dgoulet(a)torproject.org> --- changes/bug26523 | 5 +++++ src/or/control.c | 13 +++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/changes/bug26523 b/changes/bug26523 new file mode 100644 index 000000000..a739d240e --- /dev/null +++ b/changes/bug26523 @@ -0,0 +1,5 @@ + o Minor bugfixes (hidden service, control port): + - The HSPOST command wasn't parsing properly the HSADDRESS= parameter and + thus not using it. It now handles it correctly. Fixes bug 26523; bugfix on + 0.3.3.1-alpha. Patch by "akwizgran". + diff --git a/src/or/control.c b/src/or/control.c index 028339f49..06360586e 100644 --- a/src/or/control.c +++ b/src/or/control.c @@ -4342,8 +4342,10 @@ handle_control_hspost(control_connection_t *conn, smartlist_t *args = smartlist_new(); - /* If any SERVER= options were specified, try parse the options line */ - if (!strcasecmpstart(argline, opt_server)) { + /* If any SERVER= or HSADDRESS= options were specified, try to parse + * the options line. */ + if (!strcasecmpstart(argline, opt_server) || + !strcasecmpstart(argline, opt_hsaddress)) { /* encoded_desc begins after a newline character */ cp = cp + 1; encoded_desc = cp; @@ -4366,11 +4368,12 @@ handle_control_hspost(control_connection_t *conn, hs_dirs = smartlist_new(); smartlist_add(hs_dirs, node->rs); } else if (!strcasecmpstart(arg, opt_hsaddress)) { - if (!hs_address_is_valid(arg)) { + const char *address = arg + strlen(opt_hsaddress); + if (!hs_address_is_valid(address)) { connection_printf_to_buf(conn, "512 Malformed onion address\r\n"); goto done; } - onion_address = arg; + onion_address = address; } else { connection_printf_to_buf(conn, "512 Unexpected argument \"%s\"\r\n", arg); @@ -4385,6 +4388,8 @@ handle_control_hspost(control_connection_t *conn, read_escaped_data(encoded_desc, encoded_desc_len, &desc_str); if (hs_control_hspost_command(desc_str, onion_address, hs_dirs) < 0) { connection_printf_to_buf(conn, "554 Invalid descriptor\r\n"); + } else { + send_control_done(conn); } tor_free(desc_str); goto done;

1 0