jquery-form/form

Fortify SCA: Code Injection .

Open

#554 ouverte le 3 août 2018

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)JavaScript (2 199 forks)batch import
help wantedneed more info

Métriques du dépôt

Stars
 (5 209 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Please review Instructions for Reporting a Bug.

Description:

I have no idea about whether it has been fixed in later versions. but the code scan is not passed.

Expected Behavior:

Actual behavior:

Source: jquery.form.js:812 Read xhr.responseXML() 810 var ct = xhr.getResponseHeader('content-type') || '', 811 xml = type === 'xml' || !type && ct.indexOf('xml') >= 0, 812 data = xml ? xhr.responseXML : xhr.responseText; 813 814 if (xml && data.documentElement.nodeName === 'parsererror') { Sink: jquery.form.js:781 setTimeout() 779 780 // clean up 781 setTimeout(function() { 782 if (!s.iframeTarget) { 783 $io.remove();

Versions:

jqform:3.51

Guide contributeur

Direction de recherche
Examinez la vulnérabilité d'injection de code dans jquery.form.js autour des lignes 812 et 781. Le problème est que xhr.responseXML est utilisé sans assainissement et que setTimeout est appelé avec du contenu dynamique. Recherchez comment analyser en toute sécurité les réponses XML et éviter l'injection de code, en utilisant éventuellement textContent ou DOMParser avec des paramètres appropriés.
Stack technique
javascript
Domaine
frontendsecurity
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 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptjQuery
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60