[PATCH V3] tty: fix memleak in alloc_pid

2014-04-14 Thread Chen Tingjie
_tty() can avoid the race and fix the memleak. Signed-off-by: Zhang Jun Signed-off-by: Chen Tingjie --- drivers/tty/tty_io.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c index 31c4a57..72547a3 100644 --- a/drivers/tty/tty_io

RE: [PATCH] [PATCH V2] tty: memleak in alloc_pid

2014-04-14 Thread Chen, Tingjie
> > Change-Id: Ic960dda039c8f99aad3e0f4d176489a966c62f6a > > > > Why is this line here? > Tingjie, Greg is asking you the sentence of "Change-Id", which is not needed, > please remove it with one new patch. Sorry for mistaken, I will make a new patch for it. -- To unsubscribe from this list: s

RE: [PATCH] [PATCH V2] tty: memleak in alloc_pid

2014-04-14 Thread Chen, Tingjie
ignal->tty, The 2 operations should be atomic, which protected by spinlock: current->sighand->siglock. So the sentence: spin_unlock_irq(¤t->sighand->siglock); need to move down after tty has process completed. Thanks, -Original Message- From: Greg Kroah-Hartman [mailto:gre...

[PATCH V2] tty: fix memleak in alloc_pid

2014-04-14 Thread Chen Tingjie
ter get_current_tty() can avoid the race and fix the memleak. Change-Id: Ic960dda039c8f99aad3e0f4d176489a966c62f6a Signed-off-by: Zhang Jun Signed-off-by: Chen Tingjie --- drivers/tty/tty_io.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c in

[PATCH] [PATCH V2] tty: memleak in alloc_pid

2014-04-14 Thread Chen Tingjie
ter get_current_tty() can avoid the race and fix the memleak. Change-Id: Ic960dda039c8f99aad3e0f4d176489a966c62f6a Signed-off-by: Zhang Jun Signed-off-by: Chen Tingjie --- drivers/tty/tty_io.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c in

[PATCH] tty: fix memleak in alloc_pid

2014-04-09 Thread Chen Tingjie
an fix this leak. Signed-off-by: Zhang Jun Signed-off-by: Chen Tingjie --- drivers/tty/tty_io.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c index d3448a9..3411071 100644 --- a/drivers/tty/tty_io.c +++ b/drivers/tty/tty_io.c