desktop-webmail crashed by clicking a mailto link

Bug #565042 reported by marcosegato
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
desktop-webmail
Fix Released
High
Alexander Sack
desktop-webmail (Ubuntu)
Fix Released
High
Alexander Sack

Bug Description

Binary package hint: desktop-webmail

By choosing this link, and the application crashes:
mailto:?body=Nuova offerta CityDeal.it  Torino:Voglia di Drink, Djset e Divertimento? JUDAFIRE a soli 5 Euro invece di 10 Euro. Risparmi il 50% http://www.citydeal.it/in/.L3EaGh/,4641&subject=Nuova offerta CityDeal.it  in Torino:Voglia di Drink, Djset e Divertimento? JUDAFIRE a soli 5 Euro invece di 10 Euro. Risparmi il 50%

ProblemType: Crash
DistroRelease: Ubuntu 10.04
Package: desktop-webmail 002-0ubuntu1
ProcVersionSignature: Ubuntu 2.6.32-21.32-generic 2.6.32.11+drm33.2
Uname: Linux 2.6.32-21-generic i686
NonfreeKernelModules: nvidia
Architecture: i386
Date: Fri Apr 16 23:31:12 2010
ExecutablePath: /usr/bin/desktop-webmail
InstallationMedia: Ubuntu 10.04 "Lucid Lynx" - Alpha i386 (20100114)
ProcCmdline: desktop-webmail mailto:?body=Nuova%20offerta%20CityDeal.it%20%C2%A0Torino:Voglia%20di%20Drink,%20Djset%20e%20Divertimento?%20JUDAFIRE%20a%20soli%205%20Euro%20invece%20di%2010%20Euro.%20Risparmi%20il%2050%%C2%A0http://www.citydeal.it/in/.L3EaGh/,4641&subject=Nuova%20offerta%20CityDeal.it%20%C2%A0in%20Torino:Voglia%20di%20Drink,%20Djset%20e%20Divertimento?%20JUDAFIRE%20a%20soli%205%20Euro%20invece%20di%2010%20Euro.%20Risparmi%20il%2050%
ProcEnviron:
 LANG=it_IT.utf8
 SHELL=/bin/bash
SegvAnalysis:
 Segfault happened at: 0x804b226: movb $0x0,(%eax)
 PC (0x0804b226) ok
 source "$0x0" ok
 destination "(%eax)" (0x00000000) not located in a known VMA region (needed writable region)!
SegvReason: writing NULL VMA
Signal: 11
SourcePackage: desktop-webmail
StacktraceTop:
 ?? ()
 __libc_start_main () from /lib/tls/i686/cmov/libc.so.6
 ?? ()
Title: desktop-webmail crashed with SIGSEGV in __libc_start_main()
UserGroups: adm admin cdrom dialout lpadmin plugdev sambashare
XsessionErrors:
 (polkit-gnome-authentication-agent-1:1366): GLib-CRITICAL **: g_once_init_leave: assertion `initialization_value != 0' failed
 (nm-connection-editor:2118): Gtk-WARNING **: GtkSpinButton: setting an adjustment with non-zero page size is deprecated
 (nm-connection-editor:2118): Gtk-WARNING **: GtkSpinButton: setting an adjustment with non-zero page size is deprecated
 (nm-connection-editor:2118): Gtk-WARNING **: GtkSpinButton: setting an adjustment with non-zero page size is deprecated
 (nm-connection-editor:2118): Gtk-WARNING **: GtkSpinButton: setting an adjustment with non-zero page size is deprecated

Related branches

Revision history for this message
marcosegato (marco-segato) wrote :
visibility: private → public
description: updated
Revision history for this message
Apport retracing service (apport) wrote :

StacktraceTop:
 ?? ()
 __libc_start_main (
 ?? ()

Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt
tags: added: apport-failed-retrace
tags: removed: need-i386-retrace
Revision history for this message
Alexander Sack (asac) wrote :

crash happens because of unexpected ? in the url.

Changed in desktop-webmail (Ubuntu):
assignee: nobody → Alexander Sack (asac)
importance: Undecided → High
status: New → In Progress
Revision history for this message
Alexander Sack (asac) wrote :

bzr commit -m "'?' is not a delimiter in the parameter part of mailto: urls; this also
caused a crash whe opening urls of form like: mailto:<email address hidden>?parameter=This?Crashes" --fixes lp:565042
Committing to: /home/asac/Development/ubuntu/mobile/desktop-webmail/
modified src/desktop-webmail.c
Committed revision 25.

Changed in desktop-webmail (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Alexander Sack (asac) wrote :

also committed safety belt:

bzr commit -m "Make mailto: url parser more robust; don't expect the parameter part of an url to
> perfectly split in key/value pairs"
Committing to: /home/asac/Development/ubuntu/mobile/desktop-webmail/
modified src/desktop-webmail.c
Committed revision 26.

Revision history for this message
Alexander Sack (asac) wrote :

with new release 003 i dont see this crash anymore when running:

desktop-webmail 'mailto:?body=Nuova%20offerta%20CityDeal.it%20%C2%A0Torino:Voglia%20di%20Drink,%20Djset%20e%20Divertimento?%20JUDAFIRE%20a%20soli%205%20Euro%20invece%20di%2010%20Euro.%20Risparmi%20il%2050%%C2%A0http://www.citydeal.it/in/.L3EaGh/,4641&subject=Nuova%20offerta%20CityDeal.it%20%C2%A0in%20Torino:Voglia%20di%20Drink,%20Djset%20e%20Divertimento?%20JUDAFIRE%20a%20soli%205%20Euro%20invece%20di%2010%20Euro.%20Risparmi%20il%2050%'

can you confirm?

(note that gmail doesn't like this url due to %% usage ... dont think its an issue of the app)

Changed in desktop-webmail:
status: New → Fix Committed
importance: Undecided → High
assignee: nobody → Alexander Sack (asac)
milestone: none → 003
status: Fix Committed → Fix Released
Revision history for this message
Alexander Sack (asac) wrote :

added to fixed release bugset for 003 milestone (released recently).

Revision history for this message
Alexander Sack (asac) wrote :

003-0ubuntu1 upload includes this upstream fix. forgot to doc in the upload it seems.

Changed in desktop-webmail (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Alexander Sack (asac) wrote :

fix was uploaded to precise pangolin main archive. if you need it for oneiric go for https://launchpad.net/~asac/+archive/ppa

Revision history for this message
Alexander Sack (asac) wrote :

same packages for older releases are also available in my experimental sandbox ppa: https://launchpad.net/~asac/+archive/sandbox

NOTE: I don't know if they work though. feedback welcome! especially for maverick and natty.

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.