From 4cde33c1f65aa1c75e4dd06df612ba8fe895e310 Mon Sep 17 00:00:00 2001
From: Artur Signell <artur.signell@itmill.com>
Date: Thu, 25 Jun 2009 14:35:58 +0000
Subject: [PATCH] Fixed obvious NPE in file upload

svn changeset:8243/svn branch:6.0
---
 .../vaadin/terminal/gwt/server/CommunicationManager.java  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/com/vaadin/terminal/gwt/server/CommunicationManager.java b/src/com/vaadin/terminal/gwt/server/CommunicationManager.java
index 6c74f4d4db..dc83e26e9b 100644
--- a/src/com/vaadin/terminal/gwt/server/CommunicationManager.java
+++ b/src/com/vaadin/terminal/gwt/server/CommunicationManager.java
@@ -166,14 +166,14 @@ public class CommunicationManager implements Paintable.RepaintRequestListener,
                     final String pid = name.split("_")[0];
                     final Upload uploadComponent = (Upload) idPaintableMap
                             .get(pid);
-                    if (uploadComponent.isReadOnly()) {
-                        throw new FileUploadException(
-                                "Warning: ignored file upload because upload component is set as read-only");
-                    }
                     if (uploadComponent == null) {
                         throw new FileUploadException(
                                 "Upload component not found");
                     }
+                    if (uploadComponent.isReadOnly()) {
+                        throw new FileUploadException(
+                                "Warning: ignored file upload because upload component is set as read-only");
+                    }
                     synchronized (application) {
                         // put upload component into receiving state
                         uploadComponent.startUpload();
-- 
2.39.5