GNOME Bugzilla – Bug 337766
deskbar-applet failed to parse epiphany's history
Last modified: 2017-06-14 11:50:20 UTC
Please describe the problem: deskbar-applet (or python) failed to parse epiphany's ephy-history.xml when given hex 0x0B (Vertical Tab). Steps to reproduce: Not sure, but try this: 1. Open http://norvig.com/Gettysburg/sld004.htm using Epiphany 2. Add deskbar-applet to panel Actual results: Traceback (most recent call last):
+ Trace 67543
self.watcher.connect('changed', lambda watcher, f: self.watch_callback())
self._history = EpiphanyHistoryParser(self, favicon_cache).get_indexer()
self._index_history();
parser.parse(EPHY_HISTORY_FILE)
xmlreader.IncrementalParser.parse(self, source)
self.feed(buffer)
self._err_handler.fatalError(exc)
raise exception
well-formed (invalid token) Parts of epiphany's history is searchable from deskbar-applet, but not http://norvig.com/Gettysburg/sld004.htm. (I entered "success factors" and got nothing). Expected results: All of epiphany's history is searchable from deskbar-applet, and no ugly dialog box pops up. Does this happen every time? yes Other information:
Created attachment 63003 [details] snip of ephy-history.xml, line 20520..20530
I can reproduce here (nd it's really annoying ;)) I will see with ephy devs what's happening, since epiphany seems happy with the xml file..
Epiphany is also bailing out, to reproduce: xmllint --noout ephy-history.xml Assigning to libxml2 product
Created attachment 63495 [details] Simplified code for saving This is essentially the history saving code. Writes silently with no errors, fails on read.
Comment on attachment 63495 [details] Simplified code for saving Of course the real code has error handling done correctly unlike here... :-]
You're writing a character which is not acceptable as an XML Character this can't be escaped either. Daniel
*** Bug 339681 has been marked as a duplicate of this bug. ***
So whose responsibility is it to handle escaping in this case? I'm expecting the "%s" take care of it (it works for <>&) but if it's caller's job, how can we know what needs escaping and what not?
*** Bug 364303 has been marked as a duplicate of this bug. ***
*** Bug 367867 has been marked as a duplicate of this bug. ***
*** Bug 373821 has been marked as a duplicate of this bug. ***
*** Bug 392777 has been marked as a duplicate of this bug. ***
Imo, even if that was an epiphany bug, deskbar shouldn't crash/display an error box because it failed to parse epiphany xml file. Dunno if the fact that this bug is still open means that something will be done on the deskbar side or not ;)
Closing as duplicate of bug 739574 which some constructive dicussion. *** This bug has been marked as a duplicate of bug 739574 ***