diff options
| author | Aleksander Morgado <aleksander@lanedo.com> | 2012-09-12 10:41:27 +0200 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@lanedo.com> | 2012-09-14 07:05:25 +0200 |
| commit | 815decb03474754a56e7572a2823ce438a81023b (patch) | |
| tree | 620c5cb1f02530d95e5178b4e16007bf33c0473b /src/mm-sms-list.h | |
| parent | 8cbf3b7826ccf794431d37e449724caaf0f4206b (diff) | |
broadband-modem: properly avoid duplicate CMTI indications
We will look for the part being notified in the CMTI indication directly in the
list of processed parts; and if we have it there already we won't re-process it.
Still, we may get several CMTI indications for the same part one after the
other, so it may still happen that the second CMTI comes *before* we have parsed
and created the part in the SMS list. For that case, the SMS list will reject
taking the part if there is already a part with the same storage+index already
processed.
This fix removes the `known_sms_parts' hash table, which was being handled
incorrectly.
This should fix https://bugzilla.gnome.org/show_bug.cgi?id=675175
Diffstat (limited to 'src/mm-sms-list.h')
| -rw-r--r-- | src/mm-sms-list.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/mm-sms-list.h b/src/mm-sms-list.h index f7710227..16f39e51 100644 --- a/src/mm-sms-list.h +++ b/src/mm-sms-list.h @@ -61,6 +61,10 @@ MMSmsList *mm_sms_list_new (MMBaseModem *modem); GStrv mm_sms_list_get_paths (MMSmsList *self); guint mm_sms_list_get_count (MMSmsList *self); +gboolean mm_sms_list_has_part (MMSmsList *self, + MMSmsStorage storage, + guint index); + gboolean mm_sms_list_take_part (MMSmsList *self, MMSmsPart *part, MMSmsState state, |