Re: [PATCH v2 3/5] binder: check vma->vm_start in binder_vma_close()

From: Alice Ryhl

Date: Tue Sep 01 2026 - 06:52:54 EST


On Tue, Sep 01, 2026 at 03:04:14AM +0000, Carlos Llamas wrote:
> Certain operations like a failed mremap() might trigger vm_ops->close()
> on temporary mappings. To avoid tearing-down the main binder mapping on
> these, let's verify that the VMA matches the expected starting address.
>
> Cc: stable@xxxxxxxxxxxxxxx
> Fixes: 457b9a6f09f0 ("Staging: android: add binder driver")
> Reported-by: Sashiko <sashiko-bot@xxxxxxxxxx>
> Closes: https://sashiko.dev/#/patchset/20260831224145.169403-1-cmllamas%40google.com?part=1
> Signed-off-by: Carlos Llamas <cmllamas@xxxxxxxxxx>

I don't quite understand this. You explicitly made sure mremap() can't
happen to begin with, so when is this called?

Alice