Skip to content

tests: Fix should-fail-self-hardlink to conform to latest syntax#374

Merged
alexlarsson merged 1 commit intocomposefs:mainfrom
cgwalters:fix-should-fail-self-hardlink
Sep 30, 2024
Merged

tests: Fix should-fail-self-hardlink to conform to latest syntax#374
alexlarsson merged 1 commit intocomposefs:mainfrom
cgwalters:fix-should-fail-self-hardlink

Conversation

@cgwalters
Copy link
Copy Markdown
Contributor

This was generated by honggfuzz, and only later did I do a patch which denied embedded NUL characters. Update and clean this up so it fails with the expected error.

Closes: #373

This was generated by honggfuzz, and only later did I do a patch
which denied embedded NUL characters. Update and clean this up
so it fails with the expected error.

Closes: composefs#373

Signed-off-by: Colin Walters <walters@verbum.org>
Copy link
Copy Markdown
Contributor

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

I can confirm this fixes #373.

@alexlarsson alexlarsson merged commit c77ecd9 into composefs:main Sep 30, 2024
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.

Testing the file should-fail-self-hardlink.dump fails with mkcomposefs: Invalid embedded NUL character at position 22

3 participants