Skip to content

Bail out if user error bc of umask#2450

Merged
drwetter merged 3 commits into3.2from
fix_stupid_umaskerror
Dec 23, 2023
Merged

Bail out if user error bc of umask#2450
drwetter merged 3 commits into3.2from
fix_stupid_umaskerror

Conversation

@drwetter
Copy link
Copy Markdown
Collaborator

If a user chose a broken umask testssl.sh will start but emits subsequent errors.

This patch adds two sanity checks whether it is allowed to create and read files in the temp directory.

Fixes #2449

If a user chose a broken umask testssl.sh will start but emits
subsequent errors.

This patch adds two sanity checks whether it is allowed to create
and read files in the temp directory.

Fixes #2449
@drwetter drwetter merged commit 3d63b62 into 3.2 Dec 23, 2023
@drwetter drwetter deleted the fix_stupid_umaskerror branch December 23, 2023 14:02
drwetter added a commit that referenced this pull request Dec 24, 2023
Implemnation for 3.0, 3.2 see #2450

If a user chose a broken umask testssl.sh will start but emits subsequent errors.
This patch adds two sanity checks whether it is allowed to create and read files in the temp directory.

Fixes #2449
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.

Creation of tempdir assumes certain umasks

1 participant