On Wed, 15.02.12 17:26, Roberto Sassu (roberto.sassu@xxxxxxxxx) wrote: > > On 02/15/2012 03:30 PM, Gustavo Sverzut Barbieri wrote: > >On Wed, Feb 15, 2012 at 11:23 AM, Roberto Sassu<roberto.sassu@xxxxxxxxx> wrote: > >>The new function ima_setup() loads an IMA custom policy from a file in the > >>default location '/etc/sysconfig/ima-policy', if present, and writes it to > > > >isn't /etc/sysconfig too specific to Fedora? > > > > Hi Gustavo > > probably yes. I see the code in 'src/locale-setup.c' where the > the configuration directory depends on the target distribution. > I can implement something like that in my patch. We will sooner or later drop the per-distro ifdeffery. Please don't even start it for new code. Given that IMA is still new, please make sure to adopt configuration fails that are the same across all distributions. > >Also, I certainly have no such things in my system and see no point in > >calling ima_setup() on it. Or even compiling the source file in such > >case. > > > > Ok. I can enclose the code in ima-setup.c within an 'ifdef HAVE_IMA' > statement, as it happens for SELinux. However an issue is that there > is no a specific package for IMA that can be checked to set the > HAVE_IMA > definition to yes. Instead, the code can be enabled for example by > adding the parameter '--enable_ima' in the configure script. Sounds good. Lennart -- Lennart Poettering - Red Hat, Inc. -- To unsubscribe from this list: send the line "unsubscribe initramfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
