You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@guacamole.apache.org by GitBox <gi...@apache.org> on 2021/05/22 12:54:17 UTC

[GitHub] [guacamole-server] iljavs opened a new pull request #340: set client rcMask to NULL after freeing

iljavs opened a new pull request #340:
URL: https://github.com/apache/guacamole-server/pull/340


   libvncclient does not currently free rcMask, but if it ever does, the current code would trigger a double free. Set client->rcMask to NULL after freeing to prevent this scenario.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [guacamole-server] necouchman commented on pull request #340: set client rcMask to NULL after freeing

Posted by GitBox <gi...@apache.org>.

necouchman commented on pull request #340:
URL: https://github.com/apache/guacamole-server/pull/340#issuecomment-846420596


   @iljavs Thank you for the contribution. You'll need to create a JIRA issue for this and tag the pull request and the commit messages with the JIRA issue. See other existing pull requests, particularly merged ones, for examples of this.
   
   http://guacamole.apache.org/open-source/


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org