Check the value of `ret' instead of keeping separate exit point. Signed-off-by: Sergei Zviagintsev <ser...@s15v.net> --- ipc/kdbus/pool.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/ipc/kdbus/pool.c b/ipc/kdbus/pool.c index cb692a35755b..75a7366baa34 100644 --- a/ipc/kdbus/pool.c +++ b/ipc/kdbus/pool.c @@ -261,13 +261,10 @@ struct kdbus_pool_slice *kdbus_pool_slice_alloc(struct kdbus_pool *pool, s->accounted = accounted; if (accounted) pool->accounted_size += s->size; - mutex_unlock(&pool->lock); - - return s; exit_unlock: mutex_unlock(&pool->lock); - return ERR_PTR(ret); + return ret < 0 ? ERR_PTR(ret) : s; } static void __kdbus_pool_slice_release(struct kdbus_pool_slice *slice) -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/