On Thu, Mar 28, 2024 at 7:24 AM Roberto Sassu <roberto.sassu@xxxxxxxxxxxxxxx> wrote: > On 3/28/2024 12:08 PM, Christian Brauner wrote: ... > > And one another thing I'd like to point out is that the security hook is > > called "security_path_post_mknod()" while the evm and ima hooks are > > called evm_post_path_mknod() and ima_post_path_mknod() respectively. In > > other words: > > > > git grep _path_post_mknod() doesn't show the implementers of that hook > > which is rather unfortunate. It would be better if the pattern were: > > > > <specific LSM>_$some_$ordered_$words() > > I know, yes. Didn't want to change just yet since people familiar with > the IMA code know the current function name. I don't see any problem to > rename the functions. I'm sure this is what you are planning Roberto, but just so we are all clear on this, please do the simple bugfix to resolve the mknod problem and then do the parameter change and the name bikeshedding in a separate patch. -- paul-moore.com
