Changes between Version 2 and Version 3 of Ticket #1734


Ignore:
Timestamp:
Jan 22, 2009, 2:34:52 PM (12 years ago)
Author:
charles
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #1734 – Description

    v2 v3  
    11Beginning back with the "performance patch" submitted by Aloisus via the forums, Transmission has made a practice of buffering up non-critical BitTorrent protocol messages (such as `have' and periodic pex messages) and sending them in a batch to reduce socket-level packet overhead.
    22
    3 The bug is that these protocol messages are encrypted as they're written into the batch buffer.  If that is done, and then we encrypt and send piece data ''before'' the protocol messages batch is flushed, the order of the messages' encryption and the messages' transmission are reversed and the RC4 stream is corrupted.
     3The bug is that these protocol messages are encrypted as they're written into the batch buffer.  If that is done, and then we encrypt and send piece data ''before'' the protocol messages batch is flushed, then the order of the messages' encryption and transmission are reversed and the RC4 stream becomes corrupt.
    44
    55Special thanks goes to denisx and especially merlyn3d for helping track down this bug.