From: Cherry Mui <cherryyz@google.com>
Date: Sun, 22 Jun 2025 19:01:27 +0000 (-0400)
Subject: runtime: add missing unlock in sysReserveAlignedSbrk
X-Git-Tag: go1.25rc2~2^2~42
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=456a90aa1618a6c3aa49ecba46969128e2bfa26f;p=gostls13.git

runtime: add missing unlock in sysReserveAlignedSbrk

sysReserveAlignedSbrk locks memlock at entry, but it is not
unlocked at one of the return path. Add the missing unlock.

Fixes #74339.

Change-Id: Ib641bc348aca41494ec410e2c4eb9857f3362484
Reviewed-on: https://go-review.googlesource.com/c/go/+/683295
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: David Chase <drchase@google.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
---

diff --git a/src/runtime/mem_sbrk.go b/src/runtime/mem_sbrk.go
index 9d6842ae52..05f0fdb5d7 100644
--- a/src/runtime/mem_sbrk.go
+++ b/src/runtime/mem_sbrk.go
@@ -231,6 +231,7 @@ func sysReserveAlignedSbrk(size, align uintptr) (unsafe.Pointer, uintptr) {
 			memFree(unsafe.Pointer(end), endLen)
 		}
 		memCheck()
+		unlock(&memlock)
 		return unsafe.Pointer(pAligned), size
 	}