Skip to content

Commit

Permalink
seedrng: use posix positional params to avoid space splitting
Browse files Browse the repository at this point in the history
The value of ${seed_dir} may have spaces in it, making the current
argument string building method unsafe. Instead, use positional
parameters to pass these arguments safely.
  • Loading branch information
zx2c4 authored and vapier committed Apr 21, 2022
1 parent b5f7131 commit f81bb7e
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions init.d/seedrng.in
Original file line number Diff line number Diff line change
Expand Up @@ -19,26 +19,26 @@ depend()
provide urandom
}

seedrng_options()
seedrng_with_options()
{
[ -n "${seed_dir}" ] &&
echo "--seed-dir \"${seed_dir}\""
yesno "${skip_credit}" &&
echo "--skip-credit"
set --
[ -n "${seed_dir}" ] && set -- "$@" --seed-dir "${seed_dir}"
yesno "${skip_credit}" && set -- "$@" --skip-credit
seedrng "$@"
}

start()
{
ebegin "Seeding random number generator"
seedrng $(seedrng_options)
seedrng_with_options
eend $? "Error seeding random number generator"
return 0
}

stop()
{
ebegin "Saving random number generator seed"
seedrng $(seedrng_options)
seedrng_with_options
eend $? "Error saving random number generator seed"
return 0
}

0 comments on commit f81bb7e

Please sign in to comment.