[SSSD] [PATCH 1/2] Reuse sss_open_cloexec at other places in code.

Lukas Slebodnik lslebodn at redhat.com
Fri Mar 8 16:44:23 UTC 2013


>From: "Jakub Hrozek" <jhrozek at redhat.com>
>To: sssd-devel at lists.fedorahosted.org
>Sent: Friday, March 8, 2013 5:08:59 PM
>Subject: Re: [SSSD] [PATCH 1/2] Reuse sss_open_cloexec at other places in code.
>
>On Fri, Mar 08, 2013 at 10:59:22AM -0500, Simo Sorce wrote:
>> On Fri, 2013-03-08 at 10:37 -0500, Lukas Slebodnik wrote:
>> > I forget to attach patches at first time.
>> > 
>> > ----- Original Message -----
>> > From: "Lukas Slebodnik" <lslebodn at redhat.com>
>> > To: "Development of the System Security Services Daemon" <sssd-devel at lists.fedorahosted.org>
>> > Sent: Friday, March 8, 2013 4:34:37 PM
>> > Subject: [SSSD] [PATCH 1/2] Reuse sss_open_cloexec at other places in code.
>> > 
>> > Hello,
>> > 
>> > attached patches should fix ticket
>> > https://fedorahosted.org/sssd/ticket/1794
>> 
>> 
>> Shouldn't we move ss_close_exec and friends into src/utils then ?
>> 
>> Simo.
>
>Definitely, files.c is supposed to be used by tools only. Maybe we could
>have something like utils/compat.c ?

Updated patches in attachments

LS
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Reuse-sss_open_cloexec-at-other-places-in-code.patch
Type: text/x-patch
Size: 8665 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130308/12acba40/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-More-generalized-function-open_debug_file_ex.patch
Type: text/x-patch
Size: 4422 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130308/12acba40/attachment-0001.bin>


More information about the sssd-devel mailing list