"item" correctly reflects that in these loops, the object in question
is not an iterator but a reference to the actual object in the
collection.

This fixes the crash with pennytel SIP provider.

We were only ever looking at the first format (i.e. codec) for each media.

A call can send one codec and receive another.

Callers are already checking if the pointer is NULL before using it.