mirror of https://github.com/crytic/slither
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
366 lines
11 KiB
366 lines
11 KiB
[
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Contract 'naming' (tests/naming_convention.sol#3-48) is not in CapWords\n",
|
|
"elements": [
|
|
{
|
|
"target": "contract",
|
|
"convention": "CapWords",
|
|
"name": "naming",
|
|
"source_mapping": {
|
|
"start": 28,
|
|
"length": 642,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
3,
|
|
4,
|
|
5,
|
|
6,
|
|
7,
|
|
8,
|
|
9,
|
|
10,
|
|
11,
|
|
12,
|
|
13,
|
|
14,
|
|
15,
|
|
16,
|
|
17,
|
|
18,
|
|
19,
|
|
20,
|
|
21,
|
|
22,
|
|
23,
|
|
24,
|
|
25,
|
|
26,
|
|
27,
|
|
28,
|
|
29,
|
|
30,
|
|
31,
|
|
32,
|
|
33,
|
|
34,
|
|
35,
|
|
36,
|
|
37,
|
|
38,
|
|
39,
|
|
40,
|
|
41,
|
|
42,
|
|
43,
|
|
44,
|
|
45,
|
|
46,
|
|
47,
|
|
48
|
|
],
|
|
"starting_column": 1,
|
|
"ending_column": 2
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Struct 'naming.test' (tests/naming_convention.sol#14-16) is not in CapWords\n",
|
|
"elements": [
|
|
{
|
|
"target": "structure",
|
|
"convention": "CapWords",
|
|
"name": "test",
|
|
"source_mapping": {
|
|
"start": 229,
|
|
"length": 35,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
14,
|
|
15,
|
|
16
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 6
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Event 'naming.event_' (tests/naming_convention.sol#23) is not in CapWords\n",
|
|
"elements": [
|
|
{
|
|
"target": "event",
|
|
"convention": "CapWords",
|
|
"name": "event_",
|
|
"source_mapping": {
|
|
"start": 335,
|
|
"length": 19,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
23
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 24
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Function 'naming.GetOne' (tests/naming_convention.sol#30-33) is not in mixedCase\n",
|
|
"elements": [
|
|
{
|
|
"target": "function",
|
|
"convention": "mixedCase",
|
|
"name": "GetOne",
|
|
"source_mapping": {
|
|
"start": 440,
|
|
"length": 75,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
30,
|
|
31,
|
|
32,
|
|
33
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 6
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Parameter 'Number2' of naming.setInt (tests/naming_convention.sol#35) is not in mixedCase\n",
|
|
"elements": [
|
|
{
|
|
"target": "parameter",
|
|
"convention": "mixedCase",
|
|
"name": "Number2",
|
|
"source_mapping": {
|
|
"start": 551,
|
|
"length": 12,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
35
|
|
],
|
|
"starting_column": 35,
|
|
"ending_column": 47
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Constant 'naming.MY_other_CONSTANT' (tests/naming_convention.sol#9) is not in UPPER_CASE_WITH_UNDERSCORES\n",
|
|
"elements": [
|
|
{
|
|
"target": "variable_constant",
|
|
"convention": "UPPER_CASE_WITH_UNDERSCORES",
|
|
"name": "MY_other_CONSTANT",
|
|
"source_mapping": {
|
|
"start": 143,
|
|
"length": 35,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
9
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 40
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Variable 'naming.Var_One' (tests/naming_convention.sol#11) is not in mixedCase\n",
|
|
"elements": [
|
|
{
|
|
"target": "variable",
|
|
"convention": "mixedCase",
|
|
"name": "Var_One",
|
|
"source_mapping": {
|
|
"start": 185,
|
|
"length": 16,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
11
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 21
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Enum 'naming.numbers' (tests/naming_convention.sol#6) is not in CapWords\n",
|
|
"elements": [
|
|
{
|
|
"target": "enum",
|
|
"convention": "CapWords",
|
|
"name": "numbers",
|
|
"source_mapping": {
|
|
"start": 79,
|
|
"length": 23,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
6
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 28
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Modifier 'naming.CantDo' (tests/naming_convention.sol#41-43) is not in mixedCase\n",
|
|
"elements": [
|
|
{
|
|
"target": "modifier",
|
|
"convention": "mixedCase",
|
|
"name": "CantDo",
|
|
"source_mapping": {
|
|
"start": 591,
|
|
"length": 36,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
41,
|
|
42,
|
|
43
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 6
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Parameter '_used' of T.test (tests/naming_convention.sol#59) is not in mixedCase\n",
|
|
"elements": [
|
|
{
|
|
"target": "parameter",
|
|
"convention": "mixedCase",
|
|
"name": "_used",
|
|
"source_mapping": {
|
|
"start": 794,
|
|
"length": 10,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
59
|
|
],
|
|
"starting_column": 33,
|
|
"ending_column": 43
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Variable 'T._myPublicVar' (tests/naming_convention.sol#56) is not in mixedCase\n",
|
|
"elements": [
|
|
{
|
|
"target": "variable",
|
|
"convention": "mixedCase",
|
|
"name": "_myPublicVar",
|
|
"source_mapping": {
|
|
"start": 741,
|
|
"length": 17,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
56
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 22
|
|
}
|
|
}
|
|
]
|
|
},
|
|
{
|
|
"check": "naming-convention",
|
|
"impact": "Informational",
|
|
"confidence": "High",
|
|
"description": "Variable 'T.l' (tests/naming_convention.sol#67) used l, O, I, which should not be used\n",
|
|
"elements": [
|
|
{
|
|
"target": "variable",
|
|
"convention": "l_O_I_should_not_be_used",
|
|
"name": "l",
|
|
"source_mapping": {
|
|
"start": 900,
|
|
"length": 10,
|
|
"filename_used": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_relative": "tests/naming_convention.sol",
|
|
"filename_absolute": "/home/travis/build/crytic/slither/tests/naming_convention.sol",
|
|
"filename_short": "tests/naming_convention.sol",
|
|
"lines": [
|
|
67
|
|
],
|
|
"starting_column": 5,
|
|
"ending_column": 15
|
|
}
|
|
}
|
|
]
|
|
}
|
|
] |