]> git.ipfire.org Git - thirdparty/git.git/commit - lockfile.c
lockfile.c: store absolute path
authorNguyễn Thái Ngọc Duy <pclouds@gmail.com>
Sun, 2 Nov 2014 06:24:37 +0000 (07:24 +0100)
committerJunio C Hamano <gitster@pobox.com>
Mon, 3 Nov 2014 19:00:28 +0000 (11:00 -0800)
commitfa137f67a40bbcb3f583920586fdd34ce98ddeaf
tree9f82bc0f053b03141bd2e53acf2a8d98517d325b
parent4ace7ff4557350b7e0b57d024a2ea311b332e01d
lockfile.c: store absolute path

Locked paths can be saved in a linked list so that if something wrong
happens, *.lock are removed. For relative paths, this works fine if we
keep cwd the same, which is true 99% of time except:

- update-index and read-tree hold the lock on $GIT_DIR/index really
  early, then later on may call setup_work_tree() to move cwd.

- Suppose a lock is being held (e.g. by "git add") then somewhere
  down the line, somebody calls real_path (e.g. "link_alt_odb_entry"),
  which temporarily moves cwd away and back.

During that time when cwd is moved (either permanently or temporarily)
and we decide to die(), attempts to remove relative *.lock will fail,
and the next operation will complain that some files are still locked.

Avoid this case by turning relative paths to absolute before storing
the path in "filename" field.

Reported-by: Yue Lin Ho <yuelinho777@gmail.com>
Helped-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Helped-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Adapted-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
lockfile.c
t/t2107-update-index-basic.sh