On Fri, 2005-06-10 at 14:05 +0100, Anton Altaparmakov wrote:
> Here is the second patch (patch B).
>
> On Fri, 2005-06-10 at 14:01 +0100, Anton Altaparmakov wrote:
> [snip]
> > B) If we cannot safely allow buffer_new buffers to "leak out" of
> > __block_prepare_write(), then we simply would need to run a quick loop
> > over the buffers clearing buffer_new on each of them if it is set just
> > before returning "success" to the caller of __block_prepare_write().
> [snip]
>
> The patch for this is simple, too (it is below).
>
> > Andrew/Linus, I would suggest that you apply at least A and perhaps B if
> > you deem it necessary or want to be on the safe side.
> >
> > Having had a look at the code it would seem perfectly safe to leave
> > buffer_new() set and ignore patch B but I may be wrong which is why I
> > did both.
I have changed my mind having had a look at the code some more. Patch B
is definitely needed.
Otherwise you can get a situation where an allocating write followed by
an erring write to the same location would cause uptodate and already
written out buffers to be zeroed & written out thus overwritting
existing data with zero.
This could be worked around differently (e.g. by being more picky which
buffer_new buffers get zeroed&written in the error handling) it is
definitely safest to just apply patch B.
Best regards,
Anton
--
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK
Linux NTFS maintainer / IRC: #ntfs on irc.freenode.net
WWW: http://linux-ntfs.sf.net/ & http://www-stu.christs.cam.ac.uk/~aia21/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]