Re: [PATCH 1/1] tty: release_one_tty() forgets to put pids
From: Oleg Nesterov
Date: Fri Apr 02 2010 - 14:45:49 EST
On 04/02, Linus Torvalds wrote:
> On Fri, 2 Apr 2010, Oleg Nesterov wrote:
> > release_one_tty(tty) can be called when tty still has a reference
> > to pgrp/session. In this case we leak the pid.
> Hmm. Maybe we should have cleared this in tty_release() already. We
> already do some of the session clearing there (but we clear the session in
> the _tasks_ associated with the tty, not the tty session pointer).
Yes, probably we can free them earlier.
But I am very nervous about this change, I tried to defer put_pid()
as much as possible, in case something else uses ->prgp/session
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/