hyperhq / runv

Hypervisor-based Runtime for OCI
Apache License 2.0
828 stars 129 forks source link

Fix a nil pointer dereference in supervisor/container.go:199 #431

Closed wrouesnel closed 7 years ago

wrouesnel commented 7 years ago

This use of err is invalid, and crashes runv with a nil pointer deference when err was not set in the code above. It additionally means the wrong error message is printed (as the actual check is "channel closed" not err).

gao-feng commented 7 years ago

LGTM, thanks!