[SSSD] [sssd PR#103][+Pushed] sudo: do not store usn if no rules are found

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/103 Title: #103: sudo: do not store usn if no rules are found Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#103][-Accepted] sudo: do not store usn if no rules are found

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/103 Title: #103: sudo: do not store usn if no rules are found Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#103][comment] sudo: do not store usn if no rules are found

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/103 Title: #103: sudo: do not store usn if no rules are found jhrozek commented: """ master: 46703740e83a66909974a5ee8d47df6a6e5076e7 sssd-1-14: 76e97affaa05ce45709efd59d120595c5992aa21 sssd-1-13: 4e25db79aa514e09c8ad4482c45b24e7a3d4 """

[SSSD] [sssd PR#103][closed] sudo: do not store usn if no rules are found

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/103 Author: pbrezina Title: #103: sudo: do not store usn if no rules are found Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/103/head:pr103 git checkout pr103

[SSSD] [sssd PR#103][comment] sudo: do not store usn if no rules are found

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/103 Title: #103: sudo: do not store usn if no rules are found jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/58/31/summary.html The rawhide failure is unrelated.. """ See the full comment at

[SSSD] Re: [sssd PR#100][synchronized] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2016-12-08 Thread amit kumar
Hello, kindly review https://github.com/SSSD/sssd/pull/100 commit f8f208f6f1a12e25d46b80459d87ae269924117d Thanks On 12/08/2016 11:12 AM, amitkumar50 wrote: >URL: https://github.com/SSSD/sssd/pull/100 > Author: amitkumar50 > Title: #100: Updation of sssd-ad man page for case when >

[SSSD] [sssd PR#106][comment] Add a new "files" provider

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/106 Title: #106: Add a new "files" provider jhrozek commented: """ One suggestion came from simo about using FGETPWENT(3) instead of opening the NSS module directly. """ See the full comment at

[SSSD] [sssd PR#102][comment] Sssctl no case sensitive searches

2016-12-08 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/102 Title: #102: Sssctl no case sensitive searches mzidek-rh commented: """ Just updated the patches. """ See the full comment at https://github.com/SSSD/sssd/pull/102#issuecomment-265736056 ___ sssd-devel

[SSSD] [sssd PR#102][synchronized] Sssctl no case sensitive searches

2016-12-08 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/102 Author: mzidek-rh Title: #102: Sssctl no case sensitive searches Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/102/head:pr102 git checkout pr102 From

[SSSD] Re: [PATCH] Unit tests for pam_sss using pam_wrapper (need help with CI..)

2016-12-08 Thread Lukas Slebodnik
On (09/11/16 14:12), Lukas Slebodnik wrote: >On (11/10/16 19:56), Jakub Hrozek wrote: >>On Tue, Sep 13, 2016 at 09:27:32AM +0200, Lukas Slebodnik wrote: >>> On (09/08/16 10:18), Jakub Hrozek wrote: >>> >On Tue, Aug 09, 2016 at 08:04:38AM +0200, Lukas Slebodnik wrote: >>> >> On (09/05/16 10:07),

[SSSD] [sssd PR#95][closed] STAP: Check the number of transactions in the stap aggregate before dereferencing it.

2016-12-08 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/95 Author: jhrozek Title: #95: STAP: Check the number of transactions in the stap aggregate before dereferencing it. Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/95/head:pr95 git

[SSSD] [sssd PR#95][+Pushed] STAP: Check the number of transactions in the stap aggregate before dereferencing it.

2016-12-08 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/95 Title: #95: STAP: Check the number of transactions in the stap aggregate before dereferencing it. Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email

[SSSD] [sssd PR#95][comment] STAP: Check the number of transactions in the stap aggregate before dereferencing it.

2016-12-08 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/95 Title: #95: STAP: Check the number of transactions in the stap aggregate before dereferencing it. lslebodn commented: """ On (27/11/16 23:54), fidencio wrote: >Patch looks good but the indentation level seems wrong. >Feel free to fix it before pushing

[SSSD] [sssd PR#100][comment] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2016-12-08 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/100 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds amitkumar50 commented: """ Kindly review this recent Pull request.. commit f8f208f6f1a12e25d46b80459d87ae269924117d """ See the full comment at

[SSSD] [sssd PR#102][+Changes requested] Sssctl no case sensitive searches

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/102 Title: #102: Sssctl no case sensitive searches Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#103][comment] sudo: do not store usn if no rules are found

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/103 Title: #103: sudo: do not store usn if no rules are found jhrozek commented: """ ACK, thanks for the patch. I will push it once full CI finishes. """ See the full comment at https://github.com/SSSD/sssd/pull/103#issuecomment-265726131

[SSSD] [sssd PR#103][+Accepted] sudo: do not store usn if no rules are found

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/103 Title: #103: sudo: do not store usn if no rules are found Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#89][comment] nss: rewrite nss responder so it uses cache_req

2016-12-08 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req pbrezina commented: """ The crash was caused by the same issue as the valgrind log. It is fixed with: ```c diff --git a/src/responder/nss/nss_enum.c b/src/responder/nss/nss_enum.c index

[SSSD] [sssd PR#89][comment] nss: rewrite nss responder so it uses cache_req

2016-12-08 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req pbrezina commented: """ The crash was caused by the same issue as the valgrind log. It is fixed with: ```c diff --git a/src/responder/nss/nss_enum.c b/src/responder/nss/nss_enum.c index

[SSSD] [sssd PR#99][+Pushed] Prevent use after free in fd_input_available

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/99 Title: #99: Prevent use after free in fd_input_available Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#99][comment] Prevent use after free in fd_input_available

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/99 Title: #99: Prevent use after free in fd_input_available jhrozek commented: """ Since the patch looks good to me, apparently fixes a crash an none of the maintainers have another opinion, I'm ACKing the patch. """ See the full comment at

[SSSD] [sssd PR#99][closed] Prevent use after free in fd_input_available

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/99 Author: chlunde Title: #99: Prevent use after free in fd_input_available Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/99/head:pr99 git checkout pr99

[SSSD] [sssd PR#99][comment] Prevent use after free in fd_input_available

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/99 Title: #99: Prevent use after free in fd_input_available jhrozek commented: """ Fixed upstream: master: 9676b464dd428557ff5a648e1351a3972440396f sssd-1-14: fefdd70237cbe82af7d8845131e45401e73b3b07 sssd-1-13:

[SSSD] [sssd PR#99][-Accepted] Prevent use after free in fd_input_available

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/99 Title: #99: Prevent use after free in fd_input_available Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#99][+Accepted] Prevent use after free in fd_input_available

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/99 Title: #99: Prevent use after free in fd_input_available Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#97][comment] IFP: Remove "ChangeDebugTemporarily" method

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/97 Title: #97: IFP: Remove "ChangeDebugTemporarily" method jhrozek commented: """ * master: 78b4b7e5ec55861c43775581c08ae1804cd865f0 """ See the full comment at https://github.com/SSSD/sssd/pull/97#issuecomment-265710166

[SSSD] [sssd PR#97][closed] IFP: Remove "ChangeDebugTemporarily" method

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/97 Author: fidencio Title: #97: IFP: Remove "ChangeDebugTemporarily" method Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/97/head:pr97 git checkout pr97

[SSSD] [sssd PR#97][-Accepted] IFP: Remove "ChangeDebugTemporarily" method

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/97 Title: #97: IFP: Remove "ChangeDebugTemporarily" method Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#97][+Pushed] IFP: Remove "ChangeDebugTemporarily" method

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/97 Title: #97: IFP: Remove "ChangeDebugTemporarily" method Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#104][closed] Suppress a confusing error message from timestamp cache updates

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/104 Author: jhrozek Title: #104: Suppress a confusing error message from timestamp cache updates Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/104/head:pr104 git checkout pr104

[SSSD] [sssd PR#104][-Accepted] Suppress a confusing error message from timestamp cache updates

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/104 Title: #104: Suppress a confusing error message from timestamp cache updates Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#104][+Pushed] Suppress a confusing error message from timestamp cache updates

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/104 Title: #104: Suppress a confusing error message from timestamp cache updates Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#104][comment] Suppress a confusing error message from timestamp cache updates

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/104 Title: #104: Suppress a confusing error message from timestamp cache updates jhrozek commented: """ * master: ee576602d8b46b313c4f7ac8324cc31faefae46a """ See the full comment at https://github.com/SSSD/sssd/pull/104#issuecomment-265709337

[SSSD] [sssd PR#105][closed] RESPONDER: Remove dead assignment to the variable ret

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/105 Author: lslebodn Title: #105: RESPONDER: Remove dead assignment to the variable ret Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/105/head:pr105 git checkout pr105

[SSSD] [sssd PR#105][-Accepted] RESPONDER: Remove dead assignment to the variable ret

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/105 Title: #105: RESPONDER: Remove dead assignment to the variable ret Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#105][comment] RESPONDER: Remove dead assignment to the variable ret

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/105 Title: #105: RESPONDER: Remove dead assignment to the variable ret jhrozek commented: """ * master: 3d5bf48ac5b8b807facbfda225cdebff2f685cb8 """ See the full comment at https://github.com/SSSD/sssd/pull/105#issuecomment-265709046

[SSSD] [sssd PR#105][+Pushed] RESPONDER: Remove dead assignment to the variable ret

2016-12-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/105 Title: #105: RESPONDER: Remove dead assignment to the variable ret Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#89][comment] nss: rewrite nss responder so it uses cache_req

2016-12-08 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req lslebodn commented: """ Another crash from AD related test. 0xbe was initialized either by `MALLOC_PERTRUB` or `TALLOC_FREE_FILL` ``` #0 0x7fa2dd6a8423 in setent_notify

[SSSD] [sssd PR#89][comment] nss: rewrite nss responder so it uses cache_req

2016-12-08 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req pbrezina commented: """ On 12/08/2016 09:50 AM, lslebodn wrote: > Looks like there is use after free in latest version. sorry do not have > a reproducer yet; just a valgrind output > >

[SSSD] [sssd PR#89][comment] nss: rewrite nss responder so it uses cache_req

2016-12-08 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req lslebodn commented: """ Looks like there is use after free in latest version. sorry do not have a reproducer yet; just a valgrind output ``` ==6612== 18 errors in context 1 of 1: ==6612==