GitHub user mike-jumper opened a pull request:

    https://github.com/apache/incubator-guacamole-server/pull/68

    GUACAMOLE-194: Free SSH user for VNC+SFTP and RDP+SFTP only once.

    Freeing the SFTP user immediately after an init failure results in the user 
being freed twice: first within the init routine when the failure occurs, and 
again within the client free handler. Since the client free handler is always 
invoked, it makes sense to rely on that function alone to handle the cleanup.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mike-jumper/incubator-guacamole-server 
sftp-double-free

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-guacamole-server/pull/68.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #68
    
----
commit fb8a2766da914245b8af42b03d33320f4b786157
Author: Michael Jumper <[email protected]>
Date:   2017-02-26T23:02:48Z

    GUACAMOLE-194: Free SSH user for VNC+SFTP and RDP+SFTP only within client 
free.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to