Skip to content

Comments

FlipFrid - uprade#54

Merged
xMasterX merged 2 commits intoDarkFlippers:devfrom
Ganapati:dev
Sep 3, 2022
Merged

FlipFrid - uprade#54
xMasterX merged 2 commits intoDarkFlippers:devfrom
Ganapati:dev

Conversation

@Ganapati
Copy link

@Ganapati Ganapati commented Sep 1, 2022

What's new

  • Removed bad crc attack
  • Add fuzzing from existing dump
  • Changed UI

Copy link

@S2ciOnur S2ciOnur left a comment

Choose a reason for hiding this comment

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

LGTM

@azaitsev
Copy link
Contributor

azaitsev commented Sep 2, 2022

Why did you remove bad crc attack?

@xMasterX xMasterX merged commit f99c1a8 into DarkFlippers:dev Sep 3, 2022
@quantum-x
Copy link
Contributor

Hi @xMasterX -
Any chance to remove the hardcoded minimum "time_between_cards" ?
On certain readers, 0 delay causes overflows which opens the lock.
When the delay is forced to a minimum of 5s, this behaviour no longer works.

@Commmaned
Copy link

Instructions unclear now locked in Area 51

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants