[PATCH] Use sss_unique_file instead of calling mkstemp directly

[Jakub Hrozek]

Hi,

there is a patchset that adds sss_unique_file with an optional
destructor:
    https://patchwork.acksyn.org/patch/11351/

Attached are patches that use the function instead of mkstemp(). As you
can see in the patches, we already fix some bugs this way, especially
removing the tmpfiles left behind on error. In some cases, I also added
closing the fd on error.

[Pavel Březina]

On 08/12/2015 02:23 PM, Jakub Hrozek wrote:
> Hi,
>
> there is a patchset that adds sss_unique_file with an optional
> destructor:
>      https://patchwork.acksyn.org/patch/11351/
>
> Attached are patches that use the function instead of mkstemp(). As you
> can see in the patches, we already fix some bugs this way, especially
> removing the tmpfiles left behind on error. In some cases, I also added
> closing the fd on error.

Ack.

[Jakub Hrozek]

On Thu, Aug 13, 2015 at 03:43:43PM +0200, Pavel Březina wrote:
> On 08/12/2015 02:23 PM, Jakub Hrozek wrote:
> > Hi,
> >
> > there is a patchset that adds sss_unique_file with an optional
> > destructor:
> >     https://patchwork.acksyn.org/patch/11351/
> >
> > Attached are patches that use the function instead of mkstemp(). As you
> > can see in the patches, we already fix some bugs this way, especially
> > removing the tmpfiles left behind on error. In some cases, I also added
> > closing the fd on error.
>
> Ack.

* master: 
    * 84493af37d4b57294e94b7bb0596dec51e06b7b0
    * 51ae9cb4ed85b60cfe00eaf6d3a4af39ed409ddc
    * df07d54f881e6210c9cb6650de5617e6a99602b9
    * f5db13d4462faa531c9924181f0fd51364647e2d