Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

Commit

Permalink
tasking: Do not free tty
Browse files Browse the repository at this point in the history 
It took only 2 months to figure it out that this was causing unexpected crashes... lol
  • Loading branch information
@EnderIce2
EnderIce2 committed Nov 19, 2024
1 parent 4d19273 commit 7b05405
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions tasking/process.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -270,8 +270,14 @@ namespace Tasking
delete this->FileDescriptors;

debug("Deleting tty");
if (this->tty)
delete ((TTY::TeletypeDriver *)this->tty);
// if (this->tty)
// delete ((TTY::TeletypeDriver *)this->tty);
fixme("remove workarounds for stdio and tty");

/* FIXME: DON'T DELETE THE TTY
spawn.cpp is using this as workaround
tty == KernelConsole::CurrentTerminal.load();
*/

/* If we own the pointer to the
PageTable, we need to free it */
Expand Down

0 comments on commit 7b05405

Please sign in to comment.