[PATCH 3/4] Fix possible memory leak on SIM reading error.

Denis Kenzior denkenz at gmail.com
Fri Jul 31 08:09:44 PDT 2009


Hi,

> This would also stall the SIM op queue if there's a read error for a record
> other than the first.  The other solution would be "goto next;" (keep
> reading further records).

Patch has been applied.

This really shouldn't happen in practice, but if it does, we should also look 
to see if the read callback functions don't cleanup or don't behave properly 
as a result of an error mid-stream.  It looks like at least sim_msisdn_read_cb 
might be affected.  Could you check?

Regards,
-Denis


More information about the ofono mailing list