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.

Sign up for GitHub

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

Jump to bottom

check get_path_options for the right perms #12

Closed
bdekonin opened this issue Nov 8, 2022 · 2 comments
Closed

check get_path_options for the right perms #12

bdekonin opened this issue Nov 8, 2022 · 2 comments
Assignees
@lskywalker @rkieboom
Labels
invalid This doesn't seem right

Comments

@bdekonin
Copy link
Owner

bdekonin commented Nov 8, 2022

volgens mij crashde de server als je een bestandje --x------ rights gaf, vgm checken wij alleen op read. weet niet precies moeten er even naar kijken
@bdekonin
Copy link
Owner Author

bdekonin commented Nov 8, 2022 •
edited
Loading

in de functie Job::PATH_TYPE Job::get_path_options(std::string &uri) check ik op deze flag S_IXUSR. Klopt deze flag? want deze flag staat voor 'Execute by owner'. dus denk andere flag misschien. gister tijdens eval crashde hij ook als er een file met owner execution had

@bdekonin bdekonin added the invalid This doesn't seem right label Nov 8, 2022
@rkieboom
Copy link
Collaborator

rkieboom commented Nov 8, 2022

S_IRUSR instead of S_IXUSR

@rkieboom rkieboom closed this as completed Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lskywalker lskywalker

@rkieboom rkieboom

Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lskywalker @rkieboom @bdekonin