close
The Wayback Machine - https://web.archive.org/web/20220707125902/https://github.com/github/codeql/pull/9722
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.

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

Python: Timing attack #9722

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

ahmed-farid-dev
Copy link
Contributor

@ahmed-farid-dev ahmed-farid-dev commented Jun 27, 2022

A constant-time algorithm should be used for checking the value of info. In other words, the comparison time should not depend on the content of the input, Otherwise, an attacker may be able to implement a timing attacks that may reveal the value of sensitive info

it can be circumvented by using a constant-time algorithm for checking the value of sensitive info,
more precisely, the comparison time should not depend on the content of the input. Otherwise the attacker gains
information that is indirectly leaked by the application. This information is then used for malicious purposes,
such as guessing the password of a user.
Copy link
Contributor

@intrigus-lgtm intrigus-lgtm Jun 27, 2022

Choose a reason for hiding this comment

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

Do you have any information that suggests that such attacks (guessing the password of a user) are

  1. done in practice
  2. done over the internet
    ?

Copy link
Contributor Author

@ahmed-farid-dev ahmed-farid-dev Jun 28, 2022

Choose a reason for hiding this comment

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

Hi, @intrigus-lgtm
I never heard.

@tausbn tausbn self-assigned this Jun 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants