diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2019-05-03 09:34:00 +0200 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2019-05-03 09:34:00 +0200 |
commit | 81ca08f1b3ebd3d7b893e0bb39cd00e97827e1f7 (patch) | |
tree | 21e828e10bc0719aaea21b254c308195b229c09f /tun/wintun/setupapi | |
parent | 2e988467c2c9b7a94369f605bad6b08c70a9df7e (diff) |
setupapi: safer aliasing of slice types
Diffstat (limited to 'tun/wintun/setupapi')
-rw-r--r-- | tun/wintun/setupapi/setupapi_windows.go | 30 | ||||
-rw-r--r-- | tun/wintun/setupapi/setupapi_windows_test.go | 4 |
2 files changed, 26 insertions, 8 deletions
diff --git a/tun/wintun/setupapi/setupapi_windows.go b/tun/wintun/setupapi/setupapi_windows.go index dec9da2..621d12b 100644 --- a/tun/wintun/setupapi/setupapi_windows.go +++ b/tun/wintun/setupapi/setupapi_windows.go @@ -8,6 +8,7 @@ package setupapi import ( "encoding/binary" "fmt" + "runtime" "syscall" "unsafe" @@ -261,9 +262,13 @@ func SetupDiGetDeviceRegistryProperty(deviceInfoSet DevInfo, deviceInfoData *Dev func getRegistryValue(buf []byte, dataType uint32) (interface{}, error) { switch dataType { case windows.REG_SZ: - return windows.UTF16ToString(BufToUTF16(buf)), nil + ret := windows.UTF16ToString(bufToUTF16(buf)) + runtime.KeepAlive(buf) + return ret, nil case windows.REG_EXPAND_SZ: - return registry.ExpandString(windows.UTF16ToString(BufToUTF16(buf))) + ret, err := registry.ExpandString(windows.UTF16ToString(bufToUTF16(buf))) + runtime.KeepAlive(buf) + return ret, err case windows.REG_BINARY: return buf, nil case windows.REG_DWORD_LITTLE_ENDIAN: @@ -271,7 +276,7 @@ func getRegistryValue(buf []byte, dataType uint32) (interface{}, error) { case windows.REG_DWORD_BIG_ENDIAN: return binary.BigEndian.Uint32(buf), nil case windows.REG_MULTI_SZ: - bufW := BufToUTF16(buf) + bufW := bufToUTF16(buf) a := []string{} for i := 0; i < len(bufW); { j := i + wcslen(bufW[i:]) @@ -280,6 +285,7 @@ func getRegistryValue(buf []byte, dataType uint32) (interface{}, error) { } i = j + 1 } + runtime.KeepAlive(buf) return a, nil case windows.REG_QWORD_LITTLE_ENDIAN: return binary.LittleEndian.Uint64(buf), nil @@ -288,8 +294,8 @@ func getRegistryValue(buf []byte, dataType uint32) (interface{}, error) { } } -// BufToUTF16 function reinterprets []byte buffer as []uint16 -func BufToUTF16(buf []byte) []uint16 { +// bufToUTF16 function reinterprets []byte buffer as []uint16 +func bufToUTF16(buf []byte) []uint16 { sl := struct { addr *uint16 len int @@ -298,8 +304,8 @@ func BufToUTF16(buf []byte) []uint16 { return *(*[]uint16)(unsafe.Pointer(&sl)) } -// UTF16ToBuf function reinterprets []uint16 as []byte -func UTF16ToBuf(buf []uint16) []byte { +// utf16ToBuf function reinterprets []uint16 as []byte +func utf16ToBuf(buf []uint16) []byte { sl := struct { addr *byte len int @@ -334,6 +340,16 @@ func (deviceInfoSet DevInfo) SetDeviceRegistryProperty(deviceInfoData *DevInfoDa return SetupDiSetDeviceRegistryProperty(deviceInfoSet, deviceInfoData, property, propertyBuffers) } +func (deviceInfoSet DevInfo) SetDeviceRegistryPropertyString(deviceInfoData *DevInfoData, property SPDRP, str string) error { + str16, err := windows.UTF16FromString(str) + if err != nil { + return err + } + err = SetupDiSetDeviceRegistryProperty(deviceInfoSet, deviceInfoData, property, utf16ToBuf(append(str16, 0))) + runtime.KeepAlive(str16) + return err +} + //sys setupDiGetDeviceInstallParams(deviceInfoSet DevInfo, deviceInfoData *DevInfoData, deviceInstallParams *DevInstallParams) (err error) = setupapi.SetupDiGetDeviceInstallParamsW // SetupDiGetDeviceInstallParams function retrieves device installation parameters for a device information set or a particular device information element. diff --git a/tun/wintun/setupapi/setupapi_windows_test.go b/tun/wintun/setupapi/setupapi_windows_test.go index 73cad7c..d5443ff 100644 --- a/tun/wintun/setupapi/setupapi_windows_test.go +++ b/tun/wintun/setupapi/setupapi_windows_test.go @@ -6,6 +6,7 @@ package setupapi import ( + "runtime" "strings" "syscall" "testing" @@ -471,7 +472,7 @@ func TestSetupDiGetSelectedDevice(t *testing.T) { func TestUTF16ToBuf(t *testing.T) { buf := []uint16{0x0123, 0x4567, 0x89ab, 0xcdef} - buf2 := UTF16ToBuf(buf) + buf2 := utf16ToBuf(buf) if len(buf)*2 != len(buf2) || cap(buf)*2 != cap(buf2) || buf2[0] != 0x23 || buf2[1] != 0x01 || @@ -480,4 +481,5 @@ func TestUTF16ToBuf(t *testing.T) { buf2[6] != 0xef || buf2[7] != 0xcd { t.Errorf("SetupDiSetSelectedDevice(nil) should fail with ERROR_INVALID_USER_BUFFER") } + runtime.KeepAlive(buf) } |