[MMS] AMR audio attachement is recognized and shown as video file

RESOLVED INVALID

Status

P1
minor
RESOLVED INVALID
5 years ago
5 years ago

People

(Reporter: leo.bugzilla.gaia, Assigned: kaze)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [TD-56083] )

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
1. Title: AMR audio attachement is recognized and shown as video file
2. Precondition: Store some files in  SDCard
3. Tester's Action: 1) Start composing mms message and attach an amr music file
4. Detailed Symptom (ENG.) : AMR music file is shown as video in mms and the device crashes when we try to open the file
5. Expected: AMR file should be recognized and shown as audio in the MMS 
6. Reproducibility: Y
1) Frequency Rate : 100%
7. Gaia Master/v1-train: Reproduced on v1-train
8. Gaia Revision: f2d6ed54a236e6e3b94f0abad9f0dacb8a1cc7b3
9. Personal email id: sasikala.paruchuri8@gmail.com
(Reporter)

Updated

5 years ago
blocking-b2g: --- → leo+
Whiteboard: [TD-56083]
Target Milestone: --- → 1.1 QE4 (15jul)
(Reporter)

Comment 1

5 years ago
Created attachment 770704 [details]
Attached amr file
I don't know why this STR should be leo+.

Is the real STR about receiving a AMR file from another device ?

The problem here is that this filetype is recognised as a video type.
Kaze will investigate why this crashes.
Assignee: nobody → kaze
Whiteboard: [TD-56083] → [TD-56083][u=commsapps-user c=contacts p=1]
Whiteboard: [TD-56083][u=commsapps-user c=contacts p=1] → [TD-56083][u=commsapps-user c=messaging p=1]
(Reporter)

Updated

5 years ago
Priority: -- → P1
HI KAZE: you can check if the crashing is related to https://bugzilla.mozilla.org/show_bug.cgi?id=877482

if so, the bug here should be "recognizing audio files as video"
Flags: needinfo?(kaze)
steve, can you check why the audio file is recognized as video?
Flags: needinfo?(schung)
Whiteboard: [TD-56083][u=commsapps-user c=messaging p=1] → [TD-56083][u=commsapps-user c=messaging p=1], TaipeiMMS
CC dominic/SC if more information is required.
After some investigation and discussion, the type of the blob is video/3gp. minetype return from gecko only considered the extension at first. Music/video player have their own metadata parser to handle this situation, so this file could be scanned as audio file.

Instead of have a metadata parser for message app, maybe we can check the blob returning activity(not sure it's accessible or not), or check the video tag width(setting an audio to a video tag will generate a video tag with width=0), but the solutions here will need some gaia changes...

But the most important thing here is we should not crash while opening the file. Even I change to extension to audio type, it still crash. I will attach the log later. If we could open the file correctly with both video/music player, maybe there is not so necessary to classify the file as audio type?
Flags: needinfo?(schung)
Created attachment 773064 [details]
Error while opening the file as audio file
Flags: needinfo?(schien)
The PBlobStreamChild assert should be fixed at bug 887093, which is uplifted on 6/30. Maybe you can try with the latest gecko.
Flags: needinfo?(schien)
The crashing has bee fix with bug 887093.

The file recognition part here I don't believe is a blocker here given:
1. The file used to test has the wrong mime type as video/3gp per comment 6
2. There is no user impact as the file can still be attached, played - although  playing is via video player the impact is minimal.
3. Pure audio .3gp files are not very commonly used by real users, especially with the wrong mime type.

Leo-'ing here, please renom and give support information if you think required.

Clearing ni on Kaze as crash is fixed.
blocking-b2g: leo+ → ---
Flags: needinfo?(kaze)
Target Milestone: 1.1 QE4 (15jul) → ---

Updated

5 years ago
Whiteboard: [TD-56083][u=commsapps-user c=messaging p=1], TaipeiMMS → [TD-56083][u=commsapps-user c=messaging p=1],
Whiteboard: [TD-56083][u=commsapps-user c=messaging p=1], → [TD-56083]
should we resolve this bug as invalid ?

Steve, wh
should we resolve this bug as invalid ?

Steve, what do you think ?
Flags: needinfo?(schung)
Just discuss with Wayne and we agree that it could be closed as invalid(and he will handle it), thanks!
Flags: needinfo?(schung)
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.