On Apr 15, 2008, at 2:10 PM, Jeff Johnson wrote:

>
>> rpm: depends.c:1776: zapRelation: Assertion `tsi->tsi_reqx >= 0 &&
>> tsi->tsi_reqx < rpmdsCount(requires)' failed.
>> Avbrutt (SIGABRT)
>>
>
> Bingo. Thanks. Now to find where tsi->tsi_reqx goes awry ...
>

Hmmm, there's only one place that tsi->tsi_reqx is set.

And AFAICT, addRelation() is called only within an rpmds iteration,
where the value used should be valid. But the code paths are twistier
than I would like because the iteration index is accessed in the  
addRelation()
subroutine.

Can I get you to try reproducing with one more assertion?

Here is the patch:

@@ -1902,6 +1903,7 @@ static inline int addRelation(rpmts ts,

      tsi->tsi_tagn = rpmdsTagN(requires);
      tsi->tsi_reqx = rpmdsIx(requires);
+assert(tsi->tsi_reqx >= 0 && tsi->tsi_reqx < rpmdsCount(requires));

      tsi->tsi_next = rpmteTSI(q)->tsi_next;
      rpmteTSI(q)->tsi_next = tsi;

If the new assertion fires, then I'm not reading the rpmds iteration  
code correctly.

If the old assertion fires, then there's a wild pointer somewhere.

Thanks.

73 de Jeff