Commit cb7d8d7e authored Dec 04, 2017 by Hugh Dickins Committed by Greg Kroah-Hartman Jan 05, 2018

kaiser: fix unlikely error in alloc_ldt_struct()




An error from kaiser_add_mapping() here is not at all likely, but
Eric Biggers rightly points out that __free_ldt_struct() relies on
new_ldt->size being initialized: move that up.

Signed-off-by: Hugh Dickins <hughd@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 3df14617

arch/x86/kernel/ldt.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -79,11 +79,11 @@ static struct ldt_struct *alloc_ldt_struct(int size)

		ret = kaiser_add_mapping((unsigned long)new_ldt->entries, alloc_size,
		__PAGE_KERNEL);
		new_ldt->size = size;
		if (ret) {
		__free_ldt_struct(new_ldt);
		return NULL;
		}
		new_ldt->size = size;
		return new_ldt;
		}