Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test: patterns in LR after key change #3

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Test: patterns in LR after key change #3

wants to merge 4 commits into from

Conversation

taramka
Copy link
Collaborator

@taramka taramka commented Jan 9, 2025

Closes #2

@taramka taramka self-assigned this Jan 9, 2025
mbroz
mbroz reviewed Jan 12, 2025
View reviewed changes
test_rekey_patterns Outdated

echo
echo "REVERT PSID"
sudo $CONTROL psid_revert $DEV --verify-pin=$PSID
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should fail if PSID reset was not finished ok.
Similar to other commands.

Also, it should perhaps fully reset the device in the end.

test_rekey_patterns
function compare_imgs()
{
for f in $(ls *.img); do
sha256sum -b $f | tr -d '\n'
Copy link
Member

@mbroz mbroz Jan 12, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better add test that sha256sum and tr commands are available ( command -v <binary>).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mbroz mbroz mbroz left review comments

Assignees

@taramka taramka

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Test: patterns in LR after key change
2 participants
@taramka @mbroz