Opened 3 weeks ago

Closed 3 weeks ago

#2490 closed defect (fixed)

Android org.cybergarage.http.HTTPSocket.close

Reported by: meeh Owned by: meeh
Priority: minor Milestone: 0.9.40
Component: apps/android Version: 0.9.39
Keywords: Cc:
Parent Tickets:

Description

java.util.concurrent.TimeoutException: 
  at org.cybergarage.http.HTTPSocket.close (HTTPSocket.java:141)
  at org.cybergarage.http.HTTPSocket.finalize (HTTPSocket.java:56)
  at java.lang.Daemons$FinalizerDaemon.doFinalize (Daemons.java:250)
  at java.lang.Daemons$FinalizerDaemon.runInternal (Daemons.java:237)
  at java.lang.Daemons$Daemon.run (Daemons.java:103)
  at java.lang.Thread.run (Thread.java:764)

Subtickets

Attachments (2)

ANRs___crashes_-_I2P_-_Google_Play_Console.png (103.7 KB) - added by meeh 3 weeks ago.
ANRs___crashes_-_I2P_-_Google_Play_Console.2.png (96.9 KB) - added by meeh 3 weeks ago.

Download all attachments as: .zip

Change History (5)

comment:2 Changed 3 weeks ago by zzz

OK, doesn't really throw, just logs. See source link line 266. We can skip the finalize on Android but if the socket still needs closing it could get caught up in a finalize anyway.

comment:3 Changed 3 weeks ago by zzz

  • Milestone changed from undecided to 0.9.40
  • Resolution set to fixed
  • Status changed from new to closed

Replaced finalize() in HTTPSocket with try/finally in HTTPServerThread
in eb9abeaf57def7b1b04cf203f46ea80321030a78 0.9.39-11-rc

Note: See TracTickets for help on using tickets.