f-spot can't email files with ampersands in the filenames

Bug #90008 reported by Michael R. Head
4
Affects Status Importance Assigned to Milestone
F-Spot
Fix Released
Medium
f-spot (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: f-spot

Steps to reproduce:

1) import file a&b.jpg
2) select image imported
3) File/Send Mail...
4) Click "Create Mail"
5) See Evolution Error: Cannot attach file /tmp/tmp94d32c.tmp/a: No such file or directory

If I browse to /tmp/tmp94d32c.tmp I find a&b.jpg there. I can also attach this file to the email (if I'm quick enough and f-spot doesn't delete it after the error).

I'm not sure if the problem is in how f-spot is calling evo, or if it's evo's command line handling, so I'm starting by submitting the bug here in f-spot.

Revision history for this message
Michael R. Head (burner) wrote :

BTW: I haven't had a chance to test this in feisty yet. I can only confirm that this happens in edgy.

Revision history for this message
Michael R. Head (burner) wrote :

Yeah, it looks like this line is the problem:
                                        url.Append ("&attach=" + tmp_path);
in SendEmail.cs

I looked up the source from the latest in http://svn.gnome.org/viewcvs/f-spot/trunk/src/SendEmail.cs?view=annotate and the code is substantially changed in the latest version. I can't tell if something in http://svn.gnome.org/viewcvs/f-spot/trunk/src/Filters/FilterRequest.cs?view=annotate is converting ampersands properly or not.

Revision history for this message
Michael R. Head (burner) wrote :
Revision history for this message
Jonh Wendell (wendell) wrote :

Thanks for your bug report. This bug has been reported to the developers of the software. You can track it and make comments here: http://bugzilla.gnome.org/show_bug.cgi?id=424609

Changed in f-spot:
importance: Undecided → Low
status: Unconfirmed → Confirmed
Changed in f-spot:
status: Unknown → Fix Released
Revision history for this message
Jonh Wendell (wendell) wrote :

Fixed on upstream.

Changed in f-spot:
status: Confirmed → Fix Committed
Revision history for this message
Andrew Mitchell (ajmitch) wrote :

f-spot 0.4.0-0ubunt1 uploaded to gutsy.

Changed in f-spot:
status: Fix Committed → Fix Released
Changed in f-spot:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

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