Please pull this patch. It breaks JDBC1 support. The JDBC1 code no

longer compiles, due to objects being referenced in this patch that do
not exist in JDK1.1.

Barry Lind
---------------------------------------------------------------------------

The JDBC driver requires

  permission java.net.SocketPermission "host:port", "connect";

in the policy file of the application using the JDBC driver
in the postgresql.jar file.  Since the Socket() call in the
driver is not protected by AccessController.doPrivileged() this
permission must also be granted to the entire application.
This commit is contained in:
Bruce Momjian 2001-08-26 17:08:48 +00:00
parent e1635b4402
commit 3d162c0d1a
1 changed files with 2 additions and 28 deletions

View File

@ -5,13 +5,12 @@ import java.lang.*;
import java.net.*;
import java.util.*;
import java.sql.*;
import java.security.*;
import org.postgresql.*;
import org.postgresql.core.*;
import org.postgresql.util.*;
/**
* $Id: PG_Stream.java,v 1.12 2001/08/26 01:06:20 momjian Exp $
* $Id: PG_Stream.java,v 1.13 2001/08/26 17:08:48 momjian Exp $
*
* This class is used by Connection & PGlobj for communicating with the
* backend.
@ -29,25 +28,6 @@ public class PG_Stream
BytePoolDim1 bytePoolDim1 = new BytePoolDim1();
BytePoolDim2 bytePoolDim2 = new BytePoolDim2();
private static class PrivilegedSocket
implements PrivilegedExceptionAction
{
private String host;
private int port;
PrivilegedSocket(String host, int port)
{
this.host = host;
this.port = port;
}
public Object run() throws Exception
{
return new Socket(host, port);
}
}
/**
* Constructor: Connect to the PostgreSQL back end and return
* a stream connection.
@ -58,13 +38,7 @@ public class PG_Stream
*/
public PG_Stream(String host, int port) throws IOException
{
PrivilegedSocket ps = new PrivilegedSocket(host, port);
try {
connection = (Socket)AccessController.doPrivileged(ps);
}
catch(PrivilegedActionException pae){
throw (IOException)pae.getException();
}
connection = new Socket(host, port);
// Submitted by Jason Venner <jason@idiom.com> adds a 10x speed
// improvement on FreeBSD machines (caused by a bug in their TCP Stack)