You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jakarta.apache.org by se...@apache.org on 2010/07/08 11:42:13 UTC
svn commit: r961682 -
/jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
Author: sebb
Date: Thu Jul 8 09:42:13 2010
New Revision: 961682
URL: http://svn.apache.org/viewvc?rev=961682&view=rev
Log:
Properly fix NPE
Modified:
jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
Modified: jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java?rev=961682&r1=961681&r2=961682&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java (original)
+++ jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java Thu Jul 8 09:42:13 2010
@@ -1072,9 +1072,11 @@ public abstract class HTTPSamplerBase ex
// Iterate through the URLs and download each image:
if (urls != null && urls.hasNext()) {
- if (container != null) { // prevent NPE TODO incomplete solution
- res = container;
+ if (container == null) {
+ container = new HTTPSampleResult(res);
+ container.addRawSubResult(res);
}
+ res = container;
// Get the URL matcher
String re=getEmbeddedUrlRE();
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@jakarta.apache.org
For additional commands, e-mail: notifications-help@jakarta.apache.org