crash in midori_view_save_source

Bug #957674 reported by gue5t gue5t
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Midori Web Browser
Fix Released
Undecided
Unassigned

Bug Description

Midori crashes when trying to view source of the page <http://tumblr.com/twitter/check> (a user must be logged into twitter to see, but the page is essentially blank).

A backtrace is attached; the crux of the bug is that midori_view_save_source (in midori-view.c:5166) performs:

data = webkit_web_data_source_get_data (data_source);

and then assumes data is non-null (e.g. on line 5190), which is not necessarily true.

Revision history for this message
gue5t gue5t (gue5t) wrote :
tags: added: bitesize
Cris Dywan (kalikiana)
Changed in midori:
status: New → Fix Committed
Revision history for this message
gue5t gue5t (gue5t) wrote :

Line 5251 still references data->len and crashes after the patch. That's the only other place data's assumed non-null though.

Changed in midori:
status: Fix Committed → In Progress
Revision history for this message
Cris Dywan (kalikiana) wrote :

Indeed, thanks for pointing it out. Should be fixed this time.

Changed in midori:
status: In Progress → Fix Committed
Cris Dywan (kalikiana)
Changed in midori:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.