Permalink
Browse files

there was a forgotten change to test a special case

While testing the code to use the public key from
certificates instead of the box key, I made a mistake
and left over a comment to force a special path of
execution ... which prevents the use of private key
to extract the public parts for validation, but didn't
show a verification failure, if the certificate(s)
was/were found.
  • Loading branch information...
PeterPawn committed Jun 23, 2016
1 parent e46e1dd commit 5c5430cc6f72ff90a1887b4947bb4132f3a3370a
Showing with 2 additions and 3 deletions.
  1. +1 −1 README.md
  2. +1 −2 signimage/check_signed_image
View
@@ -43,5 +43,5 @@ useless and while Freetz is a really big solution, changing many aspects of the
changes and prefer a solution, which can make them more "under the hood" without interferences with the original
firmware.
-It's not possible the implement the final solution in one fell swoop ... but the building blocks are growing step
+It's not possible to implement the final solution in one fell swoop ... but the building blocks are growing step
by step and meanwhile I think, we should be able to test the first integrated version during this year.
@@ -171,8 +171,7 @@ read_box_key()
{
local rc=14 pkp found
pkp=$(which privatekeypassword)
-# found=$?
- found=1
+ found=$?
if [ $found -eq 0 ]; then
openssl rsa -in $box_key_name -pubout -passin pass:"$(privatekeypassword)" -out "$tmp/pubkey" 2>/dev/null
if [ $? -eq 0 ]; then

0 comments on commit 5c5430c

Please sign in to comment.