[geode] branch develop updated: GEODE-6557:Handling a possible null situation.

[nn...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

nnag pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 8ff0939  GEODE-6557:Handling a possible null situation.
8ff0939 is described below

commit 8ff0939631ce216f366242b5f5624857df271aa3
Author: nabarunnag <na...@cs.wisc.edu>
AuthorDate: Mon Apr 15 23:21:45 2019 -0700

    GEODE-6557:Handling a possible null situation.
    
    	* There is a possibility that type can be null, which is handled in the return statement.
    	* Similar checks were placed in the logger trace statement.
---
 .../http/converter/ServerSerializableObjectHttpMessageConverter.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/geode-web/src/main/java/org/apache/geode/management/internal/web/http/converter/ServerSerializableObjectHttpMessageConverter.java b/geode-web/src/main/java/org/apache/geode/management/internal/web/http/converter/ServerSerializableObjectHttpMessageConverter.java
index bf3b574..f04d472 100644
--- a/geode-web/src/main/java/org/apache/geode/management/internal/web/http/converter/ServerSerializableObjectHttpMessageConverter.java
+++ b/geode-web/src/main/java/org/apache/geode/management/internal/web/http/converter/ServerSerializableObjectHttpMessageConverter.java
@@ -62,7 +62,8 @@ public class ServerSerializableObjectHttpMessageConverter
   @Override
   protected boolean supports(final Class<?> type) {
     if (logger.isTraceEnabled()) {
-      logger.trace(String.format("%1$s.supports(%2$s)", getClass().getName(), type.getName()),
+      logger.trace(String.format("%1$s.supports(%2$s)", getClass().getName(),
+          type == null ? null : type.getName()),
           new Throwable());
     }