Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Trac Trac
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Issues 246
    • Issues 246
    • List
    • Boards
    • Service Desk
    • Milestones
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
  • Wiki
    • Wiki
  • Activity
  • Create a new issue
  • Issue Boards
Collapse sidebar
  • Legacy
  • TracTrac
  • Issues
  • #5324

Closed (moved)
(moved)
Open
Created Mar 06, 2012 by Roger Dingledine@arma

connection_handle_read_impl() has wrong function comment?

connection_handle_read_impl() says

 * This function gets called from conn_read() in main.c, either
 * when poll() has declared that conn wants to read, or (for OR conns)
 * when there are pending TLS bytes.

First of all, there is no conn_read() anymore. But also, I suspect this function is no longer called when there are pending TLS bytes?

I worry that libevent only calls the callback when the socket has something to read, and not when the ssl object has pending bytes. That would mean we are forced to keep our "read the whole ssl record even if it means our read bucket goes negative" feature? See #5323 (moved).

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking