Uploaded image for project: 'Guacamole'
  1. Guacamole
  2. GUACAMOLE-194

Double free() in guac_common_ssh_destroy_user()

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.9.10-incubating
    • Fix Version/s: 0.9.13-incubating
    • Component/s: RDP, VNC
    • Labels:
      None
    • Environment:
      guacd docker container on ubuntu and debian

      Description

      We are heavy users of the RDP feature with SSH/SFTP for file downloads and uploads. We are seeing frequent guacd crashes, even the first session log off can cause a crash.

      After a few sessions (Sometimes hundreds, often thousands) the container becomes unstable, causes very high system load and no new connections via guacamole are possible. We typically also see the virtual docker Ethernet adapter enter the disabled state.

      void guac_common_ssh_destroy_user(guac_common_ssh_user* user) {
      
          /* Free private key, if present */
          if (user->private_key != NULL)
              guac_common_ssh_key_free(user->private_key);
      
          /* Free all other data */
          free(user->password);
          free(user->username);
          free(user);
      
      }
      

        Attachments

        1. GATEWAY1-logs.zip
          1.42 MB
          Robin
        2. GATEWAY2-logs.zip
          25 kB
          Robin
        3. guacd-stack trace.txt
          48 kB
          Robin

          Activity

            People

            • Assignee:
              mike.jumper Michael Jumper
              Reporter:
              mrbabbage Robin
            • Votes:
              2 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: