SchildiChat / SchildiChat-android

Matrix client / Element Android fork
https://schildi.chat/android/
Apache License 2.0
385 stars 49 forks source link

Libwebp 0day #210

Closed linsui closed 9 months ago

linsui commented 9 months ago

Describe the bug SchildiChat-android uses https://github.com/zjupure/GlideWebpDecoder which uses libwebp 1.2.4. We can patch the lib by replacing https://github.com/zjupure/GlideWebpDecoder/tree/master/webp_decoder/src/main/jni/libwebp with https://github.com/webmproject/libwebp/commits/1.2.4.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Smartphone (please complete the following information):

Additional context

Upstream relevance

Add any other context about the problem here.

licaon-kter commented 9 months ago

ref: https://blog.isosceles.com/the-webp-0day/

SpiritCroc commented 9 months ago

Oh, thanks for pointing out, I'll try to get a release out tomorrow

SpiritCroc commented 9 months ago

We can patch the lib by replacing https://github.com/zjupure/GlideWebpDecoder/tree/master/webp_decoder/src/main/jni/libwebp with https://github.com/webmproject/libwebp/commits/1.2.4.

Is there a straightforward approach to do this without forking GlideWebpDecoder? If not, I can probably just kick that library out, the amount of additional media that the library enables us to view is rather small and shouldn't justify maintaining it if it becomes too annoying

linsui commented 9 months ago

I thought not. You can add it back when the lib is updated.

SpiritCroc commented 9 months ago

Addressed in v1.6.5.sc72 (for test builds: v1.6.5.sc72-test4).

licaon-kter commented 9 months ago

fyi https://gitlab.com/fdroid/fdroiddata/-/merge_requests/13782/diffs

linsui commented 9 months ago

GlideWebpDecoder released a new version with the updated libwebp so if you want you can add it back.