[SSSD] [sssd PR#498][comment] DESKPROFILE: Do not require CAP_DAC_OVERRIDE

2018-02-14 Thread olivergs
  URL: https://github.com/SSSD/sssd/pull/498
Title: #498: DESKPROFILE: Do not require CAP_DAC_OVERRIDE

olivergs commented:
"""
After testing this patch in a clean environment it worked like a charm.

Thanks!
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/498#issuecomment-365685736
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#241][comment] FleetCommander Integration

2017-08-28 Thread olivergs
  URL: https://github.com/SSSD/sssd/pull/241
Title: #241: FleetCommander Integration

olivergs commented:
"""
Tested the latest build for the patches from @fidencio COPR repo and it
worked as expected.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/241#issuecomment-325337915
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#241][comment] FleetCommander Integration

2017-08-28 Thread olivergs
  URL: https://github.com/SSSD/sssd/pull/241
Title: #241: FleetCommander Integration

olivergs commented:
"""
Tested the latest build for the patches from @fidencio COPR repo and it
worked as expected.



On Thu, Aug 17, 2017 at 4:23 PM, fidencio <notificati...@github.com> wrote:

> Patch set updated.
>
> This is the patch that I've squashed:
>
> From 97f189d3a4fcbbfe453cc6ba8eab8710ca31c4ac Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Fabiano=20Fid=C3=AAncio?= <fiden...@redhat.com>
> Date: Thu, 17 Aug 2017 13:29:24 +0200
> Subject: [PATCH] fixup! DESKPROFILE: Introduce the new IPA session provider
>
> ---
>  src/providers/ipa/ipa_deskprofile_rules_util.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/providers/ipa/ipa_deskprofile_rules_util.c 
> b/src/providers/ipa/ipa_deskprofile_rules_util.c
> index 12c6492ba..2ece5a1ee 100644
> --- a/src/providers/ipa/ipa_deskprofile_rules_util.c
> +++ b/src/providers/ipa/ipa_deskprofile_rules_util.c
> @@ -150,7 +150,7 @@ ipa_deskprofile_get_filename_path(TALLOC_CTX *mem_ctx,
>  extension,
>  NULL,
>  };
> -const uint8_t *perms = permuts[config_priority - 1];
> +const uint8_t *perms;
>  char *result;
>  errno_t ret;
>
> @@ -167,6 +167,8 @@ ipa_deskprofile_get_filename_path(TALLOC_CTX *mem_ctx,
>  goto done;
>  }
>
> +perms = permuts[config_priority - 1];
> +
>  result = talloc_strdup(tmp_ctx, "");
>  if (result == NULL) {
>  ret = ENOMEM;
> --
> 2.13.4
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/SSSD/sssd/pull/241#issuecomment-323102642>, or mute
> the thread
> <https://github.com/notifications/unsubscribe-auth/ABX4oQhaAXHRCzP43H7qrQ3aCbpAVR9Zks5sZFhZgaJpZM4NDNM2>
> .
>



-- 
Oliver Gutierrez
Associate Software Engineer - Desktop Management tools
Red Hat

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/241#issuecomment-325337915
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org