Changeset b9726a0 for router


Ignore:
Timestamp:
May 2, 2019 2:08:25 PM (11 months ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
7f60ee9
Parents:
bb86c56
Message:

UPnP: Remove finalize() in HTTPSocket (ticket #2490)

Location:
router/java/src
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • router/java/src/net/i2p/router/RouterVersion.java

    rbb86c56 rb9726a0  
    1919    public final static String ID = "Monotone";
    2020    public final static String VERSION = CoreVersion.VERSION;
    21     public final static long BUILD = 10;
     21    public final static long BUILD = 11;
    2222
    2323    /** for example "-test" */
  • router/java/src/org/cybergarage/http/HTTPServerThread.java

    rbb86c56 rb9726a0  
    4141        {
    4242                HTTPSocket httpSock = new HTTPSocket(sock);
    43                 if (httpSock.open() == false)
    44                         return;
    45                 HTTPRequest httpReq = new HTTPRequest();
    46                 httpReq.setSocket(httpSock);
    47                 while (httpReq.read() == true) {
    48                         httpServer.performRequestListener(httpReq);
    49                         if (httpReq.isKeepAlive() == false)
    50                                 break;
     43                try {
     44                        if (httpSock.open() == false)
     45                                return;
     46                        HTTPRequest httpReq = new HTTPRequest();
     47                        httpReq.setSocket(httpSock);
     48                        while (httpReq.read() == true) {
     49                                httpServer.performRequestListener(httpReq);
     50                                if (httpReq.isKeepAlive() == false)
     51                                        break;
     52                        }
     53                } finally {
     54                        httpSock.close();
    5155                }
    52                 httpSock.close();
    5356        }
    5457}
  • router/java/src/org/cybergarage/http/HTTPSocket.java

    rbb86c56 rb9726a0  
    2828package org.cybergarage.http;
    2929
     30import java.io.IOException;
    3031import java.io.InputStream;
    3132import java.io.OutputStream;
     
    5253        }
    5354       
    54         public void finalize()
    55         {
    56                 close();
    57         }
    58        
    5955        ////////////////////////////////////////////////
    6056        //      Socket
     
    134130        public boolean close()
    135131        {
    136                 try {
    137                         if (sockIn != null)
     132                if (sockIn != null)
     133                        try {
    138134                                sockIn.close();
    139                         if (sockOut != null)
     135                        } catch (IOException e) {}
     136                if (sockOut != null)
     137                        try {
    140138                                sockOut.close();
    141                         getSocket().close();
    142                 }
    143                 catch (Exception e) {
    144                         //Debug.warning(e);
    145                         return false;
    146                 }
     139                        } catch (IOException e) {}
     140                if (socket != null)
     141                        try {
     142                                socket.close();
     143                        } catch (IOException e) {}
    147144                return true;
    148145        }
Note: See TracChangeset for help on using the changeset viewer.