Skip to content

security: apparmor: apparmorfs.c Checkpatch mods

Walt Feasel kernelnewbies 1 version New

Get series with b4
b4 am 20161126165617.GA9754@localhost

Versions newest first

  1. v1 Latest0 of 7 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 7 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/7] security: apparmor: apparmorfs.c Checkpatch mods
Quoted 20 lines · Greg KH
Quoted 10 lines · Walt Feasel

Make checkpatch modifications to include:

Multiple blank lines
Blank line after declarations
Space after cast
WARN_ON vs BUG_ON
Align parenthesis
Comparison to NULL
Line continuations

Quoted 8 lines

Walt Feasel (7):
security: apparmor: apparmorfs.c Multiple blank lines
security: apparmor: apparmorfs.c Blank line after declarations
security: apparmor: apparmorfs.c Space after cast
security: apparmor: apparmorfs.c WARN_ON vs BUG_ON
security: apparmor: apparmorfs.c Align parenthesis
security: apparmor: apparmorfs.c Comparison to NULL
security: apparmor: apparmorfs.c Line continuations

Quoted 6 lines

Why are you sending apparmor patches to the kernelnewbies list and not the maintainers and subsystem list for this code instead?

thanks,

greg k-h

Because as stated in my patch notes I am not sure how to fix certain cases.
I have not been able to find reference material other than other patches and discussions for certain cases and have to try and reason why.
You wanted me to move out of staging so I have looked to other areas but do not wish to send in ridiculous patches and have me seen as being malicious.
Since this is kernelnewbies mailing list isn't that it's purpose or am I move on again?
Feel free to skip my post should they trouble you.

Read the cover letter in its thread