Re: (ITS#5931) back-meta crashes with syncrepl data

[hyc@symas.com]

Pierangelo Masarati wrote:
> ----- hyc@symas.com ha scritto:
>
>> hyc@symas.com wrote:
>>> I'll play with this code some with the decode change backed out.
>>>
>> Heh, in my current build I hadn't updated liblber yet so I didn't have
>> the
>> change. Fwiw, test036 etc. worked fine without it. Will try some
>> syncrepl
>> configs now.
>
> I ran test018, then I put a back-meta between the consumer and the
> producer,
I wiped the consumer's database and restarted it. Then I performed some
further modifications to the producer and checked that they appeared in the
consumer.

Right, I did the same first with test017. I've now repeated with test018, 
which works fine since the intermediate message carries the sync cookie.

> Please pay attention to the proxy's logs: what you could (should) get is a
series of messages complaining about the impossibility to decode attribute types.

I didn't see any such errors in the proxy's logs, and the consumer was fine.

> I might well have screwed something up while trying to fix intermediate
responses and other stuff related to this ITS. That's why I'm asking for
critical review before merging my fix into re24. With the change in liblber I
could perform syncrepl with back-meta in between.

refreshAndPersist seems to work fine without the liblber patch.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/