Hello,

manifest.c has this code:

    if( i>0 && strcmp(p->aField[i-1].zName, zName)>=0 ){
        goto manifest_syntax_error;
    }


But it doesn't take into account that names can have a '+' prepended
to the front of them.

This code fixes it:

        if (i>0){
            curName = zName;
            prevName = p->aField[i-1].zName;

            //strip off leading plus sign if it exists
            if (curName[0] == '+') curName++;
            if (prevName[0] == '+') prevName++;

            if (strcmp(prevName, curName)>=0){
                goto manifest_syntax_error;
            }
        }


Should I add a trouble ticket?
Should I submit a patch file?
I've already submitted a signed code release form, should I just check
the code in (my login no longer works)?

Thanks,
Erik Lechak
_______________________________________________
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to