github/kata-containers
1
0
Fork 2
mirror of https://github.com/kata-containers/kata-containers.git synced 2025-09-24 02:07:31 +00:00
Code Issues Projects Releases Wiki Activity

qemu: Use null QMP logger when the logger parameter is nil

Browse Source 
Or else LaunchQemu() ends up dereferencing a nil pointer and panic'ing.

Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
This commit is contained in:
Samuel Ortiz
2016-09-09 18:45:31 +02:00
parent 7d4199a449
commit f57201989b
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
qemu.go
View File

@@ -49,6 +49,10 @@ import (
// will be returned if the launch succeeds. Otherwise a string containing // will be returned if the launch succeeds. Otherwise a string containing
// the contents of stderr + a Go error object will be returned. // the contents of stderr + a Go error object will be returned.
func LaunchQemu(ctx context.Context, params []string, fds []*os.File, logger QMPLog) (string, error) { func LaunchQemu(ctx context.Context, params []string, fds []*os.File, logger QMPLog) (string, error) {
if logger == nil {
logger = qmpNullLogger{}
}
errStr := "" errStr := ""
cmd := exec.Command("qemu-system-x86_64", params...) cmd := exec.Command("qemu-system-x86_64", params...)
if len(fds) > 0 { if len(fds) > 0 {