unistack-org/go-libvirt-plain
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #2 from digitalocean/fix-buffer-overwrite

Browse Source 
fixes response overwrite bug
This commit is contained in:
Ben LeMasurier 2016-06-27 13:19:02 -06:00 committed by GitHub
commit fc88e157af
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
rpc.go
View File

@ -195,16 +195,12 @@ func (l *Libvirt) listen() {
// payload: packet length minus what was previously read // payload: packet length minus what was previously read
size := int(length) - (constants.PacketLengthSize + constants.HeaderSize) size := int(length) - (constants.PacketLengthSize + constants.HeaderSize)
buf := make([]byte, size) buf := make([]byte, size)
for n := 0; n < size; { _, err = io.ReadFull(l.r, buf)
nn, err := l.r.Read(buf)
if err != nil { if err != nil {
// invalid packet // invalid packet
continue continue
} }
n += nn
}
// route response to caller // route response to caller
l.route(h, buf) l.route(h, buf)
} }