Re: [SSSD] [PATCHES] Fix more assorted bugs found by Coverity

2010-12-15 Thread Sumit Bose
On Tue, Dec 14, 2010 at 05:13:54PM -0500, Stephen Gallagher wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 12/14/2010 04:55 PM, Sumit Bose wrote: > > Hi, > > > > here are some more fixes for issues found by Coverity. Please have a > > careful look on the patch for ticket #727. I

Re: [SSSD] [PATCH] Make manual pages translatable

2010-12-15 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/14/2010 07:24 PM, Stephen Gallagher wrote: > I think you sent the wrong patch. I'm still seeing the 'make distcheck > failure' Sorry, you're right. Attached is a new patch, which Sumit helped me fix. As discussed on IRC, the for loop in the Make

[SSSD] [PATCHES] Fix even more bugs found by Coverity

2010-12-15 Thread Sumit Bose
Hi, here are some more fixes for bugs found by Coverity. bye, Sumit From 1e82b3a7570af268c107d9f50407515dd51906a4 Mon Sep 17 00:00:00 2001 From: Sumit Bose Date: Wed, 15 Dec 2010 14:26:16 +0100 Subject: [PATCH 1/4] Fix uninitialized value error in main() in stress-tests.c https://fedorahosted.o

Re: [SSSD] [PATCHES] Fix even more bugs found by Coverity

2010-12-15 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/15/2010 09:22 AM, Sumit Bose wrote: > Hi, > > here are some more fixes for bugs found by Coverity. > Ack and pushed to master. - -- Stephen Gallagher RHCE 804006346421761 Delivering value year after year. Red Hat ranks #1 in value among sof

Re: [SSSD] [PATCHES] Fix more assorted bugs found by Coverity

2010-12-15 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/15/2010 04:53 AM, Sumit Bose wrote: >> Thanks, new version is attached. I hope you don't mind, but I've fixed >> and indentation issue in this patch, too. I you prefer I can send this >> in a separate patch. In the future, please do send indent

Re: [SSSD] [PATCH] Make manual pages translatable

2010-12-15 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/15/2010 08:15 AM, Jakub Hrozek wrote: > On 12/14/2010 07:24 PM, Stephen Gallagher wrote: >> I think you sent the wrong patch. I'm still seeing the 'make distcheck >> failure' > > Sorry, you're right. Attached is a new patch, which Sumit helped m