diff options
| author | Jacek Konieczny <jajcus@jajcus.net> | 2012-11-14 10:24:28 +0000 |
|---|---|---|
| committer | Jacek Konieczny <jajcus@jajcus.net> | 2012-11-14 10:24:28 +0000 |
| commit | e3f4dd42f626e6bf8b684eb3d157f66b60751eb4 (patch) | |
| tree | 29468286f1c8d84e3526348046b1c1a157cb1d5d /tools/hotplug | |
| parent | 9b223b2f098261f3d483c3447d22be5d7069c314 (diff) | |
| download | xen-e3f4dd42f626e6bf8b684eb3d157f66b60751eb4.tar.gz xen-e3f4dd42f626e6bf8b684eb3d157f66b60751eb4.tar.bz2 xen-e3f4dd42f626e6bf8b684eb3d157f66b60751eb4.zip | |
hotplug/Linux: Fix locking in tools/hotplug/Linux/locking.sh
The claim_lock() function would fail in the perl code with:
Invalid argument at -e line 2.
because the Perl snippet opens for reading the file descriptor, which
was earlier opened for write (append).
Signed-off-by: Jacek Konieczny <jajcus@jajcus.net>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
Committed-by: Ian Campbell <ian.campbell@citrix.com>
Diffstat (limited to 'tools/hotplug')
| -rw-r--r-- | tools/hotplug/Linux/locking.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/hotplug/Linux/locking.sh b/tools/hotplug/Linux/locking.sh index e34f155438..122bcfb53b 100644 --- a/tools/hotplug/Linux/locking.sh +++ b/tools/hotplug/Linux/locking.sh @@ -44,7 +44,7 @@ claim_lock() # See below for a correctness proof. local rightfile while true; do - eval "exec $_lockfd>>$_lockfile" + eval "exec $_lockfd<>$_lockfile" flock -x $_lockfd || return $? # We can't just stat /dev/stdin or /proc/self/fd/$_lockfd or # use bash's test -ef because those all go through what is |