Opened 6 years ago

Closed 6 years ago

#1041 closed defect (fixed)

Streaming disconnect locking

Reported by: zzz Owned by: zzz
Priority: minor Milestone: 0.9.9
Component: streaming Version: 0.9.7.1
Keywords: Cc: zab@…
Parent Tickets:

Description

In Connection.java, the various disconnect, close, and reset methods are unlocked and set non-volatile fields. Multiple DisconnectTimers? can be instantiated. Poor state management / tracking.

Subtickets

Change History (8)

comment:1 Changed 6 years ago by zab

  • Cc zab@… added

comment:2 Changed 6 years ago by zzz

  • Owner set to zzz
  • Status changed from new to accepted

Fixed in i2p.i2p.zzz.test2 91d569f040d8bf4664b02d0ab1355ccda9fa0e76 to be propped for 0.9.9

comment:3 Changed 6 years ago by zzz

Above commit was only for disconnect, not a complete fix.
For close/reset send/receive, more atomics in 82365833c3e240e13017988199bb1459b29d3377 same branch.

comment:8 Changed 6 years ago by zzz

related: #629

comment:9 Changed 6 years ago by zzz

Major changes propped as 0.9.8.1-2. Should be all of it but leaving ticket open for testing.

comment:10 Changed 6 years ago by str4d

  • Keywords testing-needed added

comment:11 Changed 6 years ago by str4d

  • Keywords testing-needed removed
  • Status changed from accepted to testing

comment:12 Changed 6 years ago by zzz

  • Resolution set to fixed
  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.