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

Reply via email to