Re: [PATCH 0/2 v3] Check if module name different than output filename

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/08/2016 11:02 AM, James Carter wrote:
Since CIL treats files as modules and does not have a separate
module statement it can cause confusion when a Refpolicy module
has a name that is not the same as its base filename because older
SELinux userspaces will refer to the module by its module name while
a CIL-based userspace will refer to it by its filename.

Because of this, provide a warning message when converting a policy
package to CIL and fail when compiling a module if the output base
filename is different than the module name.

Changes from v1:
- Added a "Warning:" prefix
- Removed checks against the input filename
- Since there are now only two checks and the base filename is used in the
warning message, it no longer made sense to create common helper functions
in libsepol.

Changes from v2:
- Check if strdup() returns NULL
- Have checkmodule fail rather than give a warning

James Carter (2):
   policycoreutils/hll/pp: Warn if module name different than output
     filename
   checkpolicy: Fail if module name different than output base filename

  checkpolicy/checkmodule.c   | 20 ++++++++++++++++++++
  policycoreutils/hll/pp/pp.c | 33 +++++++++++++++++++++++++++++----
  2 files changed, 49 insertions(+), 4 deletions(-)


Seeing no other comments or objections.

Merged.

--
James Carter <jwcart2@xxxxxxxxxxxxx>
National Security Agency
_______________________________________________
Selinux mailing list
Selinux@xxxxxxxxxxxxx
To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx.
To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.



[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux