On 11/12/2012 01:08 PM, Liu Yuan wrote: > +static bool journal_descriptor_valid(struct journal_descriptor *jd) > +{ > + if (jd->magic != JOURNAL_DESC_MAGIC) > + return false; > + if (jd->oid == 0) > + return false; Well, can OID be 0? I'm not quit sure about this. Thanks, Yuan