ldlm_lock_create() gets a resource, but don't put it on
all failure paths. It should.
Change-Id: Ib49bcafdeac834c412adad9db135034d1ea06a04
Signed-off-by: Mr. NeilBrown <neilb@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-on: https://review.whamcloud.com/45585
Tested-by: jenkins <devops@whamcloud.com>
Reviewed-by: Olaf Faaland-LLNL <faaland1@llnl.gov>
Reviewed-by: Andriy Skulysh <andriy.skulysh@hpe.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
RETURN(ERR_CAST(res));
lock = ldlm_lock_new(res);
- if (lock == NULL)
+ if (!lock) {
+ ldlm_resource_putref(res);
RETURN(ERR_PTR(-ENOMEM));
+ }
lock->l_req_mode = mode;
lock->l_ast_data = data;