You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@guacamole.apache.org by mj...@apache.org on 2017/11/15 15:06:39 UTC

[2/7] incubator-guacamole-server git commit: GUACAMOLE-424: Prevent null pointer dereference for vnc client display and cursor

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 <iv...@gmail.com>
Authored: Thu Nov 2 19:53:17 2017 +0200
Committer: itsankoff <iv...@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);
     }