[tor-dev] [patch] properly test for OPENSSL_NO_COMP

Ian Goldberg iang at cs.uwaterloo.ca
Sun Jul 13 23:23:16 UTC 2014

On Sun, Jul 13, 2014 at 07:20:29PM -0400, grarpamp wrote:
> >    /* Don't actually allow compression; it uses ram and time, but the data
> >     * we transmit is all encrypted anyway. */
> >      result->ctx->comp_methods = NULL;
> This comment is confusing. Why are you asserting/mixing the two with
> the ', but' that 'encryption anyway' is excuse to not compress due to
> 'ram/time'? They are two separate things. Either you are encrypting
> compressed data, or encrypting uncompressed data.

It seems to me the intent of the comment is that the *plaintext* data
being transmitted is already encrypted (at another layer), and so is not
going to be compressible, so don't waste ram/time trying to do so.

   - Ian

More information about the tor-dev mailing list