Bug 63458 - log iter segfault dereferencing a date empty events next_event
Summary: log iter segfault dereferencing a date empty events next_event
Status: RESOLVED INVALID
Alias: None
Product: Telepathy
Classification: Unclassified
Component: logger (show other bugs)
Version: git master
Hardware: Other All
: medium major
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-12 09:12 UTC by Alban Browaeys
Modified: 2014-01-10 23:53 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
log-iter: skip to next date if the selected one has no events. (1.33 KB, patch)
2013-04-12 09:12 UTC, Alban Browaeys
Details | Splinter Review
empathy-chat: do not disable scrollback for rooms (irc and alike). (1.05 KB, patch)
2014-01-10 23:52 UTC, Alban Browaeys
Details | Splinter Review

Description Alban Browaeys 2013-04-12 09:12:49 UTC
Created attachment 77852 [details] [review]
log-iter: skip to next date if the selected one has no events.

log-iter: skip to next date if the selected one has no events.
Comment 1 Debarshi Ray 2013-10-24 15:02:33 UTC
Comment on attachment 77852 [details] [review]
log-iter: skip to next date if the selected one has no events.

Review of attachment 77852 [details] [review]:
-----------------------------------------------------------------

This can only happen if _tpl_log_store_get_dates returns a date for which there are no events. Right? Does that really happen unless the logs have been messed with?

As far as I understand, _tpl_log_store_get_dates should only return dates for which there is a valid event. So _tpl_log_store_get_events_for_date should never return NULL for a date that was returned by _tpl_log_store_get_dates.
Comment 2 Alban Browaeys 2014-01-10 23:52:31 UTC
Created attachment 91846 [details] [review]
empathy-chat: do not disable scrollback for rooms (irc and alike).

I have scrollback enabled for chatrooms in empathy as exhibited in attached patch (not yet submitted to empathy bug tracker). Though I reverted the next_event telepathy-logger previous hack  and all looks fine as of now so I will mark it to fixed invalid.
The steps to reproduce were to scroll back: scroll the chat window up , the previous events were prepended but somewhat it was segfaulting.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.