Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Nullable GIF input causes NullPointerException in GifStreamObject.load

Aperta Adatta ai principianti
  • Qualcosa non va?
#2,216 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 3 giorni

@blihio ci sta già lavorando.

Dal 16/10/2026.

  • #2217 di @blihio — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
62/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
android, java, kotlin
Ambito
mobile

Direzione di ricerca

Inizia in encoder/src/main/java/com/pedro/encoder/input/gl/render/filters/object/GifFilterRender.kt attorno a setGif, poi leggi GifStreamObject.java, dove load chiama inputStreamGif.available() senza un controllo null. Riproduci il problema con il test su una sola riga della issue nei test unitari locali del modulo encoder. È completato quando uno stream null viene rifiutato con IOException, come suggerito dalla issue, e il test di regressione passa. La scelta dell'API tra IOException e una firma non nullable potrebbe richiedere una decisione dei maintainer.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Describe the bug

GifFilterRender.setGif(inputStream: InputStream?) accepts a nullable stream, but forwards it to GifStreamObject.load, which calls inputStreamGif.available() without checking for null. Consequently, setGif(null) throws NullPointerException.

This is an inconsistency between the public Kotlin parameter type and the Java loading implementation. The reproducer intentionally supplies null; it does not demonstrate a failure when loading a valid GIF.

To reproduce

Run this local unit test in the encoder module against commit 82a9cb4ee4babdc792d2ccee8b893f2b36207d40:

import com.pedro.encoder.input.gl.render.filters.`object`.GifFilterRender
import org.junit.Test

class GifNullInputTest {
  @Test
  fun reproduce() {
    GifFilterRender().setGif(null)
  }
}

The call fails with NullPointerException inside GifStreamObject.load at inputStreamGif.available().

Expected behavior

Handle null consistently with the public API contract. One option is to reject null with IOException, which is already used for GIF loading failures, while preserving the nullable Kotlin signature. Alternatively, the API could explicitly require a non-null stream.

Environment
  • Source version: 2.8.1, commit 82a9cb4ee4babdc792d2ccee8b893f2b36207d40.
  • Class: GifFilterRender / GifStreamObject, module encoder.
  • Reproduced in the project's Android local unit test task using JDK 17, Gradle 9.7.1 and compile SDK 35.
  • Device / Android OS / media server: not involved in this local unit test reproduction.
Additional context

Relevant source: nullable Kotlin entry point, Java dereference.

I have a small fix and a regression test ready. The test expects IOException: it fails with NPE before the fix and passes after it. With only this fix and its test applied, all 26 encoder and 98 common local unit tests pass.

Lingua principale
Kotlin
Stelle
3k
Fork
946
Merge medio
5g 1h
PR unite (30g)
6

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di pedroSG94/RootEncoder

Tutte le issue di pedroSG94/RootEncoder

Issue simili

Altre issue su Kotlin

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.