Phil Steitz <> changed:

           What    |Removed                     |Added
                 OS|                            |All

--- Comment #2 from Phil Steitz <> ---
This is a regression from the generics conversion in PoolableConnectionFactory.

The original DBCP 1.x code effectively null-checked the object to be destroyed:
    public void destroyObject(Object obj) throws Exception {
        if(obj instanceof PoolableConnection) {

Removing the instanceOf check makes NPE possible:
    public void destroyObject(PoolableConnection obj) throws Exception {

Solution is to add an explicit null check in destroyObject.

Similar changes should be made to activate, passivate, validate methods.

You are receiving this mail because:
You are the assignee for the bug.
To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to