diff options
| author | Barton E. Schaefer <schaefer@zsh.org> | 2014-09-06 22:10:30 -0700 |
|---|---|---|
| committer | Barton E. Schaefer <schaefer@zsh.org> | 2014-09-06 22:10:30 -0700 |
| commit | 4414e54ea7ffe50acca851c11c2ef49dc867c55d (patch) | |
| tree | 323cccfdc3faa0956082432f54947513d82efe5a /Src/hist.c | |
| parent | c61a927d1d156731d9e21c714dfea9585163c556 (diff) | |
| download | zsh-4414e54ea7ffe50acca851c11c2ef49dc867c55d.tar.gz zsh-4414e54ea7ffe50acca851c11c2ef49dc867c55d.tar.xz zsh-4414e54ea7ffe50acca851c11c2ef49dc867c55d.zip | |
33116: followup to 32580 to prevent double-locking with shared or incremental history
Diffstat (limited to 'Src/hist.c')
| -rw-r--r-- | Src/hist.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/Src/hist.c b/Src/hist.c index 770d559b1..d29a65afe 100644 --- a/Src/hist.c +++ b/Src/hist.c @@ -2490,6 +2490,9 @@ flockhistfile(char *fn, int keep_trying) struct flock lck; int ctr = keep_trying ? 9 : 0; + if (flock_fd >= 0) + return 0; /* already locked */ + if ((flock_fd = open(unmeta(fn), O_RDWR | O_NOCTTY)) < 0) return errno == ENOENT ? 0 : 2; /* "successfully" locked missing file */ @@ -2768,12 +2771,6 @@ lockhistfile(char *fn, int keep_trying) if (!fn && !(fn = getsparam("HISTFILE"))) return 1; -#ifdef HAVE_FCNTL_H - if (isset(HISTFCNTLLOCK) && flock_fd < 0) { - return flockhistfile(fn, keep_trying); - } -#endif - if (!lockhistct++) { struct stat sb; int fd; @@ -2786,6 +2783,11 @@ lockhistfile(char *fn, int keep_trying) # endif #endif +#ifdef HAVE_FCNTL_H + if (isset(HISTFCNTLLOCK)) + return flockhistfile(fn, keep_trying); +#endif + lockfile = bicat(unmeta(fn), ".LOCK"); /* NOTE: only use symlink locking on a link()-having host in order to * avoid a change from open()-based locking to symlink()-based. */ |