Re: [PATCH] stm class: fix device leak in open error path

2016-11-18 Thread Johan Hovold
On Fri, Nov 18, 2016 at 01:55:58PM +0200, Alexander Shishkin wrote: > Johan Hovold writes: > > > Make sure to drop the reference taken by class_find_device() also on > > allocation errors in open(). > > > > Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") > >

Re: [PATCH] stm class: fix device leak in open error path

2016-11-18 Thread Johan Hovold
On Fri, Nov 18, 2016 at 01:55:58PM +0200, Alexander Shishkin wrote: > Johan Hovold writes: > > > Make sure to drop the reference taken by class_find_device() also on > > allocation errors in open(). > > > > Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") > > Signed-off-by:

Re: [PATCH] stm class: fix device leak in open error path

2016-11-18 Thread Alexander Shishkin
Johan Hovold writes: > Make sure to drop the reference taken by class_find_device() also on > allocation errors in open(). > > Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") > Signed-off-by: Johan Hovold Good catch, thanks! I'm going to

Re: [PATCH] stm class: fix device leak in open error path

2016-11-18 Thread Alexander Shishkin
Johan Hovold writes: > Make sure to drop the reference taken by class_find_device() also on > allocation errors in open(). > > Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") > Signed-off-by: Johan Hovold Good catch, thanks! I'm going to change it a bit to look like this if

Re: [PATCH] stm class: fix device leak in open error path

2016-11-17 Thread Johan Hovold
On Tue, Nov 01, 2016 at 11:44:54AM +0100, Johan Hovold wrote: > Make sure to drop the reference taken by class_find_device() also on > allocation errors in open(). > > Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") > Signed-off-by: Johan Hovold > --- >

Re: [PATCH] stm class: fix device leak in open error path

2016-11-17 Thread Johan Hovold
On Tue, Nov 01, 2016 at 11:44:54AM +0100, Johan Hovold wrote: > Make sure to drop the reference taken by class_find_device() also on > allocation errors in open(). > > Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") > Signed-off-by: Johan Hovold > --- >

[PATCH] stm class: fix device leak in open error path

2016-11-01 Thread Johan Hovold
Make sure to drop the reference taken by class_find_device() also on allocation errors in open(). Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") Signed-off-by: Johan Hovold --- drivers/hwtracing/stm/core.c | 9 ++--- 1 file changed, 6 insertions(+), 3

[PATCH] stm class: fix device leak in open error path

2016-11-01 Thread Johan Hovold
Make sure to drop the reference taken by class_find_device() also on allocation errors in open(). Fixes: 7bd1d4093c2f ("stm class: Introduce an abstraction for...") Signed-off-by: Johan Hovold --- drivers/hwtracing/stm/core.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff