Re: [PATCH 0/2] Add option to remove HLL files and fix memory leaks

[Steve Lawrence <slawrence@tresys.com>]

On 02/26/2015 02:15 PM, Yuli Khodorkovskiy wrote:
> This patchset adds a new option to semanage.conf to allow deleting HLL files after
> compilation into CIL. It also fixes a memory leak in semanage.conf that was exposed
> by the aforementioned patch.
> 
> An option to remove HLL files after compilation was originally discussed on the
> list [1]. If enabled, the module store will use about half as much space on disk.
> Additionally, less space will be used during a policy migration. Conversely, a
> user might not want to use this option if the HLL to CIL compiler changed. This 
> allows CIL files to be updated from undeleted HLL files in the module store when
> rebuilding with semodule. Otherwise, the HLL files must be reinstalled to get
> recompiled to CIL with an updated compiler.
> 
> [1] http://marc.info/?l=selinux&m=140562680432183&w=2
> 
> Yuli Khodorkovskiy (2):
>   libsemanage: Add option to remove HLL files after compilation
>   libsemanage: Fix memory leaks when parsing semanage.conf

Acked-by: Steve Lawrence <slawrence@tresys.com>

Thanks!

>  libsemanage/man/man5/semanage.conf.5 | 13 ++++++
>  libsemanage/src/conf-parse.y         | 28 +++++++++--
>  libsemanage/src/conf-scan.l          |  9 +---
>  libsemanage/src/direct_api.c         | 91 ++++++++++++++++++++++++------------
>  libsemanage/src/semanage_conf.h      |  1 +
>  5 files changed, 101 insertions(+), 41 deletions(-)
>