Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a100d88d authored by Radim Krčmář's avatar Radim Krčmář Committed by Greg Kroah-Hartman
Browse files

hv: use correct order when freeing monitor_pages 



We try to free two pages when only one has been allocated.
Cleanup path is unlikely, so I haven't found any trace that would fit,
but I hope that free_pages_prepare() does catch it.

Cc: stable@vger.kernel.org
Signed-off-by: default avatarRadim Krčmář <rkrcmar@redhat.com>
Reviewed-by: default avatarAmos Kong <akong@redhat.com>
Acked-by: default avatarJason Wang <jasowang@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 4a2883ec

drivers/hv/connection.c

+2 −2
Original line number Diff line number Diff line
@@ -224,8 +224,8 @@ int vmbus_connect(void) 
		vmbus_connection.int_page = NULL;

	}



	free_pages((unsigned long)vmbus_connection.monitor_pages[0], 1);

	free_pages((unsigned long)vmbus_connection.monitor_pages[1], 1);

	free_pages((unsigned long)vmbus_connection.monitor_pages[0], 0);

	free_pages((unsigned long)vmbus_connection.monitor_pages[1], 0);

	vmbus_connection.monitor_pages[0] = NULL;

	vmbus_connection.monitor_pages[1] = NULL;