fix for NPE on session lookup which causes test failures

jonfreedman · Dec 1, 2017 · 67f5dbe · 67f5dbe
1 parent ee3101e
commit 67f5dbe
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/quickfixj-core/src/main/java/quickfix/MemoryStore.java b/quickfixj-core/src/main/java/quickfix/MemoryStore.java
@@ -112,8 +112,8 @@ public void setNextTargetMsgSeqNum(int next) throws IOException {
     public void refresh() throws IOException {
         // IOException is declared to maintain strict compatibility with QF JNI
         final String text = "memory store does not support refresh!";
-        if (sessionID != null) {
-            Session session = Session.lookupSession(sessionID);
+        final Session session = sessionID != null ? Session.lookupSession(sessionID) : null;
+        if (session != null) {
             session.getLog().onErrorEvent("ERROR: " + text);
         } else {
             LoggerFactory.getLogger(MemoryStore.class).error(text);