vbatts/pkg
1
0
Fork 0
Code Pull requests Releases Activity

devicemapper: fix zero-sized field access

Browse source 
Fixes: #15279

Due to
7904946eeb
the devices field is dropped.

This solution works on go1.4 and go1.5

Signed-off-by: Vincent Batts <vbatts@redhat.com>
This commit is contained in:
Vincent Batts 2015-08-07 10:18:20 -04:00
parent bad3b47438
commit 25b0312e8c
1 changed files with 15 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
devicemapper/devmapper_wrapper.go
View file

@ -38,7 +38,10 @@ static void log_with_errno_init()
*/ */
import "C" import "C"
import "unsafe" import (
"reflect"
"unsafe"
)
type ( type (
CDmTask C.struct_dm_task CDmTask C.struct_dm_task
@ -184,12 +187,21 @@ func dmTaskGetDepsFct(task *CDmTask) *Deps {
if Cdeps == nil { if Cdeps == nil {
return nil return nil
} }
// golang issue: https://github.com/golang/go/issues/11925
hdr := reflect.SliceHeader{
Data: uintptr(unsafe.Pointer(uintptr(unsafe.Pointer(Cdeps)) + unsafe.Sizeof(*Cdeps))),
Len: int(Cdeps.count),
Cap: int(Cdeps.count),
}
devices := *(*[]C.uint64_t)(unsafe.Pointer(&hdr))
deps := &Deps{ deps := &Deps{
Count: uint32(Cdeps.count), Count: uint32(Cdeps.count),
Filler: uint32(Cdeps.filler), Filler: uint32(Cdeps.filler),
} }
for _, device := range Cdeps.device { for _, device := range devices {
deps.Device = append(deps.Device, (uint64)(device)) deps.Device = append(deps.Device, uint64(device))
} }
return deps return deps
} }