On 11/12/2013 04:03 PM, Chen Hanxiao wrote: > ping > see virLXCProcessEnsureRootFS, we already make sure the root is exist for libvirt-lxc.
>> -----Original Message----- >> From: Chen Hanxiao [mailto:[email protected]] >> Sent: Tuesday, November 05, 2013 5:57 PM >> To: [email protected] >> Cc: [email protected] >> Subject: [libvirt][PATCH]lxc: make sure root wouldn't be null >> >> From: Chen Hanxiao <[email protected]> >> >> virDomainGetRootFilesystem may return null. >> We should take care of it. >> >> Signed-off-by: Chen Hanxiao <[email protected]> >> --- >> src/lxc/lxc_container.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/src/lxc/lxc_container.c b/src/lxc/lxc_container.c >> index 255c711..e8f7a75 100644 >> --- a/src/lxc/lxc_container.c >> +++ b/src/lxc/lxc_container.c >> @@ -1829,7 +1829,8 @@ static int lxcContainerChild(void *data) >> if (lxcContainerSetID(vmDef) < 0) >> goto cleanup; >> >> - root = virDomainGetRootFilesystem(vmDef); >> + if (!(root = virDomainGetRootFilesystem(vmDef))) >> + goto cleanup; >> >> if (argv->nttyPaths) { >> const char *tty = argv->ttyPaths[0]; >> -- >> 1.8.2.1 > > > > -- > libvir-list mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/libvir-list > -- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
