0004410: NPE in AbstractHttpResponse.fillCookies

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0004410

[Resin]

major

always

02-25-11 12:49

02-28-11 15:27

Reporter

brandonkearby

View Status

public

Assigned To

ferg

Priority

normal

Resolution

fixed

Status

closed

Product Version

4.0.15

Summary

0004410: NPE in AbstractHttpResponse.fillCookies

Description

When the cookie value is null an NPE is thrown. See additional info for the fix.

Additional Information

Subversion is a tool for version control.
For additional information, see http://subversion.tigris.org/ [^]
brandonmac:trunk brandon$ svn diff
Index: modules/resin/src/com/caucho/server/http/AbstractHttpResponse.java
===================================================================
--- modules/resin/src/com/caucho/server/http/AbstractHttpResponse.java (revision 7912)
+++ modules/resin/src/com/caucho/server/http/AbstractHttpResponse.java (working copy)
@@ -844,7 +844,7 @@
     else {
       cb.append("=");
       String v = cookie.getValue();
- int len = v.length();
+ int len = v == null ? 0 : v.length();

       for (int i = 0; i < len; i++) {
         char ch = v.charAt(i);

Attached Files

Relationships

Notes
(0005089) ferg 02-28-11 15:27	server/01ea

Issue History
Date Modified	Username	Field	Change
02-25-11 12:49	brandonkearby	New Issue
02-28-11 11:03	brandonkearby	Issue Monitored: brandonkearby
02-28-11 15:27	ferg	Note Added: 0005089
02-28-11 15:27	ferg	Assigned To	=> ferg
02-28-11 15:27	ferg	Status	new => closed
02-28-11 15:27	ferg	Resolution	open => fixed
02-28-11 15:27	ferg	Fixed in Version	=> 4.0.16

Mantis 1.0.0rc3[^]

30 total queries executed.
26 unique queries executed.