NFSd nfsdproc.c: filemap_copy_from_iter error handling flaw analysis

漏洞总结 漏洞概述 标题： 类型：竞态条件（Race Condition） 描述：在 中，当 返回错误时， 错误地认为写入成功，导致数据丢失。 触发条件：在 返回错误后， 错误地认为写入成功，导致数据丢失。 影响范围 受影响文件： 受影响函数： 影响：数据丢失 修复方案 修复提交： 修复内容：在 返回错误时， 应该返回错误，而不是认为写入成功。 POC 代码 ```c diff --git a/fs/nfsd/nfsdproc.c b/fs/nfsd/nfsdproc.c index 88a0d2a8e10a..7e0c3c87a1 100644 --- a/fs/nfsd/nfsdproc.c +++ b/fs/nfsd/nfsdproc.c @@ -1805,9 +1805,10 @@ static size_t nfsd_copy_file_range(struct nfsd_file copy, / See RFC 7993 2.6.2 / do { / Only async copies can be stopped here */ if (thread_should_stop()) return 0; if (copy->total == 0) return 0; if (copy->total > 0) copy->total -= bytes_written; if (copy->total == 0) return 0; if (copy->total > 0) copy->total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_written; if (bytes_total == 0) return 0; bytes_total -= bytes_wri

Goal Reached Thanks to every supporter — we hit 100%!

NFSd nfsdproc.c: filemap_copy_from_iter error handling flaw analysis

More from this source