From: Jason A. Donenfeld <Jason@zx2c4.com>
Date: Fri, 6 Sep 2019 03:48:21 +0000 (-0600)
Subject: namespaceapi: AddSIDToBoundaryDescriptor modifies the handle
X-Git-Tag: 0.0.20190908~3
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d12eb91f9a309a30dd78e89ff71050842d4a4e6e;p=thirdparty%2Fwireguard-go.git

namespaceapi: AddSIDToBoundaryDescriptor modifies the handle
---

diff --git a/tun/wintun/namespaceapi/namespaceapi_windows.go b/tun/wintun/namespaceapi/namespaceapi_windows.go
index ad73df2..ef75302 100644
--- a/tun/wintun/namespaceapi/namespaceapi_windows.go
+++ b/tun/wintun/namespaceapi/namespaceapi_windows.go
@@ -36,8 +36,8 @@ func (bd BoundaryDescriptor) Delete() {
 }
 
 // AddSid adds a security identifier (SID) to the specified boundary descriptor.
-func (bd BoundaryDescriptor) AddSid(requiredSid *windows.SID) error {
-	return addSIDToBoundaryDescriptor((*windows.Handle)(&bd), requiredSid)
+func (bd *BoundaryDescriptor) AddSid(requiredSid *windows.SID) error {
+	return addSIDToBoundaryDescriptor((*windows.Handle)(bd), requiredSid)
 }
 
 // PrivateNamespace represents a private namespace. Duh?!