In general this series looks fine.

However, checkpatch.pl is complaining about DOS line endings in your patches:

For example:
ERROR: DOS line endings
#325: FILE: libselinux/src/label_file.h:281:
+^I^Iint alloc_stems = data->alloc_stems * 2 + 16;^M$





On Fri, Apr 13, 2018 at 1:34 PM, Nicolas Iooss <nicolas.io...@m4x.org> wrote:
> In getconlist.c, main() does not use usercon. Remove this variable.
>
> Signed-off-by: Nicolas Iooss <nicolas.io...@m4x.org>
> ---
>  libselinux/utils/getconlist.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/libselinux/utils/getconlist.c b/libselinux/utils/getconlist.c
> index abfe2c742bfb..5ac0ca85075c 100644
> --- a/libselinux/utils/getconlist.c
> +++ b/libselinux/utils/getconlist.c
> @@ -19,7 +19,7 @@ static __attribute__ ((__noreturn__)) void usage(const char 
> *name, const char *d
>
>  int main(int argc, char **argv)
>  {
> -       char **list, *usercon = NULL, *cur_context = NULL;
> +       char **list, *cur_context = NULL;
>         char *user = NULL, *level = NULL;
>         int ret, i, opt;
>
> @@ -69,7 +69,6 @@ int main(int argc, char **argv)
>                 freeconary(list);
>         }
>
> -       free(usercon);
>         free(level);
>
>         return 0;
> --
> 2.17.0
>
>

Reply via email to