facebookarchive/prepack

`__evaluatePureFunction` doesn't properly validate arguments

Open

#2 385 ouverte le 3 août 2018

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)JavaScript (14 268 stars) (520 forks)batch import
bughelp wantedlevel 1 (super easy)

Description

Prepacking the following crashes Prepack.

__evaluatePureFunction("boo");

In the implementation, it shouldn't check via an invariant that the argument is a proper value, but instead it should throw a user-level exception or emit a CompilerDiagnostics error.

Guide contributeur

Stack technique
javascript
Domaine
tooling
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaScriptPrepack internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
Search for ` evaluatePureFunction` in the codebase to locate the invariant check. Replace it with a proper error using CompilerDiagnostics or a user level exception, following existing error handling patterns in the repository.