Skip to content

Commit 5522218

Browse files
committed
C++: Expand ImplicitThisFieldAccess
1 parent 1297acf commit 5522218

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

cpp/ql/lib/semmle/code/cpp/exprs/Access.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -314,7 +314,7 @@ private predicate exprHasReferenceConversion(Expr e) { referenceConversion(e.get
314314
class ImplicitThisFieldAccess extends FieldAccess {
315315
override string getAPrimaryQlClass() { result = "ImplicitThisFieldAccess" }
316316

317-
ImplicitThisFieldAccess() { not exists(this.getQualifier()) }
317+
ImplicitThisFieldAccess() { this.getQualifier().isCompilerGenerated() or not exists(this.getQualifier()) }
318318
}
319319

320320
/**

cpp/ql/test/library-tests/access/FieldAccess/FieldAccess.expected

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
| FieldAccess.cpp:11:12:11:13 | p1 | ptr |
2-
| FieldAccess.cpp:12:12:12:13 | p2 | ptr |
3-
| FieldAccess.cpp:25:12:25:13 | x1 | ptr |
1+
| FieldAccess.cpp:11:12:11:13 | p1 | ptr, this |
2+
| FieldAccess.cpp:12:12:12:13 | p2 | ptr, this |
3+
| FieldAccess.cpp:25:12:25:13 | x1 | ptr, this |
44
| FieldAccess.cpp:29:18:29:19 | x2 | ptr |
55
| FieldAccess.cpp:34:3:34:3 | d | this |
66
| FieldAccess.cpp:45:13:45:14 | x1 | ptr |
@@ -19,10 +19,10 @@
1919
| FieldAccess.cpp:91:7:91:7 | x | val |
2020
| FieldAccess.cpp:91:13:91:13 | y | ref |
2121
| FieldAccess.cpp:92:8:92:8 | x | ptr |
22-
| FieldAccess.cpp:92:12:92:12 | y | ptr |
22+
| FieldAccess.cpp:92:12:92:12 | y | ptr, this |
2323
| FieldAccess.cpp:93:8:93:8 | x | ptr |
2424
| FieldAccess.cpp:93:18:93:18 | y | ptr |
2525
| FieldAccess.cpp:94:11:94:11 | y | ptr |
2626
| FieldAccess.cpp:94:20:94:20 | y | val |
27-
| FieldAccess.cpp:113:5:113:5 | x | ptr |
27+
| FieldAccess.cpp:113:5:113:5 | x | ptr, this |
2828
| FieldAccess.cpp:116:3:116:3 | v | this |

0 commit comments

Comments
 (0)