Skip to content
This repository has been archived by the owner on Mar 29, 2022. It is now read-only.

Hardware ID and Release Counter (Resolved Merge Conflicts, Cleaner Code and Tests) #157

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

shikhar394
Copy link
Contributor

@shikhar394 shikhar394 commented Jan 27, 2018

Purpose of the PR:

This is new PR that reflects changes from a previous PR for Hardware ID and Release Counter.

Includes code that implements hardware IDs and Release Counters for ECUs and implemented 3 attacks:

Image_rollback_attack : A compromised director attempting to install an image with an outdated release counter.
Confused_bundle_attack : A compromised director attempting to install an image with mismatching release counters.
Sneaky_director_attack : A compromised director attempting to install an image with mismatching hardware IDs. Basically, intending to install an image on an ECU which it was not meant for through leveraging the ECU Serial.

All these attacks can be called in the script through the following, please see the function definitions for more advanced function calls:
The names correspond with the usual demos.
image_rollback_attack
di.image_rollback_attack("Attack1.img")
dd.image_rollback_attack("Attack1.img")
dp.update_cycle()
ds.update_cycle()

confused_bundle_attack
di.confused_bundle_attack("Attack2.img")
dd.confused_bundle_attack("Attack2.img")
dp.update_cycle()
ds.update_cycle()

sneaky_director_attack
di.sneaky_director_attack("Attack3.img")
dd.sneaky_director_attack("Attack3.img")
dp.update_cycle()

Summary of Changes:

  • Refactored and cleaner code
  • Solved merge conflicts
  • Tests

Further Requirements:

More Testing

Greetings,
I have implemented release counters and hardwareIDs in all the relevant Uptane modules, including errors and exceptions linked to it.

@shikhar394 shikhar394 changed the title Hardware_ID and Release Counter (Resolved Merge Conflicts, Cleaner Code and Tests) Hardware ID and Release Counter (Resolved Merge Conflicts, Cleaner Code and Tests) Feb 6, 2018
@awwad
Copy link
Contributor

awwad commented Feb 22, 2018

For previous comments, see #117, which I'm now closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants