Skip to content

Commit 86a73fa

Browse files
committed
Swift: Accept fixed spurious test results.
1 parent 0fd4f61 commit 86a73fa

File tree

2 files changed

+8
-16
lines changed

2 files changed

+8
-16
lines changed

swift/ql/test/library-tests/dataflow/taint/libraries/TaintInline.expected

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,5 @@
11
failures
22
testFailures
3-
| int.swift:18:21:19:1 | // $ SPURIOUS: tainted=13\n | Fixed spurious result:tainted=13 |
4-
| int.swift:33:21:34:1 | // $ SPURIOUS: tainted=28\n | Fixed spurious result:tainted=28 |
5-
| int.swift:48:23:49:1 | // $ SPURIOUS: tainted=47\n | Fixed spurious result:tainted=47 |
6-
| int.swift:84:20:85:1 | // $ SPURIOUS: tainted=83\n | Fixed spurious result:tainted=83 |
7-
| int.swift:89:23:90:1 | // $ SPURIOUS: tainted=83\n | Fixed spurious result:tainted=83 |
8-
| int.swift:132:20:133:1 | // $ tainted=131\n | Missing result:tainted=131 |
9-
| int.swift:137:30:138:1 | // $ SPURIOUS: tainted=131\n | Fixed spurious result:tainted=131 |
10-
| int.swift:147:23:148:1 | // $ SPURIOUS: tainted=142\n | Fixed spurious result:tainted=142 |
113
| string.swift:407:23:408:1 | // $ tainted=366\n | Missing result:tainted=366 |
124
| string.swift:441:20:442:1 | // $ tainted=366\n | Missing result:tainted=366 |
135
| string.swift:483:23:484:1 | // $ tainted=450\n | Missing result:tainted=450 |

swift/ql/test/library-tests/dataflow/taint/libraries/int.swift

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ func taintThroughClosurePointer() {
1515
sink(arg: myArray1[0]) // $ tainted=13
1616
let return1 = myArray1.withUnsafeBytes({
1717
ptr1 in
18-
sink(arg: ptr1) // $ SPURIOUS: tainted=13
18+
sink(arg: ptr1)
1919
sink(arg: ptr1[0]) // $ tainted=13
2020
return source()
2121
})
@@ -30,7 +30,7 @@ func taintThroughClosurePointer() {
3030
sink(arg: myArray2[0]) // $ tainted=28
3131
let return2 = myArray2.withUnsafeBufferPointer({
3232
ptr2 in
33-
sink(arg: ptr2) // $ SPURIOUS: tainted=28
33+
sink(arg: ptr2)
3434
sink(arg: ptr2[0]) // $ tainted=28
3535
return source()
3636
})
@@ -45,7 +45,7 @@ func taintThroughMutablePointer() {
4545
let return1 = myArray1.withUnsafeMutableBufferPointer({
4646
buffer in
4747
buffer.update(repeating: source())
48-
sink(arg: buffer) // $ SPURIOUS: tainted=47
48+
sink(arg: buffer)
4949
sink(arg: buffer[0]) // $ tainted=47
5050
sink(arg: buffer.baseAddress!.pointee) // $ MISSING: tainted=47
5151
return source()
@@ -81,12 +81,12 @@ func taintThroughMutablePointer() {
8181
let return3 = myArray3.withContiguousMutableStorageIfAvailable({
8282
ptr in
8383
ptr.update(repeating: source())
84-
sink(arg: ptr) // $ SPURIOUS: tainted=83
84+
sink(arg: ptr)
8585
sink(arg: ptr[0]) // $ tainted=83
8686
return source()
8787
})
8888
sink(arg: return3!) // $ tainted=86
89-
sink(arg: myArray3) // $ SPURIOUS: tainted=83
89+
sink(arg: myArray3)
9090
sink(arg: myArray3[0]) // $ tainted=83
9191

9292
// ---
@@ -129,12 +129,12 @@ func taintThroughMutablePointer() {
129129
let return6 = myMutableBuffer.withContiguousMutableStorageIfAvailable({
130130
ptr in
131131
ptr.update(repeating: source2())
132-
sink(arg: ptr) // $ tainted=131
132+
sink(arg: ptr)
133133
sink(arg: ptr[0]) // $ tainted=131
134134
return source()
135135
})
136136
sink(arg: return6!) // $ tainted=134
137-
sink(arg: myMutableBuffer) // $ SPURIOUS: tainted=131
137+
sink(arg: myMutableBuffer)
138138
sink(arg: myMutableBuffer[0]) // $ tainted=131
139139
}
140140

@@ -144,7 +144,7 @@ func taintCollections(array: inout Array<Int>, contiguousArray: inout Contiguous
144144
sink(arg: array[0]) // $ tainted=142
145145
array.withContiguousStorageIfAvailable({
146146
buffer in
147-
sink(arg: buffer) // $ SPURIOUS: tainted=142
147+
sink(arg: buffer)
148148
sink(arg: buffer[0]) // $ tainted=142
149149
sink(arg: array)
150150
sink(arg: array[0]) // $ MISSING: tainted=142

0 commit comments

Comments
 (0)