Skip to content

umask 000 leads to mxqsub.c:510: add_job: Assertion 'j->job_umask' failed. #145

Closed
pmenzel opened this issue Aug 7, 2023 · 1 comment · Fixed by #151
Closed

umask 000 leads to mxqsub.c:510: add_job: Assertion 'j->job_umask' failed. #145

pmenzel opened this issue Aug 7, 2023 · 1 comment · Fixed by #151

Comments

@pmenzel
Copy link
Contributor

pmenzel commented Aug 7, 2023

A user set their umask to 000, triggering the assert below:

mxq/mxqsub.c

Line 510 in f3d9fb8

assert(j->job_umask);

Perhaps a user understandable log messages should be shown.

@pmenzel pmenzel changed the title umask 000 leads to mxqsub.c:510: add_job: Assertion j->job_umask' failed.` umask 000 leads to mxqsub.c:510: add_job: Assertion \j->job_umask' failed.` Aug 7, 2023
@pmenzel pmenzel changed the title umask 000 leads to mxqsub.c:510: add_job: Assertion \j->job_umask' failed.` umask 000 leads to mxqsub.c:510: add_job: Assertion 'j->job_umask' failed. Aug 7, 2023
@donald
Copy link
Contributor

donald commented Aug 7, 2023

I'd say that assert should just be removed, because 000 is a valid umask.

This was referenced Dec 30, 2023
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants