Skip to content
Permalink
Browse files
Merge pull request #10972 from AdmiralCurtiss/ios-fs-metadata-nop
IOS/FS: Skip writing FST if SetMetadata() changes nothing.
  • Loading branch information
leoetlino committed Aug 11, 2022
2 parents 3f794bf + 0d39c4a commit be33e10
Showing 1 changed file with 9 additions and 5 deletions.
@@ -678,11 +678,15 @@ ResultCode HostFileSystem::SetMetadata(Uid caller_uid, const std::string& path,
if (entry->data.uid != uid && entry->data.is_file && !is_empty)
return ResultCode::FileNotEmpty;

entry->data.gid = gid;
entry->data.uid = uid;
entry->data.attribute = attr;
entry->data.modes = modes;
SaveFst();
if (entry->data.gid != gid || entry->data.uid != uid || entry->data.attribute != attr ||
entry->data.modes != modes)
{
entry->data.gid = gid;
entry->data.uid = uid;
entry->data.attribute = attr;
entry->data.modes = modes;
SaveFst();
}

return ResultCode::Success;
}

0 comments on commit be33e10

Please sign in to comment.