On Tue, Sep 02, 2014 at 09:13:05AM +0200, Sumit Bose wrote:
On Mon, Sep 01, 2014 at 04:31:30PM +0200, Jakub Hrozek wrote:
> Hi,
>
> attached is a simple patch that fixes a Coverity warning.
This is the pattern we use elsewhere as well and the patch applies well,
ACK.
Pushed to master:
bd981608d8d174cd7309a992f393c43c1334eddd
bye,
Sumit
P.S. Maybe it is a good time to open a ticket to check the umask() usage
in the SSSD code, with a simple grep I can find the following variants:
umask(0000)
umask(0022)
umask(0077)
umask(0111)
umask(0133)
umask(0177)
umask(022)
umask(077)
I agree:
https://fedorahosted.org/sssd/ticket/2424