aboutsummaryrefslogtreecommitdiffstats
path: root/package
diff options
context:
space:
mode:
authorDaniel Dickinson <daniel@cshore.neomailbox.net>2010-06-29 10:07:45 +0000
committerDaniel Dickinson <daniel@cshore.neomailbox.net>2010-06-29 10:07:45 +0000
commitc0edae12e180ccfa6b0403d279b2b9cfd39715d2 (patch)
tree87a0772e56f2aa594b8d81a882eea694f30874fd /package
parent54ffe62dedb1f1081841a48964f33ae698e55f2d (diff)
downloadupstream-c0edae12e180ccfa6b0403d279b2b9cfd39715d2.tar.gz
upstream-c0edae12e180ccfa6b0403d279b2b9cfd39715d2.tar.bz2
upstream-c0edae12e180ccfa6b0403d279b2b9cfd39715d2.zip
block-mount: Fixed mount deadlocks due to buggy use of locks in /lib/functions/block.sh
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@21987 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'package')
-rw-r--r--package/block-mount/files/block.sh34
1 files changed, 20 insertions, 14 deletions
diff --git a/package/block-mount/files/block.sh b/package/block-mount/files/block.sh
index b8a473a00f..72d6c3ab3c 100644
--- a/package/block-mount/files/block.sh
+++ b/package/block-mount/files/block.sh
@@ -118,15 +118,18 @@ config_create_swap_fstab_entry() {
[ -n "$device" ] || return 0
- local fstabnew="$(mktemp -t '.fstab.XXXXXXXX')"
+ local fstabnew
mkdir -p /var/lock
- lock /var/lock/fstab.lck
- cat /tmp/fstab | grep -E -v "^$device[[:blank:]]" >>"$fstabnew"
- [ "$enabled" -eq 1 ] && echo "$device none swap sw 0 0" >> "$fstabnew"
- cat "$fstabnew" >/tmp/fstab
- lock -u /var/lock/fstab.lck
- rm -f $fstabnew
+ lock -w /var/lock/fstab.lck && {
+ lock /var/lock/fstab.lck
+ fstabnew="$(mktemp -t '.fstab.XXXXXXXX')"
+ cat /tmp/fstab | grep -E -v "^$device[[:blank:]]" >>"$fstabnew"
+ [ "$enabled" -eq 1 ] && echo "$device none swap sw 0 0" >> "$fstabnew"
+ cat "$fstabnew" >/tmp/fstab
+ rm -f $fstabnew
+ lock -u /var/lock/fstab.lck
+ }
}
config_create_mount_fstab_entry() {
@@ -140,15 +143,18 @@ config_create_mount_fstab_entry() {
[ -n "$target" ] || return 0
[ -n "$device" ] || return 0
- local fstabnew="$(mktemp -t '.fstab.XXXXXXXX')"
+ local fstabnew
mkdir -p /var/lock
- lock /var/lock/fstab.lck
- cat /tmp/fstab | grep -E -v "^$device[[:blank:]]" | grep -v "$target" >>"$fstabnew"
- echo "$device $target $fstype $options 0 0" >>"$fstabnew"
- cat "$fstabnew" >/tmp/fstab
- lock -u /var/lock/fstab.lck
- rm -f $fstabnew
+ lock -w /var/lock/fstab.lck && {
+ lock /var/lock/fstab.lck
+ fstabnew="$(mktemp -t '.fstab.XXXXXXXX')"
+ cat /tmp/fstab | grep -E -v "^$device[[:blank:]]" | grep -v "$target" >>"$fstabnew"
+ echo "$device $target $fstype $options 0 0" >>"$fstabnew"
+ cat "$fstabnew" >/tmp/fstab
+ rm -f $fstabnew
+ lock -u /var/lock/fstab.lck
+ }
}
libmount_find_token() {