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

bugfix: parse --debug before using it #6579

Merged
merged 1 commit into from
Dec 28, 2024

Conversation

powerjungle
Copy link
Contributor

@powerjungle powerjungle commented Dec 25, 2024

The arg_debug flag is set after some functions check for it.

arg_debug = 1;

An example of a function in this situation is check_kernel_procs().
int rv = check_kernel_procs();

if (arg_debug)

@powerjungle powerjungle force-pushed the fix/main/debug-arg-below branch from 72366d0 to 2de33f8 Compare December 25, 2024 18:57
@kmk3 kmk3 force-pushed the fix/main/debug-arg-below branch from 2de33f8 to cc9c6ed Compare December 27, 2024 03:19
@kmk3 kmk3 changed the title fix: main: add check for --debug at the start of main() bugfix: parse --debug before using it Dec 27, 2024
@powerjungle powerjungle force-pushed the fix/main/debug-arg-below branch 3 times, most recently from 32ac335 to d672dbd Compare December 27, 2024 16:13
@kmk3
Copy link
Collaborator

kmk3 commented Dec 28, 2024

I see that you reworded the commit message (and undid it).

I was trying to improve the grammar with my force-push, but now I'm not sure if
you meant just the check_kernel_procs() function or that multiple functions
are affected by this issue.

I noticed that the preproc_build_firejail_dir_unlocked() function (and maybe
the other ones below) seem to also be affected:

In which case the current version of the message should be accurate, but if
that's not the case, the message could be simplified to:

The `arg_debug` flag is being set after the `check_kernel_procs()` function
checks for it.

Anyway, sorry for the confusion.

@powerjungle
Copy link
Contributor Author

powerjungle commented Dec 28, 2024

I see that you reworded the commit message (and undid it).

It was by accident, I didn't notice that you force pushed and I tried to fix the message later.

I was trying to improve the grammar with my force-push, but now I'm not sure if
you meant just the check_kernel_procs() function or that multiple functions
are affected by this issue.

I just gave it as an example, but yes there are more functions.

In which case the current version of the message should be accurate

I think the current version is fine.

Anyway, sorry for the confusion.

Sorry for the confusion too, I rushed through writing the message and I didn't notice how unclear and full with mistakes it was. 😅

I will get to the review a bit later today.

The `arg_debug` flag is being set after some functions check for it,
such as `check_kernel_procs()`.
@powerjungle powerjungle force-pushed the fix/main/debug-arg-below branch from d672dbd to ba5ec2d Compare December 28, 2024 14:40
@kmk3 kmk3 merged commit 9bd077b into netblue30:master Dec 28, 2024
13 checks passed
@powerjungle powerjungle deleted the fix/main/debug-arg-below branch December 28, 2024 16:01
@kmk3 kmk3 added the bugfix This fixes a bug label Dec 29, 2024
kmk3 added a commit that referenced this pull request Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This fixes a bug
Projects
Status: Done (on RELNOTES)
Development

Successfully merging this pull request may close these issues.

2 participants