Re: [PATCH 0/4] Fix a race condition when reading loose refs

2013-06-11 Thread Junio C Hamano
I've read the series while on a bus and found all of them sensible. I do share the worry of retry storm you expressed in the last one, and I agree that giving up after N times is a reasonable way out, when it becomes necessary. Thanks. -- To unsubscribe from this list: send the line "unsubscribe

[PATCH 0/4] Fix a race condition when reading loose refs

2013-06-11 Thread Michael Haggerty
This patch series is inspired by the first patch [1] in a series by Peff and solves the race condition that he discovered plus a second, similar race. The general problem is that loose ref files are read in two steps: lstat() followed by readlink() or open(). If another process changes the file b