[OPENMEETINGS-1595] couple NPEs are fixed

Project: http://git-wip-us.apache.org/repos/asf/openmeetings/repo
Commit: http://git-wip-us.apache.org/repos/asf/openmeetings/commit/b13eb4c1
Tree: http://git-wip-us.apache.org/repos/asf/openmeetings/tree/b13eb4c1
Diff: http://git-wip-us.apache.org/repos/asf/openmeetings/diff/b13eb4c1

Branch: refs/heads/3.2.x
Commit: b13eb4c1245bf83103cfc0f048b6bd3f122a6252
Parents: daafaa8
Author: Maxim Solodovnik <[email protected]>
Authored: Fri Mar 17 05:07:08 2017 +0000
Committer: Maxim Solodovnik <[email protected]>
Committed: Fri Mar 17 05:07:08 2017 +0000

----------------------------------------------------------------------
 .../db/dao/record/RecordingMetaDataDao.java            | 13 ++++++-------
 .../org/apache/openmeetings/web/common/MainPanel.java  |  8 +++++---
 2 files changed, 11 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/openmeetings/blob/b13eb4c1/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/record/RecordingMetaDataDao.java
----------------------------------------------------------------------
diff --git 
a/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/record/RecordingMetaDataDao.java
 
b/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/record/RecordingMetaDataDao.java
index 33d8665..d7188ee 100644
--- 
a/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/record/RecordingMetaDataDao.java
+++ 
b/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/record/RecordingMetaDataDao.java
@@ -131,13 +131,12 @@ public class RecordingMetaDataDao {
                try {
                        RecordingMetaData meta = get(metaId);
 
-                       meta.setRecordEnd(recordEnd);
-                       
-                       log.debug("updateEndDate :: Start Date :" + 
meta.getRecordStart());
-                       log.debug("updateEndDate :: End Date :" + 
meta.getRecordEnd());
-                       
-                       update(meta);
-                       
+                       if (meta != null) {
+                               meta.setRecordEnd(recordEnd);
+                               log.debug("updateEndDate :: Start Date :" + 
meta.getRecordStart());
+                               log.debug("updateEndDate :: End Date :" + 
meta.getRecordEnd());
+                               update(meta);
+                       }
                        return metaId;
                } catch (Exception ex2) {
                        log.error("[updateEndDate]: ", ex2);

http://git-wip-us.apache.org/repos/asf/openmeetings/blob/b13eb4c1/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/MainPanel.java
----------------------------------------------------------------------
diff --git 
a/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/MainPanel.java
 
b/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/MainPanel.java
index 7d7332d..3055c86 100644
--- 
a/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/MainPanel.java
+++ 
b/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/MainPanel.java
@@ -261,9 +261,11 @@ public class MainPanel extends Panel {
 
                        private void closeHandler(AbstractClientMessage msg) {
                                //no chance to stop pingTimer here :(
-                               log.debug("WebSocketBehavior::closeHandler 
[uid: {}, session: {}, key: {}]", client.getUid(), msg.getSessionId(), 
msg.getKey());
-                               exit(client);
-                               client = null;
+                               if (client != null) {
+                                       
log.debug("WebSocketBehavior::closeHandler [uid: {}, session: {}, key: {}]", 
client.getUid(), msg.getSessionId(), msg.getKey());
+                                       exit(client);
+                                       client = null;
+                               }
                        }
                });
                add(new OmAjaxClientInfoBehavior());

Reply via email to