fix: validate Netlify update response before use#62
Open
LeGeRyChEeSe wants to merge 1 commit into
Open
Conversation
checkNetlifyUpdatesWithFallback() deserializes UpdateInfo directly via Retrofit/Gson. Gson allocates Kotlin data classes without calling their constructor, so a malformed/incomplete server response can leave non-null fields (version, downloadUrl) holding null at runtime despite their declared type. Consuming them (.lowercase(), .startsWith()) would throw a deferred NPE, silently swallowed by the generic catch-all with no error shown to the user. Validate with the null-safe isNullOrBlank() (not .isBlank(), which would itself NPE on a secretly-null value) before use, same pattern already applied to the checksum field just below.
Test Results0 tests 0 ✅ 0s ⏱️ Results for commit 63f1af2. |
PR Validation Summary ✅Status: All checks passed!
Reports and artifacts (including Lint HTML) are available in the Artifacts Tab of the Action Run Summary |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problème
Le contrôle de mise à jour a deux chemins : GitHub Releases (primaire) et Netlify (secours, uniquement sur 404 GitHub). Le chemin Netlify désérialise la réponse serveur directement en
UpdateInfo(champs Kotlin non-nullables) via Retrofit/Gson.Cause
Gson alloue les data classes Kotlin sans passer par leur constructeur (
Unsafe.allocateInstanceen l'absence de constructeur sans arguments accessible). Un JSON serveur incomplet/malformé peut donc laisser des champs déclarés non-nullables (version,downloadUrl, …) ànullau runtime, malgré le type Kotlin. Les utiliser (.lowercase(),.startsWith(...)) déclenche une NPE différée. Cette NPE est rattrapée par lecatch (Exception)générique existant (pas de crash), mais échoue silencieusement sans message pour l'utilisateur — contrairement aux autres branches d'erreur qui affichent un message explicite.Le pattern défensif existait déjà pour
checksum(updateInfo.checksum.isNullOrEmpty(),MainViewModel.ktligne ~1961) mais pas pourversion/downloadUrl.Correctif
Ajout d'une validation explicite juste après réception de la réponse Netlify, avec
isNullOrBlank()(et non.isBlank(), qui NPE aussi sur une valeur secrètement nulle) — même style que la validationchecksumdéjà en place. En cas de réponse invalide, message d'erreur explicite affiché à l'utilisateur au lieu d'un échec silencieux.Vérification
scripts\gradlew.bat :app:assembleDevDebug→ succès.Pixel_7,com.vrhub.debug) : lancement à froid sans crash, logcat propre.checkNetlifyUpdatesWithFallbackest une méthode privée profondément intégrée àMainViewModel(AndroidViewModel avec de nombreuses dépendances), sans infrastructure de test existante pour ce flux ; l'ajouter isolément n'aurait pas été raisonnable dans le cadre de ce correctif ciblé.Réf :
docs/sonnet5-specs/01-bug-audit-backlog.md#b4--désérialisation-réseau-gsonretrofit