diff options
author | John Thomson <git@johnthomson.fastmail.com.au> | 2020-12-09 13:03:04 +1000 |
---|---|---|
committer | Petr Štetiar <ynezz@true.cz> | 2020-12-11 13:48:24 +0100 |
commit | d82c1912838f262b6f4b9157077f390f5f06b6c4 (patch) | |
tree | f39f0cb8e2497835a711b37c7f03e0ca4478ab87 /package/base-files/files/lib | |
parent | 064d65c2f76409759ac8d72268f2558c7b55f3b3 (diff) | |
download | upstream-d82c1912838f262b6f4b9157077f390f5f06b6c4.tar.gz upstream-d82c1912838f262b6f4b9157077f390f5f06b6c4.tar.bz2 upstream-d82c1912838f262b6f4b9157077f390f5f06b6c4.zip |
package/base-files: caldata: use dd iflag fullblock
This dd flag ensures that the requested size
is retrieved from pipes or special filesystems (if available).
Without this flag, on multi-core systems,
Piped or special filesystem data can be truncated
when a size greater than PIPE_BUF is requested.
Fixes: FS#3494
Fixes: 7557e7f ("package/base-files: caldata: work around dd's
limitation")
Cc: Thibaut VARÈNE <hacks@slashdirt.org>
Signed-off-by: John Thomson <git@johnthomson.fastmail.com.au>
Diffstat (limited to 'package/base-files/files/lib')
-rw-r--r-- | package/base-files/files/lib/functions/caldata.sh | 37 |
1 files changed, 25 insertions, 12 deletions
diff --git a/package/base-files/files/lib/functions/caldata.sh b/package/base-files/files/lib/functions/caldata.sh index e22c7d27e6..2177cf8415 100644 --- a/package/base-files/files/lib/functions/caldata.sh +++ b/package/base-files/files/lib/functions/caldata.sh @@ -3,6 +3,16 @@ . /lib/functions.sh . /lib/functions/system.sh +caldata_dd() { + local source=$1 + local target=$2 + local count=$(($3)) + local offset=$(($4)) + + dd if=$source of=$target iflag=skip_bytes,fullblock bs=$count skip=$offset count=1 2>/dev/null + return $? +} + caldata_die() { echo "caldata: " "$*" exit 1 @@ -17,7 +27,7 @@ caldata_extract() { mtd=$(find_mtd_chardev $part) [ -n "$mtd" ] || caldata_die "no mtd device found for partition $part" - dd if=$mtd of=/lib/firmware/$FIRMWARE iflag=skip_bytes bs=$count skip=$offset count=1 2>/dev/null || \ + caldata_dd $mtd /lib/firmware/$FIRMWARE $count $offset || \ caldata_die "failed to extract calibration data from $mtd" } @@ -34,7 +44,7 @@ caldata_extract_ubi() { ubi=$(nand_find_volume $ubidev $part) [ -n "$ubi" ] || caldata_die "no UBI volume found for $part" - dd if=/dev/$ubi of=/lib/firmware/$FIRMWARE iflag=skip_bytes bs=$count skip=$offset count=1 2>/dev/null || \ + caldata_dd /dev/$ubi /lib/firmware/$FIRMWARE $count $offset || \ caldata_die "failed to extract calibration data from $ubi" } @@ -64,8 +74,7 @@ caldata_from_file() { [ -n "$target" ] || target=/lib/firmware/$FIRMWARE - # dd doesn't handle partial reads from special files: use cat - cat $source | dd of=$target iflag=skip_bytes bs=$count skip=$offset count=1 2>/dev/null || \ + caldata_dd $source $target $count $offset || \ caldata_die "failed to extract calibration data from $source" } @@ -73,16 +82,20 @@ caldata_sysfsload_from_file() { local source=$1 local offset=$(($2)) local count=$(($3)) + local target_dir="/sys/$DEVPATH" + local target="$target_dir/data" - # dd doesn't handle partial reads from special files: use cat - # test extract to /dev/null first - cat $source | dd of=/dev/null iflag=skip_bytes bs=$count skip=$offset count=1 2>/dev/null || \ - caldata_die "failed to extract calibration data from $source" + [ -d "$target_dir" ] || \ + caldata_die "no sysfs dir to write: $target" - # can't fail now - echo 1 > /sys/$DEVPATH/loading - cat $source | dd of=/sys/$DEVPATH/data iflag=skip_bytes bs=$count skip=$offset count=1 2>/dev/null - echo 0 > /sys/$DEVPATH/loading + echo 1 > "$target_dir/loading" + caldata_dd $source $target $count $offset + if [ $? != 0 ]; then + echo 1 > "$target_dir/loading" + caldata_die "failed to extract calibration data from $source" + else + echo 0 > "$target_dir/loading" + fi } caldata_valid() { |