Bad practice with datasource?

[Gary Seibold]

Is this a bad practice when using datasources? I am thinking of connection pooling and wondering if there is a connection left open.

dataSource = (DataSource) new InitialContext().lookup(JDBC_DATA_SOURCE);
stmt = dataSource.getConnection().createStatement();
//do stuff
if(stmt != null)
   stmt.close();

[James Carman]

You need to close your connections, too.

[Jeanne Boyarsky]

Gary,
Also, you should close the statement and connection in a finally block.