100609 - Attachment filename corruption on send (and receive)

Status: VERIFIED WORKSFORME

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Frank Furter]

It appears that attachments exceeding 8.3 are getting corrupted filenames (this
could be an OSX problem?).

See url for screen shot (unless I can figure out how to attach the image).

Filename locally is:

"A Perfect Circle - Diary Of A Madman (Live).mp3"

Upon attaching it to the mail it becomes:

"A Perfect Circle - Di#2467C.mp3"

End recipient receives file with corrupted filename as well.

Comment 1

[Frank Furter]

Sorry, browser wouldn't let me attach screen shot, says:

You must provide a description of your attachment.

in a big red box (yes, I provided a description.)

See the URL above for a screenshot.

Thanks!

Comment 2

[(not reading, please use [e-mail] instead)]

over to ducarroz, our mac os x person.

I think you'll want to take a look at nsMsgUtils.cpp, NS_MsgHashIfNecessary().

we currently have code (#ifdef XP_MAC) that decides to hash if the file name is
longer than the Mac OS 8.x / 9.x limit of 32 chars.

that code is set at compile time.  I believe our mac os x binaries are built on
9.1, where XP_MAC is set, so this may not get fixed.

Comment 3

[Simon Fraser [no longer active]]

Our file code on Mac os X still can't deal with file names > 31 chars. If the 
attachment is still sent correctly, and received, then this should be a WFM.

Comment 4

[Greg K.]

Confirming using Fizzilla/2001091313 (0.9.4). The attachment is not saved with the original 
filename after being received.

Comment 5

[Jean-Francois Ducarroz]

does this problem append with file name < 32 characters?

Comment 6

[Jean-Francois Ducarroz]

I did a test with an attachment with a short file name and it works fine. We
have the same limitation with MacOS 9.x

We have planed to rewrite the way we manage attachment in nsMsgCompFields which
should resolve this issue as well others.

WFM.

Comment 7

[lchiang]

We should release note this.

Comment 8

[Trix Supremo]

verified wfm