On 2/22/2011 11:55 PM, Angus Salkeld wrote:
> On Tue, Feb 22, 2011 at 12:48:32PM -0700, Steven Dake wrote:
>> If the ring id file for the processor is less then 8 bytes, totemsrp would
>> assert.  Our speculation is that this condition happens during a fencing
>> operation or local filesystem corruption.
>>
>> With this patch, Corosync will create fresh ring id file data when the
>> incorrect number of bytes are read from the ring id.
>>
>> Amend to use sizeof the strerror string length and PATH_MAX for the path 
>> length.
>>
>> Signed-off-by: Steven Dake <[email protected]>
> Reviewed-by: Angus Salkeld <[email protected]>
> 
> I hope the change from:
> read (fd, &memb_ring_id->seq, sizeof (unsigned long long));
> to
> read (fd, &memb_ring_id->seq, sizeof (uint64_t));
> 
> won't cause any problems.

On i686 i noticed the file being 8 bytes (unsigned long long)... I
wonder if you shutdown corosync, update packages with the fix, then
restart.. is it going to read garbage from the file?

Fabio
_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to