[PATCHES] Various ding-lib fixes

[Sumit Bose]

Hi,

this series of patches fixes some issues in ding-libs. Patch 0003 should
fix Coverity issues 10035-10040 and 0004 and 0005 the remaining open
issues.

bye,
Sumit

[Dmitri Pal]

Sumit Bose wrote:
> Hi,
>
> this series of patches fixes some issues in ding-libs. Patch 0003 should
> fix Coverity issues 10035-10040 and 0004 and 0005 the remaining open
> issues.
>
> bye,
> Sumit
>
> sssd-devel mailing list
> [e-mail]
> https://fedorahosted.org/mailman/listinfo/sssd-devel

I have some of these already fixed.
But I have a lot of the work pending. If you push the patches for the
rest of the tree it will really make my life very hard. It is not a high
priority since I have other things to work on now but as soon as I have
a moment I will get to the ding-lib patches that are currently in my
tree and resubmit them.
Please limit the scope of changes to the DHASH ones only.

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 01/24/2011 06:23 AM, Sumit Bose wrote:
> Hi,
>
> this series of patches fixes some issues in ding-libs. Patch 0003 should
> fix Coverity issues 10035-10040 and 0004 and 0005 the remaining open
> issues.

Ack for ding_libs-0-1 to patch 0003 only

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAk2J7XsACgkQeiVVYja6o6M7wQCeN/Y95/MowbVj5GbDCMhJYgmq
> ysAAoJDWtE8d7h8/CWdWZArhFhtnuA78
> =O5K4
> -----END PGP SIGNATURE-----

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 03/23/2011 08:54 AM, Stephen Gallagher wrote:
> On 01/24/2011 06:23 AM, Sumit Bose wrote:
> > Hi,
> > this series of patches fixes some issues in ding-libs. Patch 0003 should
> > fix Coverity issues 10035-10040 and 0004 and 0005 the remaining open
> > issues.
>
> Ack for ding_libs-0-1 to patch 0003 only

Pushed patch 0003 to ding_libs-0-1

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAk2J70cACgkQeiVVYja6o6NgRwCeIrTlOyrwLa7Ur51Wfn7ldDzc
> Y8EAn3sIz8rMa7QloGaXXgx+vd9GFcHi
> =bnIQ
> -----END PGP SIGNATURE-----

[Dmitri Pal]

On 03/23/2011 09:01 AM, Stephen Gallagher wrote:
> On 03/23/2011 08:54 AM, Stephen Gallagher wrote:
> > On 01/24/2011 06:23 AM, Sumit Bose wrote:
> > > Hi,
> > > this series of patches fixes some issues in ding-libs. Patch 0003
>
> should
> > > fix Coverity issues 10035-10040 and 0004 and 0005 the remaining open
> > > issues.
> > Ack for ding_libs-0-1 to patch 0003 only
>
> Pushed patch 0003 to ding_libs-0-1

These coverity issues have been fixed for quire a while in the series of
patches I provided 3 months ago.
Why are you redoing the work?

_______________________________________________
sssd-devel mailing list
[e-mail]
https://fedorahosted.org/mailman/listinfo/sssd-devel

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 03/23/2011 09:12 AM, Dmitri Pal wrote:
> On 03/23/2011 09:01 AM, Stephen Gallagher wrote:
> > On 03/23/2011 08:54 AM, Stephen Gallagher wrote:
> > > On 01/24/2011 06:23 AM, Sumit Bose wrote:
> > > > Hi,
> > > > this series of patches fixes some issues in ding-libs. Patch 0003
> >
> > should
> > > > fix Coverity issues 10035-10040 and 0004 and 0005 the remaining open
> > > > issues.
> > > Ack for ding_libs-0-1 to patch 0003 only
> >
> > Pushed patch 0003 to ding_libs-0-1
>
> These coverity issues have been fixed for quire a while in the series of
> patches I provided 3 months ago.
> Why are you redoing the work?

Your patches are 1) nacked and 2) only for master. These were for the
stable branch.

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAk2J9qEACgkQeiVVYja6o6PA4ACgkpvisFXSRGFoT1NMJkdxf+GY
> yewAn2n0gqrAhaNXLvcrAMpzRwCshmIb
> =imaO
> -----END PGP SIGNATURE-----

[Dmitri Pal]

On 03/23/2011 09:33 AM, Stephen Gallagher wrote:
> On 03/23/2011 09:12 AM, Dmitri Pal wrote:
> > On 03/23/2011 09:01 AM, Stephen Gallagher wrote:
> > > On 03/23/2011 08:54 AM, Stephen Gallagher wrote:
> > > > On 01/24/2011 06:23 AM, Sumit Bose wrote:
> > > > > Hi,
> > > > > this series of patches fixes some issues in ding-libs. Patch 0003
> > >
> > > should
> > > > > fix Coverity issues 10035-10040 and 0004 and 0005 the remaining open
> > > > > issues.
> > > > Ack for ding_libs-0-1 to patch 0003 only
> > >
> > > Pushed patch 0003 to ding_libs-0-1
> > These coverity issues have been fixed for quire a while in the series of
> > patches I provided 3 months ago.
> > Why are you redoing the work?
>
> Your patches are 1) nacked and 2) only for master. These were for the
> stable branch.

Not all of them were nacked.

_______________________________________________
sssd-devel mailing list
[e-mail]
https://fedorahosted.org/mailman/listinfo/sssd-devel