Skip to content

Commit 3e102b1

Browse files
committed
Remove refereence to Controller
1 parent a40522a commit 3e102b1

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

javascript/frameworks/ui5/lib/advanced_security/javascript/frameworks/ui5/dataflow/UI5DataFlow.qll

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -66,14 +66,7 @@ class LocalModelContentBoundBidirectionallyToHtmlISinkControl extends DomBasedXs
6666
*/
6767
class LocalModelStringPropertySource extends DomBasedXss::Source {
6868
LocalModelStringPropertySource() {
69-
exists(UI5BindingPath bindingPath |
70-
this =
71-
bindingPath
72-
.getControlDeclaration()
73-
.getDefinition()
74-
.getMetadata()
75-
.getProperty(bindingPath.getPropertyName())
76-
)
69+
this = any(PropertyMetadata propMeta | propMeta.isUnrestrictedStringType())
7770
}
7871
}
7972

javascript/frameworks/ui5/test/queries/UI5Xss/xss-book-example/UI5Xss.expected

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,4 @@ edges
1313
| webapp/controls/Book.js:133:8:133:26 | oControl.getTitle() | webapp/controls/Book.js:132:7:134:15 | "<div>T ... </div>" |
1414
#select
1515
| webapp/controls/Book.js:132:7:134:15 | "<div>T ... </div>" | webapp/controller/App.Controller.js:23:25:23:47 | oSearch ... Value() | webapp/controls/Book.js:132:7:134:15 | "<div>T ... </div>" | XSS vulnerability due to $@. | webapp/controller/App.Controller.js:23:25:23:47 | oSearch ... Value() | user-provided value |
16+
| webapp/controls/Book.js:132:7:134:15 | "<div>T ... </div>" | webapp/controls/Book.js:17:13:17:30 | { type: "string" } | webapp/controls/Book.js:132:7:134:15 | "<div>T ... </div>" | XSS vulnerability due to $@. | webapp/controls/Book.js:17:13:17:30 | { type: "string" } | user-provided value |

0 commit comments

Comments
 (0)