diff options
author | Paul Taysom <taysom@chromium.org> | 2013-05-16 14:27:09 -0700 |
---|---|---|
committer | ChromeBot <chrome-bot@google.com> | 2013-05-21 14:00:46 -0700 |
commit | 01bc003cff03ea1e720ef25c4276ee65d5dce5fd (patch) | |
tree | cd5ebf67baca0cc38037865d288e9880bc8c4bb8 /scripts | |
parent | ed92adaddfc5407d5bdd2a9a5fd51df6591e8ded (diff) | |
download | vboot-01bc003cff03ea1e720ef25c4276ee65d5dce5fd.tar.gz |
Changed to use sed for regex pattern matching
The code was using a mixture of shell patterns and
sed regular expression. Changed code to just used
sed regex. This will let me use more complex regex
that are needed for adding PARTUUID to uuids.
BUG=chromium:241572
TEST=./security_test_image
BRANCH=none
Change-Id: I2e8570051b31d580ae36cb82f05383a33236b88e
Reviewed-on: https://gerrit.chromium.org/gerrit/51503
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Paul Taysom <taysom@chromium.org>
Tested-by: Paul Taysom <taysom@chromium.org>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/image_signing/ensure_secure_kernelparams.sh | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/scripts/image_signing/ensure_secure_kernelparams.sh b/scripts/image_signing/ensure_secure_kernelparams.sh index 7d9c050d..2b13cff6 100755 --- a/scripts/image_signing/ensure_secure_kernelparams.sh +++ b/scripts/image_signing/ensure_secure_kernelparams.sh @@ -65,6 +65,8 @@ main() { local testfail=0 # A buffer to include useful information that we dump when things fail. local output + # Copy of a string before it has been through sed + local pre_sed if [[ $# -ne 1 ]] && [[ $# -ne 2 ]]; then usage @@ -170,15 +172,12 @@ main() { # Ensure all other required regex params are present. for param in "${required_kparams_regex[@]}"; do - if [[ "$kparams_nodm" != *$param* ]]; then - echo "Kernel parameters missing required value: $param" - testfail=1 - else - # Remove matched params as we go. If all goes well, kparams_nodm - # will be nothing left but whitespace by the end. - kparams_nodm=$(echo " ${kparams_nodm} " | - sed "s${M} ${param} ${M} ${M}") - fi + pre_sed=" ${kparams_nodm} " + kparams_nodm=$(echo "${pre_sed}" | sed "s${M} ${param} ${M} ${M}") + if [[ "${pre_sed}" == "${kparams_nodm}" ]]; then + echo "Kernel parameters missing required value: ${param}" + testfail=1 + fi done # Check-off each of the allowed-but-optional params that were present. |