Messages in this thread | | | Date | Fri, 31 Aug 2012 16:05:39 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH] uprobes: don't put NULL pointer in uprobe_register() |
| |
On 08/30, Sebastian Andrzej Siewior wrote: > > alloc_uprobe() might return a NULL pointer, put_uprobe() can't deal with > this. > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> > --- > kernel/events/uprobes.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c > index 7b3112c..30f3e23 100644 > --- a/kernel/events/uprobes.c > +++ b/kernel/events/uprobes.c > @@ -897,7 +897,8 @@ int uprobe_register(struct inode *inode, loff_t offset, struct uprobe_consumer * > } > > mutex_unlock(uprobes_hash(inode)); > - put_uprobe(uprobe); > + if (uprobe) > + put_uprobe(uprobe);
Applied as cf54f5c8, thanks.
Oleg.
| |