[PATCH] Fix reconnection logic leaking unhandled requests

[Simo Sorce]

This took a while to find out, a nasty, nasty bug!

See details in git comment.

Fixes 1229

Simo.

[JR Aquino]

On Mar 7, 2012, at 3:27 PM, Simo Sorce wrote:

This took a while to find out, a nasty, nasty bug!

See details in git comment.

Fixes 1229

Simo.

Thank you VERY much for fixing this Simo!

It has been plaguing me intermittently for months!

This problem has been historically troublesome to reproduce.

How would you suggest I test the patch with 100% certainty?

Do you know of a sure fire way to reproduce the problem in order to verify the fix?

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Jr Aquino | Sr. Information Security Specialist
GIAC Certified Incident Handler | GIAC WebApp Penetration Tester
Citrix Online | 7408 Hollister Avenue | Goleta, CA 93117x-apple-data-detectors://0/0
T:  +1 805.690.3478tel:+1%C2%A0805.690.3478
C: +1 805.717.0365tel:+1%20805.717.0365
[e-mail]
http://www.citrixonline.com

[Stephen Gallagher]

See https://fedorahosted.org/sssd/ticket/1229 which has complete reproduction steps.

On Mar 7, 2012, at 9:14 PM, JR Aquino  wrote:
> On Mar 7, 2012, at 3:27 PM, Simo Sorce wrote:
>
> This took a while to find out, a nasty, nasty bug!
>
> See details in git comment.
>
> Fixes 1229
>
> Simo.
>
> Thank you VERY much for fixing this Simo!
>
> It has been plaguing me intermittently for months!
>
> This problem has been historically troublesome to reproduce.
>
> How would you suggest I test the patch with 100% certainty?
>
> Do you know of a sure fire way to reproduce the problem in order to verify the fix?
> Jr Aquino | Sr. Information Security Specialist
> GIAC Certified Incident Handler | GIAC WebApp Penetration Tester
> Citrix Online | 7408 Hollister Avenue | Goleta, CA 93117<x-apple-data-detectors://0/0>
> T:  +1 805.690.3478<tel:+1%C2%A0805.690.3478>
> C: +1 805.717.0365<tel:+1%20805.717.0365>
> [e-mail]<mailto:[e-mail]>
> http://www.citrixonline.com
> _______________________________________________
> sssd-devel mailing list
> [e-mail]
> https://fedorahosted.org/mailman/listinfo/sssd-devel

[Jakub Hrozek]

On Wed, Mar 07, 2012 at 06:27:10PM -0500, Simo Sorce wrote:
> This took a while to find out, a nasty, nasty bug!
>
> See details in git comment.
>
> Fixes 1229
>
> Simo.

Ack

[Stephen Gallagher]

On Thu, 2012-03-08 at 12:05 +0100, Jakub Hrozek wrote:
> On Wed, Mar 07, 2012 at 06:27:10PM -0500, Simo Sorce wrote:
> > This took a while to find out, a nasty, nasty bug!
> >
> > See details in git comment.
> >
> > Fixes 1229
> >
> > Simo.
>
> Ack

Pushed to master and sssd-1-8. And amazing work tracking that down,
Simo. Thanks!