Skip to content

Commit 43ee62a

Browse files
committed
C#: Update the NoDisposeCallOnLocalIDisposable and expected output.
1 parent eda345a commit 43ee62a

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

csharp/ql/test/query-tests/API Abuse/NoDisposeCallOnLocalIDisposable/NoDisposeCallOnLocalIDisposable.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,12 @@ public IDisposable Method()
7070
using (var reader = new StreamReader(new FileStream("", FileMode.Open)))
7171
;
7272

73-
// GOOD: XmlDocument.Load disposes incoming XmlReader (according to CIL)
73+
// GOOD: XmlDocument.Load disposes incoming XmlReader (False positive as this is disposed in library code)
7474
var xmlReader = XmlReader.Create(new StringReader("xml"), null);
7575
var xmlDoc = new XmlDocument();
7676
xmlDoc.Load(xmlReader);
7777

78-
// GOOD: Passed to a library. This is only detected in CIL.
78+
// GOOD: Passed to a library (False positive as this is disposed in library code).
7979
DisposalTests.Class1.Dispose(new StreamWriter("output.txt"));
8080

8181
// GOOD: Disposed automatically.

csharp/ql/test/query-tests/API Abuse/NoDisposeCallOnLocalIDisposable/NoDisposeCallOnLocalIDisposable.expected

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,6 @@
22
| NoDisposeCallOnLocalIDisposable.cs:51:18:51:73 | object creation of type FileStream | Disposable 'FileStream' is created but not disposed. |
33
| NoDisposeCallOnLocalIDisposable.cs:52:9:52:64 | object creation of type FileStream | Disposable 'FileStream' is created but not disposed. |
44
| NoDisposeCallOnLocalIDisposable.cs:74:25:74:71 | call to method Create | Disposable 'XmlReader' is created but not disposed. |
5+
| NoDisposeCallOnLocalIDisposable.cs:74:42:74:64 | object creation of type StringReader | Disposable 'StringReader' is created but not disposed. |
6+
| NoDisposeCallOnLocalIDisposable.cs:79:38:79:67 | object creation of type StreamWriter | Disposable 'StreamWriter' is created but not disposed. |
57
| NoDisposeCallOnLocalIDisposableBad.cs:8:22:8:56 | object creation of type FileStream | Disposable 'FileStream' is created but not disposed. |
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
semmle-extractor-options: --cil /r:System.Private.Xml.dll /r:System.IO.Compression.dll
1+
semmle-extractor-options: /r:System.Private.Xml.dll /r:System.IO.Compression.dll

0 commit comments

Comments
 (0)