Opened 4 months ago

Closed 4 months 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: Sensitive: no

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 4 months ago.
ANRs___crashes_-_I2P_-_Google_Play_Console.2.png (96.9 KB) - added by Meeh 4 months ago.

Download all attachments as: .zip

Change History (5)

comment:2 Changed 4 months 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 4 months ago by zzz

Milestone: undecided0.9.40
Resolution: fixed
Status: newclosed

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

Note: See TracTickets for help on using tickets.