On Tue Sep 04 11:40:30 2007, rblasch wrote:
>
> The key here is the "model." While Coverity's model captures the
> C<free> quite correctly, I don't think it recognizes the pointer update
> in the double linked list, which is done in C<subst_ins>, as important.
>
> Coverity probably sees something like the following in the inspected code:
>
> Instruction *ins, *ins2;
> for (ins = unit->instructions; ins; ins = ins->next) {
> ins2 = ins->next;
> free(ins2);
> }
>
> So, it's a false positive.
>
> Ron
>
Since this is a false positive, is there any reason to keep this ticket
open?
Christoph