[freenet-cvs] r20560 - trunk/freenet/src/freenet/clients/http

[nextg...@freenetproject.org]

Author: nextgens
Date: 2008-06-21 12:22:28 +0000 (Sat, 21 Jun 2008)
New Revision: 20560

Modified:
   trunk/freenet/src/freenet/clients/http/QueueToadlet.java
Log:
backport r20556 to trunk (Fix NPE in registerAlert)

Modified: trunk/freenet/src/freenet/clients/http/QueueToadlet.java
===================================================================
--- trunk/freenet/src/freenet/clients/http/QueueToadlet.java    2008-06-21 
12:22:10 UTC (rev 20559)
+++ trunk/freenet/src/freenet/clients/http/QueueToadlet.java    2008-06-21 
12:22:28 UTC (rev 20560)
@@ -1309,6 +1309,14 @@

        private void registerAlert(ClientRequest req) {
                final String identifier = req.getIdentifier();
+               boolean logMINOR = Logger.shouldLog(Logger.MINOR, this);
+               if(logMINOR)
+                       Logger.minor(this, "Registering alert for "+identifier);
+               if(!req.hasFinished()) {
+                       if(logMINOR)
+                               Logger.minor(this, "Request hasn't finished: 
"+req+" for "+identifier, new Exception("debug"));
+                       return;
+               }
                if(req instanceof ClientGet) {
                        FreenetURI uri = ((ClientGet)req).getURI();
                        long size = ((ClientGet)req).getDataSize();
@@ -1344,6 +1352,10 @@
                                return;
                        }
                        long size = ((ClientPut)req).getDataSize();
+                       if(uri == null) {
+                               Logger.error(this, "uri is null for "+req+" for 
"+identifier);
+                               return;
+                       }
                        String name = uri.getPreferredFilename();
                        String title = l10n("uploadSucceededTitle", "filename", 
name);
                        HTMLNode text = new HTMLNode("div");