GUACAMOLE-424: Prevent null pointer dereference for vnc client display and 
cursor


Project: http://git-wip-us.apache.org/repos/asf/incubator-guacamole-server/repo
Commit: 
http://git-wip-us.apache.org/repos/asf/incubator-guacamole-server/commit/bbafa00d
Tree: 
http://git-wip-us.apache.org/repos/asf/incubator-guacamole-server/tree/bbafa00d
Diff: 
http://git-wip-us.apache.org/repos/asf/incubator-guacamole-server/diff/bbafa00d

Branch: refs/heads/master
Commit: bbafa00df0df242ac308462e16e78c4c23e88a96
Parents: 15f6c4f
Author: itsankoff <ivaylotsanko...@gmail.com>
Authored: Thu Nov 2 19:53:17 2017 +0200
Committer: itsankoff <ivaylotsanko...@gmail.com>
Committed: Thu Nov 2 19:53:17 2017 +0200

----------------------------------------------------------------------
 src/protocols/vnc/user.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-guacamole-server/blob/bbafa00d/src/protocols/vnc/user.c
----------------------------------------------------------------------
diff --git a/src/protocols/vnc/user.c b/src/protocols/vnc/user.c
index b036785..ea09eb0 100644
--- a/src/protocols/vnc/user.c
+++ b/src/protocols/vnc/user.c
@@ -112,7 +112,7 @@ int guac_vnc_user_leave_handler(guac_user* user) {
 
     guac_vnc_client* vnc_client = (guac_vnc_client*) user->client->data;
 
-    if (vnc_client && vnc_client->display && vnc_client->display->cursor) {
+    if (vnc_client->display && vnc_client->display->cursor) {
         /* Update shared cursor state */
         guac_common_cursor_remove_user(vnc_client->display->cursor, user);
     }

Reply via email to