Skip to content

Commit f2f4f4c

Browse files
committed
Shared: Add 'security_code' sensitive data heuristic.
1 parent 5ed78d1 commit f2f4f4c

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

rust/ql/test/library-tests/sensitivedata/test.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,7 @@ fn test_private_info(
315315
sink(info.financials.credit_card_no.as_str()); // $ sensitive=private
316316
sink(info.financials.card_no.as_str()); // $ sensitive=private
317317
sink(info.financials.cardNumber.as_str()); // $ sensitive=private
318-
sink(info.financials.card_security_code.as_str()); // $ MISSING: sensitive=private
318+
sink(info.financials.card_security_code.as_str()); // $ sensitive=private
319319
sink(info.financials.credit_rating); // $ sensitive=private
320320
sink(info.financials.user_ccn.as_str()); // $ sensitive=private
321321
sink(info.financials.cvv.as_str()); // $ sensitive=private

shared/concepts/codeql/concepts/internal/SensitiveDataHeuristics.qll

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,7 @@ module HeuristicNames {
106106
// Financial data - such as credit card numbers, salary, bank accounts, and debts
107107
"(credit|debit|bank|visa).?(card|num|no|acc(ou)?nt)|(card|acc(ou)?nt).?(no|num|credit)|routing.?num|"
108108
+ "salary|billing|beneficiary|credit.?(rating|score)|([_-]|\\b)(ccn|cvv|iban)([_-]|\\b)|" +
109+
"security.?code|" +
109110
// Communications - e-mail addresses, private e-mail messages, SMS text messages, chat logs, etc.
110111
// "e(mail|_mail)|" + // this seems too noisy
111112
// Health - medical conditions, insurance status, prescription records

0 commit comments

Comments
 (0)