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.

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

Use 'aa-complain' instead of 'complain' to remove dnsmasq restriction #314

Merged

Conversation

TiPEX360
Copy link
Contributor

Using the default installation from the AUR, I ran into the following error when trying to create a hotspot:

dnsmasq: cannot read /tmp/create_ap.wlp18s0.conf.fgQTo8iD/dnsmasq.conf: Permission denied

This looks like it's related related to an old issue with create_ap discussed here, where @mehdisadeghi has found a solution, although it seems it hasn't been implemented.

Seems like complain is being used to remove restrictions on dmsmasq, although the command is supposed to be aa-complain. At least, this has been the case when testing on my system (Manjaro) as well as on Arch according to that thread.

From the OpenSUSE docs, it looks like aa-complain is the correct command on OpenSUSE distros as well, so I think it makes sense to make this change: https://doc.opensuse.org/documentation/leap/security/html/book-security/cha-apparmor-commandline.html

I imagine this requires a bit more testing though.

@lakinduakash
Copy link
Owner

Thanks for the contribution, This also needs to be compatible with complain command as well.

@TiPEX360
Copy link
Contributor Author

TiPEX360 commented May 15, 2023

Thanks for the contribution, This also needs to be compatible with complain command as well.

Thanks for the response; I've made some changes to check for both.
Do you happen to know in what context / which systems complain is used instead of aa-complain? I tried to find out more about it online but most references point me to aa-complain.

@lakinduakash
Copy link
Owner

lakinduakash commented May 15, 2023

Thanks. I think you missed the $. I fixed it.

@lakinduakash
Copy link
Owner

closes #269 #304 #218 #183

@niksingh710
Copy link

@lakinduakash when will this be merged and released in the build?

@lakinduakash lakinduakash merged commit 6a236ac into lakinduakash:master May 17, 2023
1 check passed
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.

None yet

3 participants