Re: (ITS#8354) syncprov segv

[tpretz@gmail.com]

On Tue, Jan 26, 2016 at 6:49 PM, Quanah Gibson-Mount <quanah@zimbra.com> wrote:
> --On Saturday, January 23, 2016 2:43 PM +0000 tpretz@gmail.com wrote:
>
>> Thanks,
>>
>> Essentially the same patch as i had (destroyed sop->s_mutex in abandon
>> check rather than init later).
>>
>> I did place the o_abandon test under the while( si->si_active ) loop
>> instead of above just to ensure the check is under the same lock hold
>> as the list modification.
>>
>> Ill grab whats in git and start tests on monday
>
>
> Hi Tom,
>
> Are you able to confirm if the fix is good?  I'm working on 2.4.44, and want
> to be sure this fix is included.

I'v had an instance running with this patch for a few days, no SEGV,
looks good to me.

Thanks
>
> Thanks!
>
> --Quanah
>
>
> --
>
> Quanah Gibson-Mount
> Platform Architect
> Zimbra, Inc.
> --------------------
> Zimbra ::  the leader in open source messaging and collaboration