Midori segfaults on adding IsraelList to Adblock
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Midori Web Browser |
Fix Committed
|
High
|
Unassigned |
Bug Description
Command line midori
Midori 0.5.11 ((null)) Midori
GTK+ 2.24.29 (2.24.25) Glib 2.46.2 (2.42.1)
WebKitGTK+ 2.4.9 (2.4.8) libSoup 2.52.2
cairo 1.14.4 (1.14.0) libnotify No
gcr 3.18.0 granite No
Platform X11; Linux x86_64
Identification Mozilla/5.0 (X11; Linux) AppleWebKit/538.15 (KHTML, like Gecko) Chrome/46.0.2490.86 Safari/538.15 Midori/0.5
Video Formats H264 [x] Ogg Theora [x] WebM [x]
Netscape Plugins:
Gnome Shell Integration This plugin provides integration with Gnome Shell for live extension enabling and disabling. It can be used only by extensions.
iTunes Application Detector This plug-in detects the presence of iTunes when opening iTunes Store URLs in a web page with Firefox.
Replicate(also attached):
#!/bin/sh
mv ~/.config/midori ~/.config/
mkdir ~/.config/midori; mkdir ~/.config/
touch ~/.config/
echo "[extensions]" > ~/.config/
midori
#Segmentation fault
#eof
Related branches
- Paweł Forysiuk: Approve
-
Diff: 63 lines (+20/-17)2 files modifiedextensions/adblock/extension.vala (+1/-1)
extensions/adblock/updater.vala (+19/-16)
Changed in midori: | |
status: | New → Confirmed |
Changed in midori: | |
milestone: | none → 0.5.12 |
importance: | Undecided → High |
tags: | added: adblock |
Changed in midori: | |
status: | Confirmed → In Progress |
Changed in midori: | |
status: | In Progress → Fix Committed |
It looks like this is a result of the modification date in that list being in the format "06/01/16 10:56". extensions/ adblock/ updater. vala has logic to check for '-' or '.' separators to find a numeric date string but not '/', and assumes that any string without either of those two characters will have three space-separated fields. This one doesn't, so it crashes.
I'm not sure where one finds detailed documentation on the Adblock list format. We can fix the crash but I don't know which date formats we should actually accept, and whether this block list's date is malformed or correct.