[PATCH 2 of 3] lock: add ability to store additional metadata on filesytem

Bryan O'Sullivan bos at serpentine.com
Mon Mar 20 12:49:10 EDT 2017


On Mon, Mar 20, 2017 at 12:49 AM, Phil Cohen <phillco at fb.com> wrote:


> @@ -248,6 +253,7 @@
>
                 if not self._parentheld:
>                      try:
>                          self.vfs.unlink(self.f)
> +                        self.vfs.unlink(self.f + '.info')
>                      except OSError:
>                          pass
>

This unlink ordering introduces a race condition, where one process can
delete a .info file owned by another.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20170320/55b1127d/attachment.html>


More information about the Mercurial-devel mailing list