Commit 912ae2b8 authored by Nick Mathewson's avatar Nick Mathewson 🤹
Browse files

Merge remote-tracking branch 'teor/bug27463-029' into maint-0.2.9

parents 1570f17f 9d5c6317

changes/bug27463

0 → 100644
+3 −0
Original line number Diff line number Diff line 
  o Minor bugfixes (onion services):

    - Silence a spurious compiler warning in rend_client_send_introduction().

      Fixes bug 27463; bugfix on 0.1.1.2-alpha.

src/or/rendclient.c

+9 −0
Original line number Diff line number Diff line
@@ -269,6 +269,15 @@ rend_client_send_introduction(origin_circuit_t *introcirc, 
    dh_offset = v3_shift+7+DIGEST_LEN+2+klen+REND_COOKIE_LEN;

  } else {

    /* Version 0. */



    /* Some compilers are smart enough to work out that nickname can be more

     * than 19 characters, when it's a hexdigest. They warn that strncpy()

     * will truncate hexdigests without NUL-terminating them. But we only put

     * hexdigests in HSDir and general circuit exits. */

    if (BUG(strlen(rendcirc->build_state->chosen_exit->nickname)

            > MAX_NICKNAME_LEN)) {

      goto perm_err;

    }

    strncpy(tmp, rendcirc->build_state->chosen_exit->nickname,

            (MAX_NICKNAME_LEN+1)); /* nul pads */

    memcpy(tmp+MAX_NICKNAME_LEN+1, rendcirc->rend_data->rend_cookie,