archive-access-cvs

[Archive-access-cvs] SF.net SVN: archive-access:[3599] trunk/archive-access/projects/ archive-commons/src/main/java/org/archive/resource/http/ HTTPResponseResource.java

[<vin...@us...>]

Revision: 3599
          http://archive-access.svn.sourceforge.net/archive-access/?rev=3599&view=rev
Author:   vinaygoel
Date:     2012-01-20 03:15:33 +0000 (Fri, 20 Jan 2012)
Log Message:
-----------
BUGFIX: Passing only non-negative length values to LimitInputStream method in HTTPResponseResource.java

Modified Paths:
--------------
    trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java

Modified: trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java
===================================================================
--- trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java	2012-01-19 21:44:58 UTC (rev 3598)
+++ trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java	2012-01-20 03:15:33 UTC (rev 3599)
@@ -64,7 +64,7 @@
 		for(HttpHeader h : response.getHeaders()) {
 			headers.putString(h.getName(),h.getValue());
 		}
-		if(forceCheck && (length != -1)) {
+		if(forceCheck && (length != -1) && (length >= 0)) {
 			LimitInputStream lis = new LimitInputStream(response, length);
 			countingIS = new CountingInputStream(lis);
 		} else {

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

[Archive-access-cvs] SF.net SVN: archive-access:[3600] trunk/archive-access/projects/ archive-commons/src/main/java/org/archive/resource/http/ HTTPResponseResource.java

[<vin...@us...>]

Revision: 3600
          http://archive-access.svn.sourceforge.net/archive-access/?rev=3600&view=rev
Author:   vinaygoel
Date:     2012-01-20 20:13:07 +0000 (Fri, 20 Jan 2012)
Log Message:
-----------
BUGFIX: Passing only non-negative length values to LimitInputStream method in HTTPResponseResource.java (removed redundant check)

Modified Paths:
--------------
    trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java

Modified: trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java
===================================================================
--- trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java	2012-01-20 03:15:33 UTC (rev 3599)
+++ trunk/archive-access/projects/archive-commons/src/main/java/org/archive/resource/http/HTTPResponseResource.java	2012-01-20 20:13:07 UTC (rev 3600)
@@ -64,7 +64,7 @@
 		for(HttpHeader h : response.getHeaders()) {
 			headers.putString(h.getName(),h.getValue());
 		}
-		if(forceCheck && (length != -1) && (length >= 0)) {
+		if(forceCheck && (length >= 0)) {
 			LimitInputStream lis = new LimitInputStream(response, length);
 			countingIS = new CountingInputStream(lis);
 		} else {

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.