signalapp / Signal-Android

A private messenger for Android.
https://signal.org
GNU Affero General Public License v3.0
25.66k stars 6.16k forks source link

Sent images are being corrupted when sent to sms group chats. #11469

Closed GildartsClive closed 2 years ago

GildartsClive commented 3 years ago

Bug description

Images get corrupted when sent from Signal's android client through SMS or anyone not using Signal.

Steps to reproduce

In a group or a one on one chat use the + sign to attach an image to my message either type in text or not it doesn't matter. then send it. everyone else can see the image but it comes out corrupted see examples below for what it looks like, but it always corrupts the bottom have of the image. I have a second phone number that I use for work and we use signal for communication but not SMS. So I was able to reproduce this issue with my two accounts and two phone numbers. I also tested with my wife's phone as well.

Actual result: Images sent to nonsignal users are corrupted Expected result: Images should go through with no corruption at all to everyone.

Screenshots

image before it was sent: https://imgur.com/Qplo9t8 image after it is sent to a group chat that is SMS: https://imgur.com/H1D5WCv

Device info

Device: Google Pixel 4a Android version: 11 Signal version: Beta Signal Client ver 5.16.2

GildartsClive commented 3 years ago

update: if I switch to googles messages app it works just fine, ALL images sent out get corrupted when sending from signal to anyone over SMS.

waded commented 2 years ago

I can reproduce this on a Pixel 4a on Signal 5.28.5. I'm seeing the same issue sending images from Google Messages or Facebook Messenger via SMS/MMS, though. For people seeing this style of image corruption in Signal, but not Messages, perhaps Messages is set to RCS - that's a different delivery mechanism than carrier MMS, and I have no trouble with sending the same image files over RCS.

I suspect a carrier compatibility issue of some kind - and with all apps using the same subsystem to send MMS (true?), not surprising they're all affected. I've had this issue with Messages/MMS on and off with Cricket and the Pixel 4a for a year, and when I saw Signal had a similar bug report, thought I'd drop in to reproduce/corroborate. I've also heard of others having this issue with Messages/MMS and AT&T, again Pixel 4a.

greyson-signal commented 2 years ago

Yeah, unfortunately I wouldn't be surprised if this is carrier-specific. MMS messages are unencrypted, and carriers will often perform their own compression on media in-transit, which could be buggy.

stale[bot] commented 2 years ago

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.