On Thu, Jun 25, 2020 at 10:55 AM Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> wrote: > - current_inferior ()->terminal = xstrdup (terminal_name); > + current_inferior ()->terminal.reset (xstrdup (terminal_name)); Perhaps it really should be a std::string? Christian