External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-07-11 04:15:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

network: Fix upgrade from libvirt older than 1.2.4

Browse Source 
Starting from libvirt-1.2.4, network state XML files moved to another
directory (see commit b9e95491) and libvirt automatically migrates the
network state files to a new location. However, the code used
dirent.d_type which is not supported by all filesystems. Thus, when
libvirt was upgraded on a host which used such filesystem, network state
XMLs were not properly moved and running networks disappeared from
libvirt.

This patch falls back to lstat() whenever dirent.d_type is DT_UNKNOWN to
fix this issue.

https://bugzilla.redhat.com/show_bug.cgi?id=1167145
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
This commit is contained in:
Jiri Denemark 2014-11-26 16:24:27 +01:00
parent f37627ee72
commit dabb23e6d9
1 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/network/bridge_driver.c
View File

@ -499,15 +499,34 @@ networkMigrateStateFiles(void)
}
while ((direrr = virDirRead(dir, &entry, oldStateDir)) > 0) {
if (entry->d_type != DT_UNKNOWN &&
entry->d_type != DT_REG)
continue;
if (entry->d_type != DT_REG ||
STREQ(entry->d_name, ".") ||
if (STREQ(entry->d_name, ".") ||
STREQ(entry->d_name, ".."))
continue;
if (virAsprintf(&oldPath, "%s/%s",
oldStateDir, entry->d_name) < 0)
goto cleanup;
if (entry->d_type == DT_UNKNOWN) {
struct stat st;
if (lstat(oldPath, &st) < 0) {
virReportSystemError(errno,
_("failed to stat network status file '%s'"),
oldPath);
goto cleanup;
}
if (!S_ISREG(st.st_mode)) {
VIR_FREE(oldPath);
continue;
}
}
if (virFileReadAll(oldPath, 1024*1024, &contents) < 0)
goto cleanup;