Github user darchons commented on a diff in the pull request:

    https://github.com/apache/guacamole-server/pull/132#discussion_r188500310
  
    --- Diff: src/terminal/display.c ---
    @@ -317,6 +319,9 @@ guac_terminal_display* 
guac_terminal_display_alloc(guac_client* client,
     
     void guac_terminal_display_free(guac_terminal_display* display) {
     
    +    /* Free default palette. */
    +    free((void*) display->default_palette);
    --- End diff --
    
    I made `default_palette` to have `const` elements to ensure correct usage 
elsewhere, but that meant we need to jump through this hoop when calling 
`free()`


---

Reply via email to