The boolean actually tells whether the file existed when the function
was called and using it in more places later on makes them
confusing (e.g. do something with a file if it does not exist). To
better reflect the above and prepare for next patch rename this
variable.
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
Reviewed-by: Peter Krempa <pkrempa@redhat.com>
bool isDev = S_ISCHR(data->sb.st_mode) || S_ISBLK(data->sb.st_mode);
bool isReg = S_ISREG(data->sb.st_mode) || S_ISFIFO(data->sb.st_mode) || S_ISSOCK(data->sb.st_mode);
bool isDir = S_ISDIR(data->sb.st_mode);
- bool exists = false;
+ bool existed = false;
if (virFileExists(data->file))
- exists = true;
+ existed = true;
if (virFileMakeParentPath(data->file) < 0) {
virReportSystemError(errno,
virFileMoveMount(data->target, data->file) < 0)
goto cleanup;
- ret = exists;
+ ret = existed;
cleanup:
if (ret < 0 && delDevice) {
if (isDir)