On 1/10/19 7:28 PM, Russell Coker wrote:
On Friday, 11 January 2019 10:08:22 AM AEDT Nicolas Iooss wrote:
I guess this is a race between patches. Should be named
init_read_pid_symlinks. Same feedback as GitHub PR
https://github.com/SELinuxProject/refpolicy/pull/17
I like "init_read_runtime_symlinks" more than "init_read_pid_symlinks"
so I have uploaded my Pull Request. If your prefer the second name, I
can change it again.
Compared to this patch, my PR does not use files_search_pids() but
directly read_lnk_files_pattern() like some other interfaces. I have
no strong preferences regarding this.
I'm really not bothered by the name. Whatever gets it included in git the
fastest is good for me.
Also I'd really like to see a new upstream release before the end of the month
if convenient.
I've merged the PR, though this means the init_*_pid_*() interfaces will
need to be deprecated and replaced, for consistency.
--
Chris PeBenito
