Skip to content

Edit the list of problematic formats#689

Merged
github-actions[bot] merged 2 commits intomainfrom
docs/disabled
Dec 25, 2024
Merged

Edit the list of problematic formats#689
github-actions[bot] merged 2 commits intomainfrom
docs/disabled

Conversation

@ghost
Copy link

@ghost ghost commented Dec 17, 2024

Describe your changes

Add dmg-opencl, rar-opencl to the list of problematic formats.

Document all formats that fail and therefore need to be disabled during testing.

Side effect of openwall/john#5613.

See also openwall/john#5615.

Checklist before requesting a review

  • I checked that all workflows return a success.
  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.
  • I followed the Conventional Commit spec.

Maintainer tasks

  • Label as either: bug, ci, docker, documentation, enhancement.
  • Sign unsigned commits.

@ghost ghost added documentation Improvements or additions to documentation ci Continuous integration and/or continuous delivery labels Dec 17, 2024
@ghost
Copy link
Author

ghost commented Dec 18, 2024

openwall/john#5615 fixed both formats.

Testing: dmg-opencl, Apple DMG [PBKDF2-SHA1 3DES/AES OpenCL]... PASS
Testing: rar-opencl, RAR3 (length 5) [SHA1 OpenCL AES]... (2xOMP) PASS

Source code now is probably trashing something and rawSHA512 isn't clearing the buffers properly before the test. I'll disable the raw-SHA512-free-opencl format using this PR.

Testing: raw-SHA512-free-opencl [SHA512 OpenCL (inefficient, development use mostly)]... FAILED (cmp_all(49))
Testing: raw-SHA512-opencl [SHA512 OpenCL]... PASS
[...]
1 out of 86 tests have FAILED

ghost pushed a commit that referenced this pull request Dec 18, 2024
Add raw-SHA512-free-opencl to the list of problematic formats.

Side effect of openwall/john#5613 and openwall/john#5615.

Document all formats that fail and therefore need to be disabled
during testing.

Signed-off-by: Claudio André <dev@claudioandre.slmail.me>
@ghost ghost force-pushed the docs/disabled branch from 5f9dc9e to 1fd24c1 Compare December 18, 2024 14:35
@ghost ghost changed the title Add more formats to issues (#689) Add more formats to issues Dec 19, 2024
ghost pushed a commit that referenced this pull request Dec 22, 2024
Remove OpenCL streebog/gost from the list of problematic formats.

Side effect of openwall/john#5627.

The formats have been corrected by
openwall/john@c35747b.

Signed-off-by: Claudio André <dev@claudioandre.slmail.me>
@ghost ghost changed the title Add more formats to issues Edit the list of problematic formats Dec 22, 2024
ghost pushed a commit that referenced this pull request Dec 22, 2024
Remove OpenCL streebog/gost from the list of problematic formats.

Side effect of openwall/john#5627.

The formats have been corrected by
openwall/john@469cf5a.

Signed-off-by: Claudio André <dev@claudioandre.slmail.me>
@ghost ghost force-pushed the docs/disabled branch from 52ca1e8 to bd81dcd Compare December 22, 2024 18:24
Claudio André added 2 commits December 25, 2024 09:49
Add raw-SHA512-free-opencl to the list of problematic formats.

Side effect of openwall/john#5613 and openwall/john#5615.

Document all formats that fail and therefore need to be disabled
during testing.

Signed-off-by: Claudio André <dev@claudioandre.slmail.me>
Remove OpenCL streebog/gost from the list of problematic formats.

Side effect of openwall/john#5627.

The formats have been corrected by
openwall/john@469cf5a.

Signed-off-by: Claudio André <dev@claudioandre.slmail.me>
@ghost ghost force-pushed the docs/disabled branch from bd81dcd to 6398d94 Compare December 25, 2024 12:49
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Auto approved PR

@ghost
Copy link
Author

ghost commented Dec 25, 2024

bot: MERGE

@github-actions github-actions bot merged commit 6398d94 into main Dec 25, 2024
@ghost ghost deleted the docs/disabled branch December 25, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous integration and/or continuous delivery documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants