[CASSANDRA-12329] Unreleased Resource: Sockets

• Type: Sub-task
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 3.0.10, 3.10
• Component/s: Legacy/Streaming and Messaging
• Labels: easyfix newbie patch
• Assignee: Arunkumar M
• Reporter: Eduardo Aguinaga

Authors

Arunkumar M

Reviewers

Yuki Morishita

• Created: 27/Jul/16 18:45
• Updated: 16/Apr/19 09:30
• Resolved: 20/Oct/16 14:49

Description

Overview:
In May through June of 2016 a static analysis was performed on version 3.0.5 of the Cassandra source code. The analysis included an automated analysis using HP Fortify v4.21 SCA and a manual analysis utilizing SciTools Understand v4. The results of that analysis includes the issue below.

Issue:
Sockets are low level resources that must be explicitly released so subsequent callers will have access to previously used sockets. In the file SSLFactory.java on line 62 a SSL server socket is acquired and eventually returned to the caller on line 69.

If an exception is thrown by any of the code between lines 62 and 69 the socket acquired on line 62 will not be released for subsequent reuse..

SSLFactory.java, lines 59-70:
59 public static SSLServerSocket getServerSocket(EncryptionOptions options, InetAddress address, int port) throws IOException
60 {
61     SSLContext ctx = createSSLContext(options, true);
62     SSLServerSocket serverSocket = (SSLServerSocket)ctx.getServerSocketFactory().createServerSocket();
63     serverSocket.setReuseAddress(true);
64     String[] suites = filterCipherSuites(serverSocket.getSupportedCipherSuites(), options.cipher_suites);
65     serverSocket.setEnabledCipherSuites(suites);
66     serverSocket.setNeedClientAuth(options.require_client_auth);
67     serverSocket.setEnabledProtocols(ACCEPTED_PROTOCOLS);
68     serverSocket.bind(new InetSocketAddress(address, port), 500);
69     return serverSocket;
70 }