From ac00154a7c6a8f3b2a98c1cf3cef893af2b0abac Mon Sep 17 00:00:00 2001 From: Vaibhav Agarwal Date: Tue, 17 May 2016 22:29:08 +0530 Subject: [PATCH] greybus: audio: gb_manager: Use valid argument while removing ida allocated id. module->id is used as an argument to ida_simple_remove(). Since module is already dereferenced, module->id might contain invalid data. So fix this. Fixes: da4cc2d0b066 ("audio:gb_manager: Use proper locking around kobject_xxx") Signed-off-by: Vaibhav Agarwal Reviewed-by: Mark Greer Signed-off-by: Greg Kroah-Hartman --- drivers/staging/greybus/audio_manager.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_manager.c b/drivers/staging/greybus/audio_manager.c index 9def014cd47c..aa6508b44fab 100644 --- a/drivers/staging/greybus/audio_manager.c +++ b/drivers/staging/greybus/audio_manager.c @@ -76,7 +76,7 @@ int gb_audio_manager_remove(int id) list_del(&module->list); kobject_put(&module->kobj); up_write(&modules_rwsem); - ida_simple_remove(&module_id, module->id); + ida_simple_remove(&module_id, id); return 0; } EXPORT_SYMBOL_GPL(gb_audio_manager_remove); -- 2.39.5