Author: sebb
Date: Tue Sep  4 08:43:54 2007
New Revision: 572719

URL: http://svn.apache.org/viewvc?rev=572719&view=rev
Log:
Add fix for Derby NPE when no ? markers provided

Modified:
    
jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java

Modified: 
jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java?rev=572719&r1=572718&r2=572719&view=diff
==============================================================================
--- 
jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java
 (original)
+++ 
jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java
 Tue Sep  4 08:43:54 2007
@@ -258,7 +258,11 @@
                        String argument = arguments[i];
                        String argumentType = argumentsTypes[i];
                    int targetSqlType = getJdbcType(argumentType);
-                   pstmt.setObject(i+1, argument, targetSqlType);
+                   try {
+                               pstmt.setObject(i+1, argument, targetSqlType);
+                       } catch (NullPointerException e) { // thrown by Derby 
JDBC (at least) if there are no "?" markers in statement
+                               throw new SQLException("Could not set argument 
no: "+(i+1)+" - missing parameter marker?");
+                       }
                }
        }
     



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to