/*@notfunction@*/
# define idemSpecialFlag TRUE, TRUE, FALSE, FALSE, ARG_NONE
/*@notfunction@*/
-# define valueFlag FALSE, FALSE, FALSE, FALSE, ARG_VALUE
+# define valueFlag FALSE, FALSE, FALSE, FALSE, ARG_NUMBER
+
+/*@notfunction@*/
+# define charFlag FALSE, FALSE, FALSE, FALSE, ARG_CHAR
+
/*@notfunction@*/
-# define modeValueFlag FALSE, FALSE, FALSE, TRUE, ARG_VALUE
+# define modeValueFlag FALSE, FALSE, FALSE, TRUE, ARG_NUMBER
/*@notfunction@*/
-# define specialValueFlag TRUE, FALSE, FALSE, FALSE, ARG_VALUE
+# define specialValueFlag TRUE, FALSE, FALSE, FALSE, ARG_NUMBER
/*@notfunction@*/
# define debugFlag FALSE, TRUE, FALSE, FALSE, ARG_NONE
/*@notfunction@*/
-# define debugValueFlag FALSE, TRUE, FALSE, FALSE, ARG_VALUE
+# define debugValueFlag FALSE, TRUE, FALSE, FALSE, ARG_NUMBER
/*@notfunction@*/
# define specialDebugFlag TRUE, TRUE, FALSE, FALSE, ARG_NONE
/*@notfunction@*/
/*@notfunction@*/
# define idemGlobalFlag FALSE, TRUE, TRUE, FALSE, ARG_NONE
/*@notfunction@*/
-# define globalValueFlag FALSE, FALSE, TRUE, FALSE, ARG_VALUE
+# define globalValueFlag FALSE, FALSE, TRUE, FALSE, ARG_NUMBER
+
+/* Note: type argument missing */
/*@notfunction@*/
-# define regStringFlag FALSE, FALSE, FALSE, FALSE, ARG_STRING
+# define regStringFlag FALSE, FALSE, FALSE, FALSE
/*@notfunction@*/
-# define idemStringFlag FALSE, TRUE, FALSE, FALSE, ARG_STRING
+# define idemStringFlag FALSE, TRUE, FALSE, FALSE
/*@notfunction@*/
-# define globalStringFlag FALSE, FALSE, TRUE, FALSE, ARG_STRING
+# define globalStringFlag FALSE, FALSE, TRUE, FALSE
+
/*@notfunction@*/
# define extraArgFlag FALSE, FALSE, FALSE, FALSE, ARG_SPECIAL
/*@notfunction@*/
# define globalExtraArgFlag FALSE, FALSE, TRUE, FALSE, ARG_SPECIAL
+/*@notfunction@*/
+# define globalFileFlag FALSE, TRUE, TRUE, FALSE, ARG_FILE
/*@=namechecks@*/
/*
static flaglist flags =
{
+ /*
+ ** Null Dereferences (Manual Section 2)
+ */
+
{
- FK_BOOL, FK_HELP, plainFlag,
- "likelybool",
- FLG_LIKELYBOOL,
- "type name is probably a boolean type but does not match default "
- "boolean type name, \"bool\", and alternate name is not set",
- "Use the -booltype, -boolfalse and -booltrue flags to change the "
- "name of the default boolean type.",
+ FK_NULL, FK_MEMORY, specialFlag,
+ "null",
+ FLG_NULL,
+ "misuses of null pointer",
+ "A possibly null pointer is misused (sets nullderef, nullpass, "
+ "nullref, nullassign, and nullstate).",
0, 0
},
{
- FK_ABSTRACT, FK_IMPLICIT, plainFlag,
- "impabstract",
- FLG_IMPABSTRACT,
- "assume user type definitions are abstract (unless /*@concrete@*/ is used)",
- NULL,
+ FK_NULL, FK_MEMORY, modeFlag,
+ "nullderef",
+ FLG_NULLDEREF,
+ "possible dereferencce of null pointer",
+ "A possibly null pointer is dereferenced. Value is "
+ "either the result of a function which may return null "
+ "(in which case, code should check it is not null), or a "
+ "global, parameter or structure field declared with the "
+ "null qualifier.",
0, 0
},
{
- FK_ABSTRACT, FK_NAMES, specialFlag,
- "accessall",
- FLG_ACCESSALL,
- "set accessmodule, accessfile and accessczech",
- "Sets accessmodule, accessfile and accessczech",
+ FK_NULL, FK_MEMORY, modeFlag,
+ "nullpass",
+ FLG_NULLPASS,
+ "possibly null pointer passed as formal with no null annotation",
+ "A possibly null pointer is passed as a parameter corresponding to "
+ "a formal parameter with no /*@null@*/ annotation. If NULL "
+ "may be used for this parameter, add a /*@null@*/ annotation "
+ "to the function parameter declaration.",
0, 0
},
{
- FK_ABSTRACT, FK_NAMES, plainFlag,
- "accessmodule",
- FLG_ACCESSMODULE,
- "allow access to abstract types in definition module",
- "The representation of an abstract type defined in <M>.<x> is "
- "accessible anywhere in a file named <M>.<y>.",
+ FK_NULL, FK_MEMORY, modeFlag,
+ "nullret",
+ FLG_NULLRET,
+ "possibly null pointer returned as result with no null annotation",
+ "Function returns a possibly null pointer, but is not declared "
+ "using /*@null@*/ annotation of result. If function may "
+ "return NULL, add /*@null@*/ annotation to the return "
+ "value declaration.",
0, 0
},
{
- FK_ABSTRACT, FK_NAMES, plainFlag,
- "accessfile",
- FLG_ACCESSFILE,
- "allow access to abstract types by file name convention",
- "The representation of an abstract type named <t> is "
- "accessible anywhere in a file named <t>.<x>.",
+ FK_NULL, FK_MEMORY, modeFlag,
+ "nullstate",
+ FLG_NULLSTATE,
+ "possibly null pointer reachable from a reference with no null annotation",
+ "A possibly null pointer is reachable from a parameter or global "
+ "variable that is not declared using a /*@null@*/ annotation.",
0, 0
},
{
- FK_ABSTRACT, FK_NAMES, plainFlag,
- "accessczech",
- FLG_ACCESSCZECH,
- "allow access to abstract types by czech naming convention",
- "The representation of an abstract type named <t> is accessible "
- "in the definition of a function or constant named <t>_<name>",
+ FK_NULL, FK_MEMORY, modeFlag,
+ "nullassign",
+ FLG_NULLASSIGN,
+ "inconsistent assignment or initialization involving null pointer",
+ "A reference with no null annotation is assigned or initialized "
+ "to NULL. Use /*@null@*/ to declare the reference as "
+ "a possibly null pointer.",
0, 0
},
{
- FK_ABSTRACT, FK_NAMES, plainFlag,
- "accessslovak",
- FLG_ACCESSSLOVAK,
- "allow access to abstract types by slovak naming convention",
- "The representation of an abstract type named <t> is accessible "
- "in the definition of a function or constant named <t><Name>",
+ FK_NULL, FK_MEMORY, plainFlag,
+ "nullinit",
+ FLG_NULLINIT,
+ "inconsistent initialization involving null pointer",
+ "A reference with no null annotation is initialized "
+ "to NULL. Use /*@null@*/ to declare the reference as "
+ "a possibly null pointer.",
0, 0
},
+
+ /*
+ ** Undefined Values (Section 3)
+ */
+
{
- FK_ABSTRACT, FK_NAMES, plainFlag,
- "accessczechoslovak",
- FLG_ACCESSCZECHOSLOVAK,
- "allow access to abstract types by czechoslovak naming convention",
- "The representation of an abstract type named <t> is accessible "
- "in the definition of a function or constant named <t>_<name> or <t><Name>",
+ FK_DEF, FK_NONE, modeFlag,
+ "usedef",
+ FLG_USEDEF,
+ "use before definition",
+ "An rvalue is used that may not be initialized to a value on some execution path.",
0, 0
},
{
- FK_ABSTRACT, FK_NONE, plainFlag,
- "abstract",
- FLG_ABSTRACT,
- "data abstraction barriers",
- "An abstraction barrier is broken. If necessary, use /*@access <type>@*/ to allow access to an abstract type.",
+ FK_MEMORY, FK_DEF, modeFlag,
+ "mustdefine",
+ FLG_MUSTDEFINE,
+ "out storage not defined before return or scope exit",
+ "An out parameter or global is not defined before control is transferred.",
0, 0
},
{
- FK_ABSTRACT, FK_NONE, modeFlag,
- "mutrep",
- FLG_MUTREP,
- "representation of mutable type has sharing semantics",
- "LCL semantics requires that a mutable type exhibits sharing semantics. In order for objects to be shared a indirection is necessary in the representation. A mutable type may be represented by a pointer or an abstract mutable type. Handles into static data are fine, too, but will generate this error message unless it is suppressed.",
+ FK_MEMORY, FK_DEF, modeFlag,
+ "uniondef",
+ FLG_UNIONDEF,
+ "at least one field of a union must be defined",
+ "No field of a union is defined. Generally, one field of a union is "
+ "expected to be defined.",
0, 0
},
{
- FK_ALIAS, FK_GLOBALS, specialFlag,
- "globalias",
- FLG_GLOBALIAS,
- "function returns with global aliasing external state (sets "
- "checkstrictglobalias, checkedglobalias, checkmodglobalias and "
- "uncheckedglobalias)",
- "A global variable aliases externally-visible state when the function returns.",
+ FK_MEMORY, FK_DEF, modeFlag,
+ "compdef",
+ FLG_COMPDEF,
+ "parameter, return value or global completely defined",
+ "Storage derivable from a parameter, return value or global is "
+ "not defined. Use /*@out@*/ to denote passed or returned "
+ "storage which need not be defined.",
0, 0
},
{
- FK_ALIAS, FK_GLOBALS, modeFlag,
- "checkstrictglobalias",
- FLG_CHECKSTRICTGLOBALIAS,
- "function returns with a checkstrict global aliasing external state",
- "A global variable aliases externally-visible state when the function returns.",
+ FK_DEF, FK_NONE, plainFlag,
+ "fullinitblock",
+ FLG_FULLINITBLOCK,
+ "initializer sets all fields",
+ "Initializer does not set every field in the structure.",
0, 0
},
{
- FK_ALIAS, FK_GLOBALS, modeFlag,
- "checkedglobalias",
- FLG_CHECKEDGLOBALIAS,
- "function returns with a checked global aliasing external state",
- "A global variable aliases externally-visible state when the function returns.",
+ FK_DEF, FK_NONE, plainFlag,
+ "initallelements",
+ FLG_INITALLELEMENTS,
+ "initializer defines all array elements",
+ "Initializer does not define all elements of a declared array.",
0, 0
},
{
- FK_ALIAS, FK_GLOBALS, modeFlag,
- "checkmodglobalias",
- FLG_CHECKMODGLOBALIAS,
- "function returns with a checkmod global aliasing external state",
- "A global variable aliases externally-visible state when the function returns.",
+ FK_DEF, FK_NONE, plainFlag,
+ "initsize",
+ FLG_INITSIZE,
+ "initializer defines extra array elements",
+ "Initializer block contains more elements than the size of a declared array.",
0, 0
},
+
{
- FK_ALIAS, FK_GLOBALS, modeFlag,
- "uncheckedglobalias",
- FLG_UNCHECKEDGLOBALIAS,
- "function returns with an unchecked global aliasing external state",
- "A global variable aliases externally-visible state when the function returns.",
- 0, 0
+ FK_DEF, FK_IMPLICIT, modeFlag,
+ "impouts",
+ FLG_IMPOUTS,
+ "pointer parameters to unspecified functions may be implicit out parameters",
+ NULL, 0, 0
},
+
+ /*
+ ** Types (Section 4)
+ */
+
{
- FK_ALIAS, FK_MEMORY, modeFlag,
- "aliasunique",
- FLG_ALIASUNIQUE,
- "unique parameter is aliased",
- "A unique or only parameter is aliased by some other parameter or visible global.",
+ FK_DECL, FK_TYPE, modeFlag,
+ "incondefs",
+ FLG_INCONDEFS,
+ "function, variable or constant redefined with inconsistent type",
+ "A function, variable or constant is redefined with a different type.",
0, 0
},
{
- FK_ALIAS, FK_MEMORY, modeFlag,
- "mayaliasunique",
- FLG_MAYALIASUNIQUE,
- "unique parameter may be aliased",
- "A unique or only parameter may be aliased by some other parameter or visible global.",
+ FK_DECL, FK_TYPE, modeFlag,
+ "matchfields",
+ FLG_MATCHFIELDS,
+ "struct or enum type redefined with inconsistent fields or members",
+ "A struct, union or enum type is redefined with inconsistent fields or members.",
0, 0
},
{
- FK_ALIAS, FK_MEMORY, modeFlag,
- "mustnotalias",
- FLG_MUSTNOTALIAS,
- "temp storage aliased at return point or scope exit",
- "An alias has been added to a temp-qualifier parameter or global that is visible externally when the function returns. If the aliasing is needed, use the /*@shared@*/ annotation to indicate that new aliases to the parameter may be created.",
+ FK_TYPE, FK_NONE, modeFlag,
+ "fcnderef",
+ FLG_FCNDEREF,
+ "dereferencce of a function type",
+ "A function type is dereferenced. The ANSI standard allows this "
+ "because of implicit conversion of function designators, however the "
+ "dereference is unnecessary.",
0, 0
},
{
- FK_ALIAS, FK_NONE, modeFlag,
- "retalias",
- FLG_RETALIAS,
- "function returns alias to parameter or global",
- "The returned value shares storage with a parameter or global. If a parameter is to be returned, use the returned qualifier. If the result is not modified, use the observer qualifier on the result type. Otherwise, exposed can be used, but limited checking is done.", 0, 0
+ FK_OPS, FK_NONE, modeFlag,
+ "realcompare",
+ FLG_REALCOMPARE,
+ "dangerous comparison between reals (dangerous because of inexact "
+ "floating point representations)",
+ "Two real (float, double, or long double) values are compared "
+ "directly using a C primitive. "
+ "This may produce unexpected results since floating point "
+ "representations are inexact. Instead, compare the difference to "
+ "FLT_EPSILON or DBL_EPSILON.",
+ 0, 0,
},
{
- FK_ANSI, FK_PROTOS, modeFlag,
- "noparams",
- FLG_NOPARAMS,
- "function declaration has no parameter list",
- "A function declaration does not have a parameter list.",
- 0, 0
+ FK_OPS, FK_NONE, modeFlag,
+ "unsignedcompare",
+ FLG_UNSIGNEDCOMPARE,
+ "comparison using <, <=, >= between an unsigned integral and zero constant",
+ "An unsigned value is used in a comparison with zero in a way that is either a bug or confusing.",
+ 0, 0,
},
{
- FK_ANSI, FK_PROTOS, modeFlag,
- "oldstyle",
- FLG_OLDSTYLE,
- "old style function definition",
- "Function definition is in old style syntax. Standard prototype "
- "syntax is preferred.",
- 0, 0
+ FK_OPS, FK_POINTER, modeFlag,
+ "ptrarith",
+ FLG_POINTERARITH,
+ "arithmetic involving pointer and integer",
+ "Pointer arithmetic using pointer and integer.", 0, 0
},
{
- FK_ANSI, FK_SYNTAX, plainFlag,
- "gnuextensions",
- FLG_GNUEXTENSIONS,
- "support some gnu (gcc) language extensions",
- "ANSI C does not allow some language features supported by gcc and other compilers. "
- "Use +gnuextensions to allow some of these extensions.", 0, 0
+ FK_OPS, FK_POINTER, modeFlag,
+ "nullptrarith",
+ FLG_NULLPOINTERARITH,
+ "arithmetic involving possibly null pointer and integer",
+ "Pointer arithmetic using a possibly null pointer and integer.", 0, 0
},
{
- FK_ANSI, FK_NONE, plainFlag,
- "usevarargs",
- FLG_USEVARARGS,
- "non-standard <varargs.h> included",
- "Header <varargs.h> is not part of ANSI Standard. "
- "Should use <stdarg.h> instead.",
+ FK_OPS, FK_POINTER, modeFlag,
+ "ptrcompare",
+ FLG_PTRNUMCOMPARE,
+ "comparison between pointer and number",
+ "A pointer is compared to a number.", 0, 0
+ },
+ {
+ FK_OPS, FK_TYPE, modeFlag,
+ "strictops",
+ FLG_STRICTOPS,
+ "primitive operation does not type check strictly",
+ "A primitive operation does not type check strictly.", 0, 0
+ },
+ {
+ FK_OPS, FK_TYPE, modeFlag,
+ "bitwisesigned",
+ FLG_BITWISEOPS,
+ "a bitwise logical operator does not have unsigned operands",
+ "An operand to a bitwise operator is not an unsigned values. This "
+ "may have unexpected results depending on the signed "
+ "representations.", 0, 0
+ },
+ {
+ FK_OPS, FK_TYPE, modeFlag,
+ "shiftnegative",
+ FLG_SHIFTNEGATIVE,
+ "a shift right operand may be negative",
+ "The right operand to a shift operator may be negative (behavior undefined).",
0, 0
},
{
- FK_ANSI, FK_LIBS, plainFlag,
- "warnposixheaders",
- FLG_WARNPOSIX,
- "a POSIX header is included, but the POSIX library is not used",
- "Header name matches a POSIX header, but the POSIX library is not selected.",
+ FK_OPS, FK_TYPE, modeFlag,
+ "shiftimplementation",
+ FLG_SHIFTIMPLEMENTATION,
+ "a shift left operand may be negative",
+ "The left operand to a shift operator may be negative (behavior is implementation-defined).",
0, 0
},
{
- FK_BEHAVIOR, FK_ANSI, modeFlag,
- "exitarg",
- FLG_EXITARG,
- "argument to exit has implementation defined behavior",
- "The argument to exit should be 0, EXIT_SUCCESS or EXIT_FAILURE",
+ FK_OPS, FK_TYPE, modeFlag,
+ "sizeoftype",
+ FLG_SIZEOFTYPE,
+ "sizeof operator has a type argument",
+ "Operand of sizeof operator is a type. (Safer to use expression, "
+ "int *x = sizeof (*x); instead of sizeof (int).)",
0, 0
},
{
- FK_BEHAVIOR, FK_ANSI, modeFlag,
- "evalorder",
- FLG_EVALORDER,
- "code has unspecified or implementation-dependent behavior "
- "because of order of evaluation",
- "Code has unspecified behavior. "
- "Order of evaluation of function parameters or subexpressions "
- "is not defined, so if a value is used and modified in different "
- "places not separated by a sequence point constraining "
- "evaluation order, then the result of the expression is "
- "unspecified.",
+ FK_OPS, FK_TYPE, plainFlag,
+ "sizeofformalarray",
+ FLG_SIZEOFFORMALARRAY,
+ "sizeof operator has an array formal parameter argument",
+ "Operand of a sizeof operator is a function parameter declared as "
+ "an array. The value of sizeof will be the size of a pointer to the "
+ "element type, not the number of elements in the array.",
0, 0
},
{
- FK_BEHAVIOR, FK_ANSI, modeFlag,
- "evalorderuncon",
- FLG_EVALORDERUNCON,
- "code involving call to unspecified function has undefined or implementation-dependent behavior",
- "Code involving a call to function with no modifies or globals clause "
- "may have undefined or implementation-dependent behavior (LCLint assumes the "
- "unconstrained call may modify any reachable state or use any global). Add a "
- "specification for the function.",
+ FK_DECL, FK_TYPE, plainFlag,
+ "fixedformalarray",
+ FLG_FIXEDFORMALARRAY,
+ "formal parameter of type array is declared with size",
+ "A formal parameter is declared as an array with size. The size of the array "
+ "is ignored in this context, since the array formal parameter is treated "
+ "as a pointer.",
0, 0
},
{
- FK_BOOL, FK_NONE, regStringFlag,
- "boolfalse",
- FLG_BOOLFALSE,
- "set name of boolean false (default FALSE)",
- NULL, 0, 0
+ FK_DECL, FK_TYPE, plainFlag,
+ "incompletetype",
+ FLG_INCOMPLETETYPE,
+ "formal parameter has an incomplete type",
+ "A formal parameter is declared with an incomplete type.",
+ 0, 0
+ },
+ {
+ FK_DECL, FK_TYPE, plainFlag,
+ "formalarray",
+ FLG_FORMALARRAY,
+ "formal parameter is an array",
+ "A formal parameter is declared as an array. This can be confusing, since "
+ "a formal array parameter is treated as a pointer.",
+ 0, 0
},
+
+
+ /*
+ ** Booleans (4.2)
+ */
+
{
- FK_BOOL, FK_NONE, regStringFlag,
+ FK_BOOL, FK_NONE, regStringFlag, ARG_STRING,
"booltype",
FLG_BOOLTYPE,
"set name of boolean type (default bool)",
NULL, 0, 0
},
{
- FK_BOOL, FK_NONE, regStringFlag,
+ FK_BOOL, FK_NONE, regStringFlag, ARG_STRING,
+ "boolfalse",
+ FLG_BOOLFALSE,
+ "set name of boolean false (default false)",
+ NULL, 0, 0
+ },
+ {
+ FK_BOOL, FK_NONE, regStringFlag, ARG_STRING,
"booltrue",
FLG_BOOLTRUE,
- "set name of boolean true (default TRUE)",
+ "set name of boolean true (default true)",
NULL, 0, 0
},
{
- FK_COMMENTS, FK_ABSTRACT, plainFlag,
- "noaccess",
- FLG_NOACCESS,
- "ignore access comments",
- NULL, 0, 0
+ FK_BOOL, FK_HELP, plainFlag,
+ "likelybool",
+ FLG_LIKELYBOOL,
+ "type name is probably a boolean type but does not match default "
+ "boolean type name, \"bool\", and alternate name is not set",
+ "Use the -booltype, -boolfalse and -booltrue flags to change the "
+ "name of the default boolean type.",
+ 0, 0
},
+
{
- FK_COMMENTS, FK_SUPPRESS, plainFlag,
- "nocomments",
- FLG_NOCOMMENTS,
- "ignore all stylized comments",
- NULL, 0, 0
+ FK_BOOL, FK_OPS, modeFlag,
+ "boolcompare",
+ FLG_BOOLCOMPARE,
+ "comparison between bools (dangerous because of multiple true values)",
+ "Two bool values are compared directly using a C primitive. This "
+ "may produce unexpected results since all non-zero values are "
+ "considered true, so different true values may not be equal. "
+ "The file bool.h (included in splint/lib) provides bool_equal "
+ "for safe bool comparisons.", 0, 0
},
{
- FK_COMMENTS, FK_SYNTAX, plainFlag,
- "unrecogcomments",
- FLG_UNRECOGCOMMENTS,
- "stylized comment is unrecognized",
- "Word after a stylized comment marker does not correspond to a "
- "stylized comment.",
+ FK_BOOL, FK_OPS, modeFlag,
+ "boolops",
+ FLG_BOOLOPS,
+ "primitive operation (!, && or ||) does not has a boolean argument",
+ "The operand of a boolean operator is not a boolean. Use +ptrnegate "
+ "to allow ! to be used on pointers.",
0, 0
},
{
- FK_COMMENTS, FK_SYNTAX, plainFlag,
- "unrecogflagcomments",
- FLG_UNRECOGFLAGCOMMENTS,
- "stylized flag comment uses an unrecognized flag",
- "Semantic comment attempts to set a flag that is not recognized.",
- 0, 0
+ FK_BOOL, FK_POINTER, modeFlag,
+ "ptrnegate",
+ FLG_PTRNEGATE,
+ "allow ! to be used on pointer operand",
+ "The operand of ! operator is a pointer.", 0, 0
},
{
- FK_COMMENTS, FK_SYNTAX, plainFlag,
- "continuecomment",
- FLG_CONTINUECOMMENT,
- "line continuation marker (\\) in comment before */ on same line",
- "A line continuation marker (\\) appears inside a comment on the same "
- "line as the comment close. Preprocessors should handle this "
- "correctly, but it causes problems for some preprocessors.",
+ FK_BOOL, FK_PRED,plainFlag,
+ "predassign",
+ FLG_PREDASSIGN,
+ "condition test (if, while or for) is an assignment",
+ "The condition test is an assignment expression. Probably, you mean "
+ "to use == instead of =. If an assignment is intended, add an "
+ "extra parentheses nesting (e.g., if ((a = b)) ...) to suppress "
+ "this message.",
0, 0
},
{
- FK_COMMENTS, FK_SYNTAX, plainFlag,
- "nestcomment",
- FLG_NESTCOMMENT,
- "comment begins inside comment",
- "A comment open sequence (/*) appears within a comment. This usually "
- "means an earlier comment was not closed.",
- 0, 0
+ FK_BOOL, FK_PRED, specialFlag,
+ "predbool",
+ FLG_PREDBOOL,
+ "type of condition test (if, while or for) not bool (sets predboolint, "
+ "predboolptr and predboolothers)",
+ "Test expression type is not boolean.", 0, 0
},
{
- FK_COMMENTS, FK_SUPPRESS, modeFlag,
- "tmpcomments",
- FLG_TMPCOMMENTS,
- "interpret t comments (ignore errors in lines marked with /*@t<n>@*/",
- NULL, 0, 0
+ FK_PRED, FK_BOOL, modeFlag,
+ "predboolint",
+ FLG_PREDBOOLINT,
+ "type of condition test (if, while or for) is an integral type",
+ "Test expression type is not boolean or int.", 0, 0
},
{
- FK_COMMENTS, FK_SUPPRESS, plainFlag,
- "lintcomments",
- FLG_LINTCOMMENTS,
- "interpret traditional lint comments (/*FALLTHROUGH*/, /*NOTREACHED*/)",
- NULL, 0, 0
+ FK_BOOL, FK_PRED, modeFlag,
+ "predboolptr",
+ FLG_PREDBOOLPTR,
+ "type of condition test (if, while or for) is a pointer",
+ "Test expression type is not boolean.", 0, 0
},
{
- FK_COMMENTS, FK_SUPPRESS, modeFlag,
- "warnlintcomments",
- FLG_WARNLINTCOMMENTS,
- "print a warning and suggest an alternative when a traditional lint "
- "comment is used",
- "A traditional lint comment is used. Some traditional lint comments "
- "are interpreted by LCLint to enable easier checking of legacy "
- "code. It is preferable to replace these comments with the "
- "suggested LCLint alternative.",
- 0, 0
+ FK_BOOL, FK_PRED, modeFlag,
+ "predboolothers",
+ FLG_PREDBOOLOTHERS,
+ "type of condition test (if, while or for) not bool, int or pointer",
+ "Test expression type is not boolean.", 0, 0
},
+
+ /*
+ ** 4.3 Abstract types
+ */
+
{
- FK_COMPLETE, FK_NONE, modeFlag,
- "declundef",
- FLG_DECLUNDEF,
- "function or variable declared but never defined",
- "A function or variable is declared, but not defined in any source code file.",
+ FK_ABSTRACT, FK_NONE, plainFlag,
+ "abstract",
+ FLG_ABSTRACT,
+ "data abstraction barriers",
+ "An abstraction barrier is broken. If necessary, use /*@access <type>@*/ to allow access to an abstract type.",
0, 0
},
{
- FK_COMPLETE, FK_SPEC, modeFlag,
- "specundef",
- FLG_SPECUNDEF,
- "function or variable specified but never defined",
- "A function or variable is declared in an .lcl file, but not defined in any source code file.",
+ FK_ABSTRACT, FK_NONE, modeFlag,
+ "abstractcompare",
+ FLG_ABSTRACTCOMPARE,
+ "object equality comparison on abstract type operands",
+ "An object comparison (== or !=) is used on operands of abstract type.",
0, 0
},
+
{
- FK_COMPLETE, FK_SPEC, plainFlag,
- "specundecl",
- FLG_SPECUNDECL,
- "function or variable specified but never declared in a source file",
- "A function or variable is declared in an .lcl file, but not declared "
- "in any source code file.",
+ FK_ABSTRACT, FK_NONE, plainFlag,
+ "numabstract",
+ FLG_NUMABSTRACT,
+ "data abstraction barriers",
+ "An abstraction barrier involving a numabstract type is broken. If necessary, use /*@access <type>@*/ to allow access to a numabstract type.",
0, 0
},
{
- FK_CONTROL, FK_MEMORY, modeFlag,
- "loopexec",
- FLG_LOOPEXEC,
- "assume all loops execute at least once",
- NULL, 0, 0
+ FK_ABSTRACT, FK_NONE, modeFlag,
+ "numabstractcast",
+ FLG_NUMABSTRACTCAST,
+ "numeric literal cast to numabstract type",
+ "A numeric literal is cast to a numabstract type.",
+ 0, 0
},
{
- FK_CONTROL, FK_NONE, plainFlag,
- "control",
- FLG_CONTROL,
- NULL, NULL,
+ FK_ABSTRACT, FK_NONE, modeFlag,
+ "numabstractlit",
+ FLG_NUMABSTRACTLIT,
+ "numeric literal can used as numabstract type",
+ "To allow a numeric literal to be used as a numabstract type, use +numabstractlit.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "infloops",
- FLG_INFLOOPS,
- "likely infinite loop is detected",
- "This appears to be an infinite loop. Nothing in the body of the "
- "loop or the loop test modifies the value of the loop test. "
- "Perhaps the specification of a function called in the loop "
- "body is missing a modification.",
+ FK_ABSTRACT, FK_TYPEEQ, modeFlag,
+ "numabstractindex",
+ FLG_NUMABSTRACTINDEX,
+ "a numabstract type can be used to index an array",
+ "To allow numabstract types to index arrays, use +numabstractindex.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "infloopsuncon",
- FLG_INFLOOPSUNCON,
- "likely infinite loop is detected (may result from unconstrained function)",
- "This appears to be an infinite loop. Nothing in the body of the "
- "loop or the loop test modifies the value of the loop test. "
- "There may be a modification through a call to an unconstrained "
- "function, or an unconstrained function in the loop test may use "
- "a global variable modified by the loop body.",
+ FK_ABSTRACT, FK_NONE, modeFlag,
+ "numabstractprint",
+ FLG_NUMABSTRACTPRINT,
+ "a numabstract value is printed using %d format code",
+ "A numabstract value is printed usind %d format code in a printf.",
0, 0
},
{
- FK_CONTROL, FK_NONE, specialFlag,
- "deepbreak",
- FLG_DEEPBREAK,
- "break inside nested while or for or switch",
- "A break statement appears inside the body of a nested while, for or "
- "switch statement. Sets looploopbreak, loopswitchbreak, "
- "switchloopbreak, switchswitchbreak, and looploopcontinue.",
+ FK_ABSTRACT, FK_IMPLICIT, plainFlag,
+ "impabstract",
+ FLG_IMPABSTRACT,
+ "assume user type definitions are abstract (unless /*@concrete@*/ is used)",
+ NULL,
0, 0
},
+
+ /* 4.3.1 Access */
+
{
- FK_CONTROL, FK_NONE, modeFlag,
- "looploopbreak",
- FLG_LOOPLOOPBREAK,
- "break inside nested while or for",
- "A break statement appears inside the body of a nested while or for "
- "statement. This is perfectly reasonable code, but check that "
- "the break is intended to break only the inner loop. The "
- "break statement may be preceded by /*@innerbreak@*/ to suppress "
- "the message for this break only.",
+ FK_ABSTRACT, FK_NAMES, plainFlag,
+ "accessmodule",
+ FLG_ACCESSMODULE,
+ "allow access to abstract types in definition module",
+ "The representation of an abstract type defined in <M>.<x> is "
+ "accessible anywhere in a file named <M>.<y>.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "switchloopbreak",
- FLG_SWITCHLOOPBREAK,
- "break in loop inside switch",
- "A break statement appears inside the body of a while or for "
- "statement within a switch. This is perfectly reasonable code, but check that "
- "the break is intended to break only the inner loop. The "
- "break statement may be preceded by /*@loopbreak@*/ to suppress "
- "the message for this break only.",
+ FK_ABSTRACT, FK_NAMES, plainFlag,
+ "accessfile",
+ FLG_ACCESSFILE,
+ "allow access to abstract types by file name convention",
+ "The representation of an abstract type named <t> is "
+ "accessible anywhere in a file named <t>.<x>.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "loopswitchbreak",
- FLG_LOOPSWITCHBREAK,
- "break in switch inside loop",
- "A break statement appears inside a switch statement within a while or "
- "for loop. This is perfectly reasonable code, but check that "
- "the break is intended to break only the inner loop. The "
- "break statement may be preceded by /*@switchbreak@*/ to suppress "
- "the message for this break only.",
+ FK_ABSTRACT, FK_NAMES, plainFlag,
+ "accessczech",
+ FLG_ACCESSCZECH,
+ "allow access to abstract types by czech naming convention",
+ "The representation of an abstract type named <t> is accessible "
+ "in the definition of a function or constant named <t>_<name>",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "switchswitchbreak",
- FLG_SWITCHSWITCHBREAK,
- "break in switch inside switch",
- "A break statement appears inside a switch statement within another "
- "switch statement. This is perfectly reasonable code, but check that "
- "the break is intended to break only the inner switch. The "
- "break statement may be preceded by /*@innerbreak@*/ to suppress "
- "the message for this break only.",
+ FK_ABSTRACT, FK_NAMES, plainFlag,
+ "accessslovak",
+ FLG_ACCESSSLOVAK,
+ "allow access to abstract types by slovak naming convention",
+ "The representation of an abstract type named <t> is accessible "
+ "in the definition of a function or constant named <t><Name>",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "looploopcontinue",
- FLG_LOOPLOOPCONTINUE,
- "continue inside nested loop",
- "A continue statement appears inside a loop within a loop. "
- "This is perfectly reasonable code, but check that "
- "the continue is intended to continue only the inner loop. The "
- "continue statement may be preceded by /*@innercontinue@*/ to suppress "
- "the message for this continue only.",
+ FK_ABSTRACT, FK_NAMES, plainFlag,
+ "accessczechoslovak",
+ FLG_ACCESSCZECHOSLOVAK,
+ "allow access to abstract types by czechoslovak naming convention",
+ "The representation of an abstract type named <t> is accessible "
+ "in the definition of a function or constant named <t>_<name> or <t><Name>",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "unreachable",
- FLG_UNREACHABLE,
- "unreachable code detected",
- "This code will never be reached on any possible execution.",
+ FK_ABSTRACT, FK_NAMES, specialFlag,
+ "accessall",
+ FLG_ACCESSALL,
+ "set accessmodule, accessfile and accessczech",
+ "Sets accessmodule, accessfile and accessczech",
0, 0
},
+
+ /* 4.3.2 Mutability */
{
- FK_CONTROL, FK_NONE, modeFlag,
- "whileempty",
- FLG_WHILEEMPTY,
- "a while statement has no body",
- "While statement has no body.",
+ FK_ABSTRACT, FK_NONE, modeFlag,
+ "mutrep",
+ FLG_MUTREP,
+ "representation of mutable type has sharing semantics",
+ "LCL semantics requires that a mutable type exhibits sharing semantics. "
+ "In order for objects to be shared a indirection is necessary in the representation. "
+ "A mutable type may be represented by a pointer or an abstract mutable type. Handles "
+ "into static data are fine, too, but will generate this error message unless it is suppressed.",
0, 0
},
+
+
+ /*
+ ** Memory Management (5)
+ */
+
+ /* Deallocation Errors */
{
- FK_CONTROL, FK_NONE, modeFlag,
- "whileblock",
- FLG_WHILEBLOCK,
- "the body of a while statement is not a block",
- "While body is a single statement, not a compound block.",
+ FK_MEMORY, FK_LEAK, modeFlag,
+ "mustfreefresh",
+ FLG_MUSTFREEFRESH,
+ "freshly allocated storage not released before return or scope exit",
+ "A memory leak has been detected. Storage allocated locally "
+ "is not released before the last reference to it is lost.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "forempty",
- FLG_FOREMPTY,
- "a for statement has no body",
- "For statement has no body.",
+ FK_MEMORY, FK_LEAK, modeFlag,
+ "mustfreeonly",
+ FLG_MUSTFREEONLY,
+ "only storage not released before return or scope exit",
+ "A memory leak has been detected. Only-qualified storage is not released before the last "
+ "reference to it is lost.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "forblock",
- FLG_FORBLOCK,
- "the body of a for statement is not a block",
- "Loop body is a single statement, not a compound block.",
+ FK_MEMORY, FK_LEAK, specialFlag,
+ "mustfree",
+ FLG_MUSTFREE,
+ "fresh or only storage not released before return or scope exit (sets mustfreefresh and mustfreeonly)",
+ "A memory leak has been detected.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "ifempty",
- FLG_IFEMPTY,
- "an if statement has no body",
- "If statement has no body.",
+ FK_MEMORY, FK_DEAD, modeFlag,
+ "usereleased",
+ FLG_USERELEASED,
+ "storage used after release",
+ "Memory is used after it has been released (either by passing "
+ "as an only param or assigning to an only global).",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "ifblock",
- FLG_IFBLOCK,
- "the body of an if statement is not a block",
- "If body is a single statement, not a compound block.",
+ FK_MEMORY, FK_DEAD, modeFlag,
+ "strictusereleased",
+ FLG_STRICTUSERELEASED,
+ "element used after it may have been released",
+ "Memory (through fetch) is used after it may have been released "
+ "(either by passing as an only param or assigning to an only global).",
0, 0
},
{
- FK_CONTROL, FK_NONE, specialFlag,
- "allempty",
- FLG_ALLEMPTY,
- "an if, while or for statement has no body (sets ifempty, "
- "whileempty and forempty",
- NULL,
+ FK_MEMORY, FK_LEAK, modeFlag,
+ "compdestroy",
+ FLG_COMPDESTROY,
+ "all only references derivable from void pointer out only parameter are released",
+ "A storage leak due to incomplete deallocation of a structure or deep "
+ "pointer is suspected. Unshared storage that is reachable from "
+ "a reference that is being deallocated has not yet been deallocated. "
+ "Splint assumes when an object is passed "
+ "as an out only void pointer that the outer object will be "
+ "deallocated, but the inner objects will not.",
0, 0
},
{
- FK_CONTROL, FK_NONE, specialFlag,
- "allblock",
- FLG_ALLBLOCK,
- "the body of an if, while or for statement is not a block "
- "(sets ifblock, whileblock and forblock)",
- "Body is a single statement, not a compound block.",
+ FK_MEMORY, FK_LEAK, modeFlag,
+ "strictdestroy",
+ FLG_STRICTDESTROY,
+ "report complete destruction errors for array elements that "
+ "may have been released",
+ NULL,
+ 0, 0
+ },
+ {
+ FK_MEMORY, FK_ARRAY, modeFlag,
+ "deparrays",
+ FLG_DEPARRAYS,
+ "array elements are dependent storage",
+ "When an element is fetched from an array, Splint analysis is "
+ "not able to determine if the same element is reused. "
+ "If +deparrays, Splint will mark local storage assigned from "
+ "array fetches as dependent.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "elseifcomplete",
- FLG_ELSEIFCOMPLETE,
- "if ... else if chains must have final else",
- "There is no final else following an else if construct.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "branchstate",
+ FLG_BRANCHSTATE,
+ "storage has inconsistent states of alternate paths through a branch",
+ "The state of a variable is different depending on which branch "
+ "is taken. This means no annotation can sensibly be applied "
+ "to the storage.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "noret",
- FLG_NORETURN,
- "path with no return detected in non-void function",
- "There is a path through a function declared to return a value on "
- "which there is no return statement. This means the execution "
- "may fall through without returning a meaningful result to "
- "the caller.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "strictbranchstate",
+ FLG_STRICTBRANCHSTATE,
+ "storage through array fetch has inconsistent states of alternate "
+ "paths through a branch",
+ "The state of a variable through an array fetch is different depending "
+ "on which branch is taken. This means no annotation can sensibly be applied "
+ "to the storage.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "casebreak",
- FLG_CASEBREAK,
- "non-empty case in a switch without preceding break",
- "Execution falls through from the previous case.", 0, 0
+ FK_MEMORY, FK_NONE, specialFlag,
+ "memchecks",
+ FLG_MEMCHECKS,
+ "sets all dynamic memory checking flags (memimplicit, mustfree, mustdefine, "
+ "mustnotalias, null, memtrans)",
+ NULL, 0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "misscase",
- FLG_MISSCASE,
- "switch on enum type missing case for some value",
- "Not all values in an enumeration are present as cases in the switch.",
+ FK_MEMORY, FK_DEF, modeFlag,
+ "compmempass",
+ FLG_COMPMEMPASS,
+ "actual parameter matches alias kind of formal parameter completely ",
+ "Storage derivable from a parameter does not match the alias kind "
+ "expected for the formal parameter.",
0, 0
},
{
- FK_CONTROL, FK_NONE, modeFlag,
- "firstcase",
- FLG_FIRSTCASE,
- "first statement in switch is not a case",
- "The first statement after a switch is not a case.",
+ FK_MEMORY, FK_DEAD, modeFlag,
+ "stackref",
+ FLG_RETSTACK,
+ "external reference to stack-allocated storage is created",
+ "A stack reference is pointed to by an external reference when the "
+ "function returns. The stack-allocated storage is destroyed "
+ "after the call, leaving a dangling reference.",
0, 0
},
+
{
- FK_DEBUG, FK_NONE, specialDebugFlag,
- "grammar",
- FLG_GRAMMAR,
- NULL, NULL,
- 0, 0
+ FK_MEMORY, FK_NONE, specialFlag,
+ "memtrans",
+ FLG_MEMTRANS,
+ "memory transfer errors (sets all *trans flags)",
+ "Memory is transferred in a way that violates annotations.",
+ 0, 0
},
{
- FK_DEBUG, FK_NONE, debugFlag,
- "nopp",
- FLG_NOPP,
- NULL, NULL,
+ FK_MEMORY, FK_NONE, modeFlag,
+ "dependenttrans",
+ FLG_DEPENDENTTRANS,
+ "dependent transfer errors",
+ "Dependent storage is transferred to a non-dependent reference.",
0, 0
},
{
- FK_DECL, FK_NONE, modeFlag,
- "shadow",
- FLG_SHADOW,
- "declaration reuses name visible in outer scope",
- "An outer declaration is shadowed by the local declaration.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "newreftrans",
+ FLG_NEWREFTRANS,
+ "new reference transfer to reference counted reference",
+ "A new reference is transferred to a reference counted reference.",
0, 0
},
{
- FK_DECL, FK_LIBS, modeFlag,
- "incondefslib",
- FLG_INCONDEFSLIB,
- "function, variable or constant defined in a library is redefined with inconsistent type",
- "A function, variable or constant previously defined in a library is "
- "redefined with a different type.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "onlytrans",
+ FLG_ONLYTRANS,
+ "only storage transferred to non-only reference (memory leak)",
+ "The only reference to this storage is transferred to another "
+ "reference (e.g., by returning it) that does not have the "
+ "only annotation. This may lead to a memory leak, since the "
+ "new reference is not necessarily released.",
0, 0
},
{
- FK_DECL, FK_LIBS, modeFlag,
- "overload",
- FLG_WARNOVERLOAD,
- "library function overloaded",
- "A function, variable or constant defined in the library is redefined "
- "with a different type.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "onlyunqglobaltrans",
+ FLG_ONLYUNQGLOBALTRANS,
+ "only storage transferred to an unqualified global or "
+ "static reference (memory leak)",
+ "The only reference to this storage is transferred to another "
+ "reference that does not have an aliasing annotation. "
+ "This may lead to a memory leak, since the "
+ "new reference is not necessarily released.",
0, 0
},
{
- FK_DECL, FK_NONE, modeFlag,
- "nestedextern",
- FLG_NESTEDEXTERN,
- "an extern declaration is inside a function scope",
- "An extern declaration is used inside a function scope.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "ownedtrans",
+ FLG_OWNEDTRANS,
+ "owned storage transferred to non-owned reference (memory leak)",
+ "The owned reference to this storage is transferred to another "
+ "reference (e.g., by returning it) that does not have the "
+ "owned annotation. This may lead to a memory leak, since the "
+ "new reference is not necessarily released.",
0, 0
- },
+ },
{
- FK_DECL, FK_NONE, modeFlag,
- "redecl",
- FLG_REDECL,
- "function or variable redeclared",
- "A function or variable is declared in more than one place. This is "
- "not necessarily a problem, since the declarations are consistent.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "freshtrans",
+ FLG_FRESHTRANS,
+ "fresh storage transferred to non-only reference (memory leak)",
+ "Fresh storage (newly allocated in this function) is transferred "
+ "in a way that the obligation to release storage is not "
+ "propagated. Use the /*@only@*/ annotation to indicate "
+ "the a return value is the only reference to the returned "
+ "storage.",
0, 0
- },
+ },
{
- FK_DECL, FK_NONE, plainFlag,
- "redef",
- FLG_REDEF,
- "function or variable redefined",
- "A function or variable is redefined. One of the declarations should use extern.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "sharedtrans",
+ FLG_SHAREDTRANS,
+ "shared storage transferred to non-shared reference",
+ "Shared storage is transferred to a non-shared reference. The other "
+ "reference may release storage needed by this reference.",
0, 0
},
{
- FK_DECL, FK_TYPE, modeFlag,
- "incondefs",
- FLG_INCONDEFS,
- "function, variable or constant redefined with inconsistent type",
- "A function, variable or constant is redefined with a different type.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "temptrans",
+ FLG_TEMPTRANS,
+ "temp storage transferred to non-temporary reference",
+ "Temp storage (associated with a formal parameter) is transferred "
+ "to a non-temporary reference. The storage may be released "
+ "or new aliases created.",
0, 0
},
{
- FK_DECL, FK_TYPE, modeFlag,
- "imptype",
- FLG_IMPTYPE,
- "variable declaration has unknown (implicitly int) type",
- "A variable declaration has no explicit type. The type is implicitly int.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "kepttrans",
+ FLG_KEPTTRANS,
+ "kept storage transferred to non-temporary reference",
+ "storage is transferred "
+ "to a non-temporary reference after being passed as keep parameter. The storage may be released "
+ "or new aliases created.",
0, 0
},
{
- FK_DECL, FK_TYPE, modeFlag,
- "matchfields",
- FLG_MATCHFIELDS,
- "struct or enum type redefined with inconsistent fields or members",
- "A struct, union or enum type is redefined with inconsistent fields or members.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "keeptrans",
+ FLG_KEEPTRANS,
+ "keep storage transferred inconsistently",
+ "Keep storage is transferred inconsistently --- either in a way "
+ "that may add a new alias to it, or release it.",
0, 0
},
{
- FK_DEF, FK_NONE, modeFlag,
- "usedef",
- FLG_USEDEF,
- "use before definition",
- "An rvalue is used that may not be initialized to a value on some execution path.",
+ FK_MEMORY, FK_NONE, modeFlag,
+ "immediatetrans",
+ FLG_IMMEDIATETRANS,
+ "an immediate address (result of &) is transferred inconsistently",
+ "An immediate address (result of & operator) is transferred "
+ "inconsistently.",
0, 0
},
{
- FK_DEF, FK_IMPLICIT, modeFlag,
- "impouts",
- FLG_IMPOUTS,
- "pointer parameters to unspecified functions may be implicit out parameters",
- NULL, 0, 0
+ FK_MEMORY, FK_NONE, modeFlag,
+ "refcounttrans",
+ FLG_REFCOUNTTRANS,
+ "reference counted storage is transferred in an inconsistent way",
+ "Reference counted storage is transferred in a way that may not "
+ "be consistent with the reference count.",
+ 0, 0
},
{
- FK_DIRECT, FK_FILES, globalStringFlag,
- "tmpdir",
- FLG_TMPDIR,
- "set directory for writing temp files",
- NULL, 0, 0
+ FK_MEMORY, FK_NONE, modeFlag,
+ "statictrans",
+ FLG_STATICTRANS,
+ "static storage is transferred in an inconsistent way",
+ "Static storage is transferred in an inconsistent way.",
+ 0, 0
},
{
- FK_DIRECT, FK_FILES, globalStringFlag,
- "larchpath",
- FLG_LARCHPATH,
- "set path for searching for library files (overrides LARCH_PATH environment variable)",
- NULL, 0, 0
+ FK_MEMORY, FK_NONE, modeFlag,
+ "unqualifiedtrans",
+ FLG_UNKNOWNTRANS,
+ "unqualified storage is transferred in an inconsistent way",
+ "Unqualified storage is transferred in an inconsistent way.",
+ 0, 0
},
{
- FK_DIRECT, FK_FILES, globalStringFlag,
- "lclimportdir",
- FLG_LCLIMPORTDIR,
- "set directory to search for LCL import files (overrides LCLIMPORTDIR)",
- NULL, 0, 0
+ FK_MEMORY, FK_NONE, modeFlag,
+ "staticinittrans",
+ FLG_STATICINITTRANS,
+ "static storage is used as an initial value in an inconsistent way",
+ "Static storage is used as an initial value in an inconsistent way.",
+ 0, 0
},
{
- FK_DIRECT, FK_FILES, globalStringFlag,
- "sysdirs",
- FLG_SYSTEMDIRS,
- "set directories for system files (default /usr/include). Separate "
- "directories with colons (e.g., \"/usr/include:/usr/local/lib\"). "
- "Flag settings propagate to files in a system directory. If "
- "-sysdirerrors is set, no errors are reported for files in "
- "system directories.",
- NULL, 0, 0
+ FK_MEMORY, FK_NONE, modeFlag,
+ "unqualifiedinittrans",
+ FLG_UNKNOWNINITTRANS,
+ "unqualified storage is used as an initial value in an inconsistent way",
+ "Unqualified storage is used as an initial value in an inconsistent way.",
+ 0, 0
},
{
- FK_DIRECT, FK_FILES, plainFlag,
- "skipansiheaders",
- FLG_SKIPANSIHEADERS,
- "prevent inclusion of header files in a system directory with "
- "names that match standard ANSI headers. The symbolic information "
- "in the standard library is used instead. Flag in effect only "
- "if a library including the ANSI library is loaded. The ANSI "
- "headers are: assert, ctype, errno, float, limits, locale, math, "
- "setjmp, signal, stdarg, stddef, stdio, stdlib, strings, string, "
- "time, and wchar.",
- NULL, 0, 0
+ FK_MEMORY, FK_NONE, modeFlag,
+ "readonlytrans",
+ FLG_READONLYTRANS,
+ "report memory transfer errors for initializations to read-only string literals",
+ "A read-only string literal is assigned to a non-observer reference.",
+ 0, 0
},
{
- FK_DIRECT, FK_FILES, plainFlag,
- "skipposixheaders",
- FLG_SKIPPOSIXHEADERS,
- "prevent inclusion of header files in a system directory with "
- "names that match standard POSIX headers. The symbolic information "
- "in the posix library is used instead. The POSIX headers are: "
- "dirent, fcntl, grp, pwd, termios, sys/stat, sys/times, "
- "sys/types, sys/utsname, sys/wait, unistd, and utime.",
+ FK_MEMORY, FK_PARAMS, modeFlag,
+ "passunknown",
+ FLG_PASSUNKNOWN,
+ "passing a value as an un-annotated parameter clears its annotation",
NULL, 0, 0
},
+
+ /* 5.3 Implicit Memory Annotations */
+
{
- FK_DIRECT, FK_SUPPRESS, modeFlag,
- "sysdirerrors",
- FLG_SYSTEMDIRERRORS,
- "report errors in files in system directories (set by -sysdirs)",
- NULL, 0, 0
+ FK_MEMORY, FK_NONE, modeFlag,
+ "readonlystrings",
+ FLG_READONLYSTRINGS,
+ "string literals are read-only (error if one is modified or released)",
+ "String literals are read-only. An error is reported "
+ "if a string literal may be modified or released.",
+ 0, 0
},
{
- FK_HEADERS, FK_MACROS, plainFlag,
- "sysdirexpandmacros",
- FLG_SYSTEMDIREXPAND,
- "expand macros in system directories regardless of other settings, "
- "except for macros corresponding to names defined in a load library",
+ FK_MEMORY, FK_IMPLICIT, modeFlag,
+ "memimp",
+ FLG_MEMIMPLICIT,
+ "memory errors for unqualified storage",
NULL, 0, 0
},
-
{
- FK_DIRECT, FK_HEADERS, globalExtraArgFlag,
- "I<directory>",
- FLG_INCLUDEPATH,
- "add to C include path",
+ FK_MEMORY, FK_IMPLICIT, plainFlag,
+ "paramimptemp",
+ FLG_PARAMIMPTEMP,
+ "assume unannotated parameter is temp",
NULL, 0, 0
},
{
- FK_DIRECT, FK_SPEC, globalExtraArgFlag,
- "S<directory>",
- FLG_SPECPATH,
- "add to spec path",
+ FK_MEMORY, FK_IMPLICIT, specialFlag,
+ "allimponly",
+ FLG_ALLIMPONLY,
+ "sets globimponly, retimponly, structimponly, specglobimponly, "
+ "specretimponly and specstructimponly",
NULL, 0, 0
- },
+ },
{
- FK_DISPLAY, FK_ERRORS, plainFlag,
- "quiet",
- FLG_QUIET,
- "suppress herald and error count",
+ FK_MEMORY, FK_IMPLICIT, specialFlag,
+ "codeimponly",
+ FLG_CODEIMPONLY,
+ "sets globimponly, retimponly and structimponly",
NULL, 0, 0
- },
+ },
{
- FK_DISPLAY, FK_ERRORS, plainFlag,
- "usestderr",
- FLG_USESTDERR,
- "send error messages to standard error (instead of standard out)",
+ FK_MEMORY, FK_IMPLICIT, specialFlag,
+ "specimponly",
+ FLG_SPECALLIMPONLY,
+ "sets specglobimponly, specretimponly and specstructimponly",
+ NULL, 0, 0
+ },
+ {
+ FK_MEMORY, FK_IMPLICIT, plainFlag,
+ "globimponly",
+ FLG_GLOBIMPONLY,
+ "assume unannotated global storage is only",
NULL, 0, 0
},
{
- FK_DISPLAY, FK_ERRORS, plainFlag,
- "showsummary",
- FLG_SHOWSUMMARY,
- "show summary of all errors reported and suppressed",
+ FK_MEMORY, FK_IMPLICIT, plainFlag,
+ "retimponly",
+ FLG_RETIMPONLY,
+ "assume unannotated returned storage is only",
NULL, 0, 0
},
{
- FK_DISPLAY, FK_FILES, plainFlag,
- "showscan",
- FLG_SHOWSCAN,
- "show file names are they are processed",
+ FK_MEMORY, FK_IMPLICIT, plainFlag,
+ "structimponly",
+ FLG_STRUCTIMPONLY,
+ "assume unannotated structure field is only",
NULL, 0, 0
},
{
- FK_DISPLAY, FK_NONE, globalFlag,
- "stats",
- FLG_STATS,
- "display lines processed and time",
+ FK_MEMORY, FK_IMPLICIT, plainFlag,
+ "specglobimponly",
+ FLG_SPECGLOBIMPONLY,
+ "assume unannotated global storage is only",
NULL, 0, 0
},
{
- FK_DISPLAY, FK_NONE, globalFlag,
- "timedist",
- FLG_TIMEDIST,
- "display time distribution",
+ FK_MEMORY, FK_IMPLICIT, plainFlag,
+ "specretimponly",
+ FLG_SPECRETIMPONLY,
+ "assume unannotated returned storage is only",
NULL, 0, 0
},
{
- FK_DISPLAY, FK_USE, globalFlag,
- "showalluses",
- FLG_SHOWUSES,
- "show sorted list of uses of all globals",
+ FK_MEMORY, FK_IMPLICIT, plainFlag,
+ "specstructimponly",
+ FLG_SPECSTRUCTIMPONLY,
+ "assume unannotated structure field is only",
NULL, 0, 0
},
+
+ /* Reference Counting */
+
+
+
+ /*
+ ** 6. Sharing
+ */
+
+ /* 6.1 Aliasing warnings */
+
{
- FK_EFFECT, FK_CONTROL, modeFlag,
- "noeffect",
- FLG_NOEFFECT,
- "statement with no effect",
- "Statement has no visible effect --- no values are modified.",
+ FK_ALIAS, FK_MEMORY, modeFlag,
+ "aliasunique",
+ FLG_ALIASUNIQUE,
+ "unique parameter is aliased",
+ "A unique or only parameter is aliased by some other parameter or visible global.",
0, 0
},
{
- FK_EFFECT, FK_CONTROL, modeFlag,
- "noeffectuncon",
- FLG_NOEFFECTUNCON,
- "statement with no effect (except possibly through call to "
- "unconstrained function)",
- "Statement has no visible effect --- no values are modified. It may "
- "modify something through a call to an unconstrained function.",
+ FK_ALIAS, FK_MEMORY, modeFlag,
+ "mayaliasunique",
+ FLG_MAYALIASUNIQUE,
+ "unique parameter may be aliased",
+ "A unique or only parameter may be aliased by some other parameter or visible global.",
0, 0
},
{
- FK_EXPORT, FK_SPEC, specialFlag,
- "exportany",
- FLG_EXPORTANY,
- "variable, function or type exported but not specified",
- "A variable, function or type is exported, but not specified.",
+ FK_ALIAS, FK_MEMORY, modeFlag,
+ "mustnotalias",
+ FLG_MUSTNOTALIAS,
+ "temp storage aliased at return point or scope exit",
+ "An alias has been added to a temp-qualifier parameter or global that is visible externally when the function returns. If the aliasing is needed, use the /*@shared@*/ annotation to indicate that new aliases to the parameter may be created.",
0, 0
},
{
- FK_EXPORT, FK_SPEC, modeFlag,
- "exportfcn",
- FLG_EXPORTFCN,
- "function exported but not specified",
- "A function is exported, but not specified.", 0, 0
+ FK_ALIAS, FK_NONE, modeFlag,
+ "retalias",
+ FLG_RETALIAS,
+ "function returns alias to parameter or global",
+ "The returned value shares storage with a parameter or global. If a parameter is to be returned, use the returned qualifier. If the result is not modified, use the observer qualifier on the result type. Otherwise, exposed can be used, but limited checking is done.", 0, 0
},
+
+ /* Global aliasing */
{
- FK_EXPORT, FK_SPEC, modeFlag,
- "exportmacro",
- FLG_EXPORTMACRO,
- "expanded macro exported but not specified",
- "A macro is exported, but not specified.", 0, 0
+ FK_ALIAS, FK_GLOBALS, specialFlag,
+ "globalias",
+ FLG_GLOBALIAS,
+ "function returns with global aliasing external state (sets "
+ "checkstrictglobalias, checkedglobalias, checkmodglobalias and "
+ "uncheckedglobalias)",
+ "A global variable aliases externally-visible state when the function returns.",
+ 0, 0
},
{
- FK_EXPORT, FK_SPEC, modeFlag,
- "exporttype",
- FLG_EXPORTTYPE,
- "type definition exported but not specified",
- "A type is exported, but not specified.", 0, 0
+ FK_ALIAS, FK_GLOBALS, modeFlag,
+ "checkstrictglobalias",
+ FLG_CHECKSTRICTGLOBALIAS,
+ "function returns with a checkstrict global aliasing external state",
+ "A global variable aliases externally-visible state when the function returns.",
+ 0, 0
},
{
- FK_EXPORT, FK_SPEC, modeFlag,
- "exportvar",
- FLG_EXPORTVAR,
- "variable exported but not specified",
- "A variable is exported, but not specified.", 0, 0
+ FK_ALIAS, FK_GLOBALS, modeFlag,
+ "checkedglobalias",
+ FLG_CHECKEDGLOBALIAS,
+ "function returns with a checked global aliasing external state",
+ "A global variable aliases externally-visible state when the function returns.",
+ 0, 0
},
{
- FK_EXPORT, FK_SPEC, modeFlag,
- "exportconst",
- FLG_EXPORTCONST,
- "constant exported but not specified",
- "A constant is exported, but not specified.", 0, 0
+ FK_ALIAS, FK_GLOBALS, modeFlag,
+ "checkmodglobalias",
+ FLG_CHECKMODGLOBALIAS,
+ "function returns with a checkmod global aliasing external state",
+ "A global variable aliases externally-visible state when the function returns.",
+ 0, 0
},
{
- FK_EXPORT, FK_SPEC, modeFlag,
- "exportiter",
- FLG_EXPORTITER,
- "constant exported but not specified",
- "A constant is exported, but not specified.", 0, 0
+ FK_ALIAS, FK_GLOBALS, modeFlag,
+ "uncheckedglobalias",
+ FLG_UNCHECKEDGLOBALIAS,
+ "function returns with an unchecked global aliasing external state",
+ "A global variable aliases externally-visible state when the function returns.",
+ 0, 0
+ },
+
+ /* 6.2 Exposure */
+
+ {
+ FK_MEMORY, FK_NONE, modeFlag,
+ "exposetrans",
+ FLG_EXPOSETRANS,
+ "exposure transfer errors",
+ "Exposed storage is transferred to a non-exposed, non-observer reference.",
+ 0, 0
+ },
+ {
+ FK_MEMORY, FK_NONE, modeFlag,
+ "observertrans",
+ FLG_OBSERVERTRANS,
+ "observer transfer errors",
+ "Observer storage is transferred to a non-observer reference.",
+ 0, 0
},
{
FK_EXPOSURE, FK_ABSTRACT, specialFlag,
"pointer into the abstract representation.",
0, 0
},
+ {
+ FK_DECL, FK_TYPE, modeFlag,
+ "redundantsharequal",
+ FLG_REDUNDANTSHAREQUAL,
+ "declaration uses observer qualifier that is always true",
+ "A declaration of an immutable object uses a redundant observer qualifier.",
+ 0, 0
+ } ,
+ {
+ FK_DECL, FK_TYPE, modeFlag,
+ "misplacedsharequal",
+ FLG_MISPLACEDSHAREQUAL,
+ "declaration of unsharable storage uses sharing annotation",
+ "A declaration of an unsharable object uses a sharing annotation.",
+ 0, 0
+ } ,
+
+ /*
+ ** 7. Function Interfaces
+ */
+
+ /* 7.1 Modifications */
{
- FK_FORMAT, FK_DISPLAY, valueFlag,
- "linelen",
- FLG_LINELEN,
- "set length of messages (number of chars)",
- NULL, 0, 0
+ FK_MODIFIES, FK_SPEC, plainFlag,
+ "mods",
+ FLG_MODIFIES,
+ "unspecified modification of caller-visible state",
+ "An externally-visible object is modified by a function, but not "
+ "listed in its modifies clause.",
+ 0, 0
},
{
- FK_FORMAT, FK_DISPLAY, valueFlag,
- "indentspaces",
- FLG_INDENTSPACES,
- "set number of spaces to indent sub-messages",
- NULL, 0, 0
+ FK_MODIFIES, FK_SPEC, modeFlag,
+ "mustmod",
+ FLG_MUSTMOD,
+ "specified modification is not detected",
+ "An object listed in the modifies clause is not modified by the "
+ "implementation of the function. The modification may not "
+ "be detected if it is done through a call to an unspecified "
+ "function.",
+ 0, 0
},
{
- FK_FORMAT, FK_DISPLAY, plainFlag,
- "showcolumn",
- FLG_SHOWCOL,
- "show column number where error is found",
- NULL, 0, 0
+ FK_MODIFIES, FK_MEMORY, plainFlag,
+ "modobserver",
+ FLG_MODOBSERVER,
+ "possible modification of observer storage",
+ "Storage declared with observer is possibly modified. Observer "
+ "storage may not be modified.",
+ 0, 0
},
{
- FK_FORMAT, FK_DISPLAY, plainFlag,
- "parenfileformat",
- FLG_PARENFILEFORMAT,
- "show column number where error is found",
+ FK_MODIFIES, FK_MEMORY, modeFlag,
+ "modobserveruncon",
+ FLG_MODOBSERVERUNCON,
+ "possible modification of observer storage through unconstrained call",
+ "Storage declared with observer may be modified through a call to an "
+ "unconstrained function.",
+ 0, 0
+ },
+ {
+ FK_MODIFIES, FK_MEMORY, modeFlag,
+ "modinternalstrict",
+ FLG_MODINTERNALSTRICT,
+ "possible modification of internal storage through function call",
+ "A function that modifies internalState is called from a function that "
+ "does not list internalState in its modifies clause",
+ 0, 0
+ },
+ {
+ FK_MODIFIES, FK_UNSPEC, modeFlag,
+ "modfilesys",
+ FLG_MODFILESYSTEM,
+ "report undocumented file system modifications (applies to unspecified "
+ "functions if modnomods is set)",
NULL, 0, 0
},
{
- FK_FORMAT, FK_NONE, plainFlag,
- "showfunc",
- FLG_SHOWFUNC,
- "show name of function containing error",
+ FK_MODIFIES, FK_UNSPEC, specialFlag,
+ "modunspec",
+ FLG_MODUNSPEC,
+ "modification in unspecified functions (sets modnomods, "
+ "modglobunspec and modstrictglobsunspec)",
NULL, 0, 0
},
{
- FK_FORMAT, FK_NONE, plainFlag,
- "showallconjs",
- FLG_SHOWALLCONJS,
- "show all possible types",
- "When a library function is declared with multiple possible type, the "
- "alternate types are shown only if +showallconjs.",
+ FK_MODIFIES, FK_UNSPEC, modeFlag,
+ "modnomods",
+ FLG_MODNOMODS,
+ "modification in a function with no modifies clause",
+ "An externally-visible object is modified by a function with no "
+ "/*@modifies@*/ comment. The /*@modifies ... @*/ control "
+ "comment can be used to give a modifies list for an "
+ "unspecified function.",
0, 0
},
{
- FK_LIBS, FK_NONE, plainFlag,
- "impconj",
- FLG_IMPCONJ,
- "make all alternate types implicit (useful for making system libraries",
- NULL, 0, 0
- } ,
+ FK_MODIFIES, FK_UNSPEC, modeFlag,
+ "moduncon",
+ FLG_MODUNCON,
+ "possible modification through a call to an unconstrained function",
+ "An unconstrained function is called in a function body where "
+ "modifications are checked. Since the unconstrained function "
+ "may modify anything, there may be undetected modifications in "
+ "the checked function.",
+ 0, 0
+ },
{
- FK_GLOBAL, FK_ERRORS, globalValueFlag,
- "expect",
- FLG_EXPECT,
- "expect <int> code errors",
- NULL, 0, 0
+ FK_MODIFIES, FK_UNSPEC, modeFlag,
+ "modunconnomods",
+ FLG_MODUNCONNOMODS,
+ "possible modification through a call to an unconstrained function in "
+ "a function with no modifies clause",
+ "An unconstrained function is called in a function body where "
+ "modifications are checked. Since the unconstrained function "
+ "may modify anything, there may be undetected modifications in "
+ "the checked function.",
+ 0, 0
},
{
- FK_GLOBAL, FK_ERRORS, globalValueFlag,
- "lclexpect",
- FLG_LCLEXPECT,
- "expect <int> spec errors",
- NULL, 0, 0
+ FK_MODIFIES, FK_GLOBALS, modeFlag,
+ "globsimpmodsnothing",
+ FLG_GLOBALSIMPMODIFIESNOTHING,
+ "functions declared with a globals list but no modifies clause are "
+ "assumed to modify nothing",
+ "An implicit modifies nothing clause is assumed for a function "
+ "declared with a globals list but not modifies clause.",
+ 0, 0
},
{
- FK_GLOBAL, FK_USE, idemSpecialFlag,
- "partial",
- FLG_PARTIAL,
- "check as partial system (-specundef, -declundef, -exportlocal, "
- "don't check macros in headers without corresponding .c files)",
- NULL, 0, 0
+ FK_MODIFIES, FK_GLOBALS, modeFlag,
+ "modsimpnoglobs",
+ FLG_MODIFIESIMPNOGLOBALS,
+ "functions declared with a modifies clause but no globals list "
+ "are assumed to use no globals",
+ "An implicit empty globals list is assumed for a function "
+ "declared with a modifies clause but no globals list.",
+ 0, 0
+ },
+
+ /*
+ ** Globals
+ */
+
+ {
+ FK_GLOBALS, FK_NONE, modeFlag,
+ "globstate",
+ FLG_GLOBSTATE,
+ "returns with global in inconsistent state (null or undefined)",
+ "A global variable does not satisfy its annotations when control is transferred.",
+ 0, 0
},
+
{
FK_GLOBALS, FK_SPEC, plainFlag,
"globs",
"warnmissingglobs",
FLG_WARNMISSINGGLOBALS,
"global variable used in modifies clause is not listed in globals list",
- "A global variable is used in the modifies clause, but it not listed in "
+ "A global variable is used in the modifies clause, but it is not listed in "
"the globals list. The variable will be added to the globals list.",
0, 0
},
"its modifies clause.",
0, 0
},
+
+ /*
+ ** State Clauses
+ */
+
+ /*
+ ** 8. Control Flow
+ */
+
+ /* 8.1 Execution */
+
{
- FK_HEADERS, FK_DEBUG, debugFlag,
- "keep",
- FLG_KEEP,
- NULL, NULL,
- 0, 0
- },
- {
- FK_HEADERS, FK_SPEC, globalFlag,
- "lh",
- FLG_DOLH,
- "generate .lh files", NULL,
- 0, 0
- },
- {
- FK_HEADERS, FK_SPEC, globalFlag,
- "lcs",
- FLG_DOLCS,
- "generate .lcs files", NULL,
- 0, 0
- },
- {
- FK_HEADERS, FK_SPEED, globalFlag,
- "singleinclude",
- FLG_SINGLEINCLUDE,
- "optimize header inclusion to eliminate redundant includes",
- "When checking multiple files, each header file is processed only "
- "once. This may change the meaning of the code, if the "
- "same header file is included in different contexts (e.g., the "
- "header file includes #if directives and the values are "
- "different when it is included in different places.)",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "noret",
+ FLG_NORETURN,
+ "path with no return detected in non-void function",
+ "There is a path through a function declared to return a value on "
+ "which there is no return statement. This means the execution "
+ "may fall through without returning a meaningful result to "
+ "the caller.",
0, 0
},
{
- FK_HEADERS, FK_SPEED, globalFlag,
- "neverinclude",
- FLG_NEVERINCLUDE,
- "optimize header inclusion to not include any header files",
- "Ignore header includes. Only works if relevant information is "
- "loaded from a library.",
+ FK_CONTROL, FK_NONE, plainFlag,
+ "emptyret",
+ FLG_EMPTYRETURN,
+ "empty return in function declared to return value",
+ NULL,
0, 0
},
{
- FK_HEADERS, FK_SPEED, globalFlag,
- "skipsysheaders",
- FLG_SKIPSYSHEADERS,
- "do not include header files in system directories (set by -sysdirs)",
- "Do not include header files in system directories (set by -sysdirs)",
+ FK_CONTROL, FK_NONE, plainFlag,
+ "alwaysexits",
+ FLG_ALWAYSEXITS,
+ "loop predicate always exits",
+ NULL,
0, 0
},
+
{
- FK_HELP, FK_NONE, plainFlag,
- "warnflags",
- FLG_WARNFLAGS,
- "warn when command line sets flag to default value in mode",
+ FK_CONTROL, FK_MEMORY, specialFlag,
+ "loopexec",
+ FLG_LOOPEXEC,
+ "assume all loops execute at least once (sets forloopexec, whileloopexec and iterloopexec)",
NULL, 0, 0
},
{
- FK_HELP, FK_NONE, plainFlag,
- "warnunixlib",
- FLG_WARNUNIXLIB,
- "warn when the unix library is used",
+ FK_CONTROL, FK_MEMORY, plainFlag,
+ "forloopexec",
+ FLG_FORLOOPEXEC,
+ "assume all for loops execute at least once",
NULL, 0, 0
},
{
- FK_HELP, FK_NONE, plainFlag,
- "badflag",
- FLG_BADFLAG,
- "a flag is not recognized or used in an incorrect way",
+ FK_CONTROL, FK_MEMORY, plainFlag,
+ "whileloopexec",
+ FLG_WHILELOOPEXEC,
+ "assume all while loops execute at least once",
NULL, 0, 0
},
{
- FK_HELP, FK_NONE, plainFlag,
- "forcehints",
- FLG_FORCEHINTS,
- "provide hints for all errors reported",
+ FK_CONTROL, FK_MEMORY, plainFlag,
+ "iterloopexec",
+ FLG_ITERLOOPEXEC,
+ "assume all iterator loops execute at least once",
NULL, 0, 0
},
{
- FK_HELP, FK_NONE, globalExtraArgFlag,
- "help",
- FLG_HELP,
- "-help <flags> will describe flags",
+ FK_CONTROL, FK_MEMORY, plainFlag,
+ "obviousloopexec",
+ FLG_OBVIOUSLOOPEXEC,
+ "assume loop that can be determined to always execute always does",
NULL, 0, 0
},
+
+ /* 8.2 Undefined Behavior */
+
{
- FK_HELP, FK_FORMAT, plainFlag,
- "hints",
- FLG_HINTS,
- "provide hints on using lclint (hint only reported for first related error)",
- NULL, 0, 0
+ FK_BEHAVIOR, FK_ANSI, modeFlag,
+ "evalorder",
+ FLG_EVALORDER,
+ "code has unspecified or implementation-dependent behavior "
+ "because of order of evaluation",
+ "Code has unspecified behavior. "
+ "Order of evaluation of function parameters or subexpressions "
+ "is not defined, so if a value is used and modified in different "
+ "places not separated by a sequence point constraining "
+ "evaluation order, then the result of the expression is "
+ "unspecified.",
+ 0, 0
},
{
- FK_IGNORERET, FK_NONE, specialFlag,
- "retval",
- FLG_RETVAL,
- "return value ignored (sets retvalint, retvalbool and retvalother)",
- "Result returned by function call is not used. If this is intended, "
- "cast result to (void) to eliminate message.",
+ FK_BEHAVIOR, FK_ANSI, modeFlag,
+ "evalorderuncon",
+ FLG_EVALORDERUNCON,
+ "code involving call to unspecified function has undefined or implementation-dependent behavior",
+ "Code involving a call to function with no modifies or globals clause "
+ "may have undefined or implementation-dependent behavior (Splint assumes the "
+ "unconstrained call may modify any reachable state or use any global). Add a "
+ "specification for the function.",
0, 0
},
+
+ /* 8.3 Problematic Control Structures */
+
+ /* 8.3.1 Infinite Loops */
+
{
- FK_IGNORERET, FK_BOOL, modeFlag,
- "retvalother",
- FLG_RETVALOTHER,
- "return value of type other than bool or int ignored",
- "Result returned by function call is not used. If this is intended, "
- "can cast result to (void) to eliminate message.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "infloops",
+ FLG_INFLOOPS,
+ "likely infinite loop is detected",
+ "This appears to be an infinite loop. Nothing in the body of the "
+ "loop or the loop test modifies the value of the loop test. "
+ "Perhaps the specification of a function called in the loop "
+ "body is missing a modification.",
0, 0
},
{
- FK_IGNORERET, FK_BOOL, modeFlag,
- "retvalbool",
- FLG_RETVALBOOL,
- "return value of manifest type bool ignored",
- "Result returned by function call is not used. If this is intended, "
- "can cast result to (void) to eliminate message.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "infloopsuncon",
+ FLG_INFLOOPSUNCON,
+ "likely infinite loop is detected (may result from unconstrained function)",
+ "This appears to be an infinite loop. Nothing in the body of the "
+ "loop or the loop test modifies the value of the loop test. "
+ "There may be a modification through a call to an unconstrained "
+ "function, or an unconstrained function in the loop test may use "
+ "a global variable modified by the loop body.",
0, 0
},
+
+ /* 8.3.2 Switches */
+
{
- FK_IGNORERET, FK_NONE, modeFlag,
- "retvalint",
- FLG_RETVALINT,
- "return value of type int ignored",
- "Result returned by function call is not used. If this is intended, "
- "can cast result to (void) to eliminate message.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "casebreak",
+ FLG_CASEBREAK,
+ "non-empty case in a switch without preceding break",
+ "Execution falls through from the previous case.", 0, 0
+ },
+ {
+ FK_CONTROL, FK_NONE, modeFlag,
+ "misscase",
+ FLG_MISSCASE,
+ "switch on enum type missing case for some value",
+ "Not all values in an enumeration are present as cases in the switch.",
0, 0
},
{
- FK_INIT, FK_FILES, globalExtraArgFlag,
- "f",
- FLG_OPTF,
- "load options file (default ~/.lclintrc not loaded)",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, modeFlag,
+ "firstcase",
+ FLG_FIRSTCASE,
+ "first statement in switch is not a case",
+ "The first statement after a switch is not a case.",
+ 0, 0
},
{
- FK_INIT, FK_FILES, globalExtraArgFlag,
- "i",
- FLG_INIT,
- "set LCL initilization file",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, plainFlag,
+ "duplicatecases",
+ FLG_DUPLICATECASES,
+ "duplicate cases in switch",
+ "Duplicate cases in switch.",
+ 0, 0
},
+
+ /* 8.3.3 Deep Breaks */
+
{
- FK_INIT, FK_FILES, globalFlag,
- "nof",
- FLG_NOF,
- "do not read options file",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, specialFlag,
+ "deepbreak",
+ FLG_DEEPBREAK,
+ "break inside nested while or for or switch",
+ "A break statement appears inside the body of a nested while, for or "
+ "switch statement. Sets looploopbreak, loopswitchbreak, "
+ "switchloopbreak, switchswitchbreak, and looploopcontinue.",
+ 0, 0
},
{
- FK_INIT, FK_SPEC, plainFlag,
- "needspec",
- FLG_NEEDSPEC,
- "information in specifications is not also included in syntactic comments",
- "There is information in the specification that is not duplicated "
- "in syntactic comments. Normally, this is not an "
- "error, but it may be useful to detect it to make "
- "sure checking incomplete systems without the specifications will "
- "still use this information.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "looploopbreak",
+ FLG_LOOPLOOPBREAK,
+ "break inside nested while or for",
+ "A break statement appears inside the body of a nested while or for "
+ "statement. This is perfectly reasonable code, but check that "
+ "the break is intended to break only the inner loop. The "
+ "break statement may be preceded by /*@innerbreak@*/ to suppress "
+ "the message for this break only.",
0, 0
},
{
- FK_DECL, FK_LIBS, plainFlag,
- "newdecl",
- FLG_NEWDECL,
- "report new global declarations in source files",
- "There is a new declaration that is not declared in a loaded library "
- "or earlier file. (Use this flag to check for consistency "
- "against a library.)",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "switchloopbreak",
+ FLG_SWITCHLOOPBREAK,
+ "break in loop inside switch",
+ "A break statement appears inside the body of a while or for "
+ "statement within a switch. This is perfectly reasonable code, but check that "
+ "the break is intended to break only the inner loop. The "
+ "break statement may be preceded by /*@loopbreak@*/ to suppress "
+ "the message for this break only.",
0, 0
},
{
- FK_ITER, FK_NONE, plainFlag,
- "iter",
- FLG_ITER,
- NULL, NULL,
+ FK_CONTROL, FK_NONE, modeFlag,
+ "loopswitchbreak",
+ FLG_LOOPSWITCHBREAK,
+ "break in switch inside loop",
+ "A break statement appears inside a switch statement within a while or "
+ "for loop. This is perfectly reasonable code, but check that "
+ "the break is intended to break only the inner loop. The "
+ "break statement may be preceded by /*@switchbreak@*/ to suppress "
+ "the message for this break only.",
0, 0
},
{
- FK_ITER, FK_NONE, plainFlag,
- "hasyield",
- FLG_HASYIELD,
- "iter declaration has no yield parameters",
- "An iterator has been declared with no parameters annotated with "
- "yield. This may be what you want, if the iterator is meant "
- "to do something a fixed number of times, but returns no "
- "information to the calling context. Probably, a parameter "
- "is missing the yield annotation to indicate that it is "
- "assigned a value in the calling context.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "switchswitchbreak",
+ FLG_SWITCHSWITCHBREAK,
+ "break in switch inside switch",
+ "A break statement appears inside a switch statement within another "
+ "switch statement. This is perfectly reasonable code, but check that "
+ "the break is intended to break only the inner switch. The "
+ "break statement may be preceded by /*@innerbreak@*/ to suppress "
+ "the message for this break only.",
0, 0
},
{
- FK_LIBS, FK_FILES, globalStringFlag,
- "dump",
- FLG_DUMP,
- "save state for merging (default suffix .lcd)",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, modeFlag,
+ "looploopcontinue",
+ FLG_LOOPLOOPCONTINUE,
+ "continue inside nested loop",
+ "A continue statement appears inside a loop within a loop. "
+ "This is perfectly reasonable code, but check that "
+ "the continue is intended to continue only the inner loop. The "
+ "continue statement may be preceded by /*@innercontinue@*/ to suppress "
+ "the message for this continue only.",
+ 0, 0
},
+
+ /* 8.3.4 Loop and If Bodies */
+
{
- FK_LIBS, FK_FILES, globalStringFlag,
- "load",
- FLG_MERGE,
- "load state from dump file (default suffix .lcd)",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, modeFlag,
+ "whileempty",
+ FLG_WHILEEMPTY,
+ "a while statement has no body",
+ "While statement has no body.",
+ 0, 0
},
{
- FK_LIBS, FK_INIT, idemGlobalFlag,
- "nolib",
- FLG_NOLIB,
- "do not load standard library",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, modeFlag,
+ "whileblock",
+ FLG_WHILEBLOCK,
+ "the body of a while statement is not a block",
+ "While body is a single statement, not a compound block.",
+ 0, 0
},
{
- FK_LIBS, FK_INIT, globalFlag,
- "ansilib",
- FLG_ANSILIB,
- "use ANSI (ISO) standard library",
- "Library based on the ANSI standard library specification is used.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "forempty",
+ FLG_FOREMPTY,
+ "a for statement has no body",
+ "For statement has no body.",
0, 0
},
{
- FK_LIBS, FK_INIT, globalFlag,
- "strictlib",
- FLG_STRICTLIB,
- "interpret standard library strictly",
- "Stricter version of the standard library is used. (The default "
- "library is standard.lcd; strict library is strict.lcd.)",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "forblock",
+ FLG_FORBLOCK,
+ "the body of a for statement is not a block",
+ "Loop body is a single statement, not a compound block.",
0, 0
},
{
- FK_LIBS, FK_INIT, globalFlag,
- "unixlib",
- FLG_UNIXLIB,
- "use UNIX (sort-of) standard library",
- "UNIX version of the standard library is used.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "ifempty",
+ FLG_IFEMPTY,
+ "an if statement has no body",
+ "If statement has no body.",
0, 0
},
{
- FK_LIBS, FK_INIT, globalFlag,
- "unixstrictlib",
- FLG_UNIXSTRICTLIB,
- "use strict version of UNIX (sort-of) library",
- "strict version of the UNIX library is used.",
+ FK_CONTROL, FK_NONE, modeFlag,
+ "ifblock",
+ FLG_IFBLOCK,
+ "the body of an if statement is not a block",
+ "If body is a single statement, not a compound block.",
0, 0
},
{
- FK_LIBS, FK_INIT, globalFlag,
- "posixlib",
- FLG_POSIXLIB,
- "use POSIX standard library",
- "POSIX version of the standard library is used.",
+ FK_CONTROL, FK_NONE, specialFlag,
+ "allempty",
+ FLG_ALLEMPTY,
+ "an if, while or for statement has no body (sets ifempty, "
+ "whileempty and forempty",
+ NULL,
0, 0
},
{
- FK_LIBS, FK_INIT, globalFlag,
- "posixstrictlib",
- FLG_POSIXSTRICTLIB,
- "use strict POSIX standard library",
- "POSIX version of the strict standard library is used.",
+ FK_CONTROL, FK_NONE, specialFlag,
+ "allblock",
+ FLG_ALLBLOCK,
+ "the body of an if, while or for statement is not a block "
+ "(sets ifblock, whileblock and forblock)",
+ "Body is a single statement, not a compound block.",
0, 0
},
+
+ /* 8.3.5 Complete Logic */
+
{
- FK_LIBS, FK_INIT, globalFlag,
- "whichlib",
- FLG_WHICHLIB,
- "show standard library filename",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, modeFlag,
+ "elseifcomplete",
+ FLG_ELSEIFCOMPLETE,
+ "if ... else if chains must have final else",
+ "There is no final else following an else if construct.",
+ 0, 0
},
+
+ /* 8.4 Suspicious Statements */
+
{
- FK_LIBS, FK_FILES, globalStringFlag,
- "mts",
- FLG_MTSFILE,
- "load meta state declaration and corresponding xh file",
- NULL, 0, 0
+ FK_CONTROL, FK_NONE, modeFlag,
+ "unreachable",
+ FLG_UNREACHABLE,
+ "unreachable code detected",
+ "This code will never be reached on any possible execution.",
+ 0, 0
},
+
+ /* 8.4.1 Statements with No Effects */
+
{
- FK_INIT, FK_COMMENTS, valueFlag,
- "commentchar",
- FLG_COMMENTCHAR,
- "set marker character for syntactic comments (default is '@')",
- "Set the marker character for syntactic comments. Comments beginning "
- "with /*<char> are interpreted by lclint, where <char> is the "
- "comment marker character.",
+ FK_EFFECT, FK_CONTROL, modeFlag,
+ "noeffect",
+ FLG_NOEFFECT,
+ "statement with no effect",
+ "Statement has no visible effect --- no values are modified.",
0, 0
},
{
- FK_MACROS, FK_NONE, plainSpecialFlag,
- "allmacros",
- FLG_ALLMACROS,
- "sets fcnmacros and constmacros",
- "All macros (not preceded by /*@notfunction@*/) are checked as functions or "
- "constants depending on whether or not they have parameter lists.",
- 0, 0
+ FK_EFFECT, FK_CONTROL, modeFlag,
+ "noeffectuncon",
+ FLG_NOEFFECTUNCON,
+ "statement with no effect (except possibly through call to "
+ "unconstrained function)",
+ "Statement has no visible effect --- no values are modified. It may "
+ "modify something through a call to an unconstrained function.",
+ 0, 0
},
+
+ /* 8.4.2 Ignored Return Values */
+
{
- FK_MACROS, FK_NONE, plainFlag,
- "libmacros",
- FLG_LIBMACROS,
- "check all macros with declarations in library as functions",
- "Every macro declared in the load library is checked.",
- 0, 0
+ FK_IGNORERET, FK_NONE, specialFlag,
+ "retval",
+ FLG_RETVAL,
+ "return value ignored (sets retvalint, retvalbool and retvalother)",
+ "Result returned by function call is not used. If this is intended, "
+ "cast result to (void) to eliminate message.",
+ 0, 0
},
{
- FK_MACROS, FK_NONE, plainFlag,
- "specmacros",
- FLG_SPECMACROS,
- "check all macros corresponding to specified functions or constants",
- "Every macro declared a specification file is checked.",
- 0, 0
+ FK_IGNORERET, FK_BOOL, modeFlag,
+ "retvalother",
+ FLG_RETVALOTHER,
+ "return value of type other than bool or int ignored",
+ "Result returned by function call is not used. If this is intended, "
+ "can cast result to (void) to eliminate message.",
+ 0, 0
},
{
- FK_MACROS, FK_NONE, plainFlag,
- "fcnmacros",
- FLG_FCNMACROS,
- "check all macros with parameter lists as functions",
- "Every parameterized macro (not preceded by /*@notfunction@*/) "
- "is checked as a function.",
- 0, 0
+ FK_IGNORERET, FK_BOOL, modeFlag,
+ "retvalbool",
+ FLG_RETVALBOOL,
+ "return value of manifest type bool ignored",
+ "Result returned by function call is not used. If this is intended, "
+ "can cast result to (void) to eliminate message.",
+ 0, 0
},
{
- FK_MACROS, FK_NONE, plainFlag,
- "constmacros",
- FLG_CONSTMACROS,
- "check all macros without parameter lists as constants",
- "Every non-parameterized macro (not preceded by /*@notfunction@*/) "
- "is checked as a constant.",
- 0, 0
+ FK_IGNORERET, FK_NONE, modeFlag,
+ "retvalint",
+ FLG_RETVALINT,
+ "return value of type int ignored",
+ "Result returned by function call is not used. If this is intended, "
+ "can cast result to (void) to eliminate message.",
+ 0, 0
+ },
+
+ /*
+ ** 9. Buffer Sizes
+ */
+
+ {
+ FK_BOUNDS, FK_MEMORY, modeFlag,
+ "nullterminated",
+ FLG_NULLTERMINATED,
+ "misuse of nullterminated allocation",
+ "A possibly non-nullterminated string/memory is used/referenced as a nullterminated one.",
+ 0, 0
+ },
+ {
+ FK_BOUNDS, FK_MEMORY, specialFlag,
+ "bounds",
+ FLG_BOUNDS,
+ "memory bounds checking (sets boundsread and boundswrite)",
+ "Memory read or write may be out of bounds of allocated storage.", 0, 0
+ },
+ {
+ FK_BOUNDS, FK_MEMORY, specialFlag,
+ "likelybounds",
+ FLG_LIKELYBOUNDS,
+ "memory bounds checking (sets likelyboundsread and likelyboundswrite)",
+ "Memory read or write may be out of bounds of allocated storage.", 0, 0
+ },
+ {
+ FK_BOUNDS, FK_MEMORY, modeFlag,
+ "likelyboundsread",
+ FLG_LIKELYBOUNDSREAD,
+ "likely out of bounds read",
+ "A memory read references memory beyond the allocated storage.",
+ 0, 0
+ },
+ {
+ FK_BOUNDS, FK_MEMORY, modeFlag,
+ "likelyboundswrite",
+ FLG_LIKELYBOUNDSWRITE,
+ "likely buffer overflow from an out of bounds write",
+ "A memory write may write to an address beyond the allocated buffer.",
+ 0, 0
+ },
+
+ {
+ FK_BOUNDS, FK_MEMORY, modeFlag,
+ "boundsread",
+ FLG_BOUNDSREAD,
+ "possible out of bounds read",
+ "A memory read references memory beyond the allocated storage.",
+ 0, 0
+ },
+ {
+ FK_BOUNDS, FK_MEMORY, modeFlag,
+ "boundswrite",
+ FLG_BOUNDSWRITE,
+ "possible buffer overflow from an out of bounds write",
+ "A memory write may write to an address beyond the allocated buffer.",
+ 0, 0
+ },
+
+ {
+ FK_BOUNDS, FK_DISPLAY, plainFlag,
+ "fcnpost",
+ FLG_FUNCTIONPOST,
+ "display function post conditions",
+ "Display function post conditions.",
+ 0, 0
+ },
+ {
+ FK_BOUNDS, FK_DISPLAY, plainFlag,
+ "redundantconstraints",
+ FLG_REDUNDANTCONSTRAINTS,
+ "display seemingly redundant constraints",
+ "Display seemingly redundant constraints",
+ 0, 0
+ },
+ /*drl7x added 6/18/01 */
+ {
+ FK_BOUNDS, FK_MEMORY, modeFlag,
+ "checkpost",
+ FLG_CHECKPOST,
+ "unable to verify predicate in ensures clause",
+ "The function implementation may not satisfy a post condition given in an ensures clause.",
+ 0, 0
+ },
+
+ {
+ FK_BOUNDS, FK_MEMORY, plainFlag,
+ "impboundsconstraints",
+ FLG_IMPLICTCONSTRAINT,
+ "generate implicit constraints for functions",
+ NULL,
+ 0, 0
+ },
+ /*drl7x added 4/29/01 */
+ {
+ FK_BOUNDS, FK_MEMORY, plainFlag,
+ "orconstraint",
+ FLG_ORCONSTRAINT,
+ "use limited OR expressions to resolve constraints",
+ NULL,
+ 0, 0
+ },
+ {
+ FK_BOUNDS, FK_DISPLAY, plainFlag,
+ "showconstraintparens",
+ FLG_PARENCONSTRAINT,
+ "display parentheses around constraint terms",
+ NULL,
+ 0, 0
+ },
+ /*drl added 2/4/2002*/
+ {
+ FK_BOUNDS, FK_DISPLAY, plainFlag,
+ "boundscompacterrormessages",
+ FLG_BOUNDSCOMPACTERRORMESSAGES,
+ "Display fewer new lines in bounds checking error messages",
+ NULL,
+ 0, 0
+ },
+ {
+ FK_BOUNDS, FK_DISPLAY, plainFlag,
+ "showconstraintlocation",
+ FLG_CONSTRAINTLOCATION,
+ "display location for every constraint generated",
+ NULL,
+ 0, 0
+ }, /*drl added flag 4/26/01*/
+
+ { /* evans added 2003-06-08 */
+ FK_BOUNDS, FK_MEMORY, modeFlag,
+ "allocmismatch",
+ FLG_ALLOCMISMATCH,
+ "type conversion involves storage of non-divisble size",
+ NULL, 0, 0
+ },
+
+ /*
+ ** 10. Extensible Checking
+ */
+
+ {
+ FK_EXTENSIBLE, FK_FILES, globalStringFlag, ARG_FILE,
+ "mts",
+ FLG_MTSFILE,
+ "load meta state declaration and corresponding xh file",
+ NULL, 0, 0
+ },
+ {
+ FK_EXTENSIBLE, FK_MEMORY, modeFlag,
+ "statetransfer",
+ FLG_STATETRANSFER,
+ "storage has been transfered with invalid state",
+ "Transfer violates user-defined state rules.",
+ 0, 0
+ },
+ {
+ FK_EXTENSIBLE, FK_MEMORY, modeFlag,
+ "statemerge",
+ FLG_STATEMERGE,
+ "control paths merge with storage in incompatible states",
+ "Control path merge violates user-defined state merge rules.",
+ 0, 0
},
+
+ /*
+ ** 11. Macros
+ */
+
{
FK_MACROS, FK_NONE, modeFlag,
- "macromatchname",
- FLG_MACROMATCHNAME,
- "macro definition does not match iter or constant declaration",
- "A iter or constant macro is defined using a different name from the "
- "one used in the previous syntactic comment",
+ "macroredef",
+ FLG_MACROREDEF,
+ "macro redefined",
+ "A macro is defined in more than one place.", 0, 0
+ },
+ {
+ FK_MACROS, FK_UNRECOG, modeFlag,
+ "macrounrecog",
+ FLG_MACROUNDEF,
+ "unrecognized identifier in macro",
+ "An unrecognized identifier appears in a macro. If the identifier "
+ "is defined before the macro is used, then this is okay.",
0, 0
},
+
+ /* 11.1 Constant Macros */
+
{
- FK_MACROS, FK_NONE, plainFlag,
- "nextlinemacros",
- FLG_MACRONEXTLINE,
- "the line after a constant or iter declaration must be a macro definition",
- "A constant or iter declaration is not immediately followed by a macro definition.",
+ FK_MACROS, FK_PROTOS, modeFlag,
+ "macroconstdecl",
+ FLG_MACROCONSTDECL,
+ "non-parameterized macro without prototype or specification",
+ "Macro constant has no declaration. Use /*@constant ...@*/ to "
+ "declare the macro.",
0, 0
},
+
+ /* 11.2 Function-like Macros */
+
{
FK_MACROS, FK_NONE, modeFlag,
"macrostmt",
FLG_MACROSTMT,
"macro definition is syntactically not equivalent to function",
- "A macro is defined in a way that may cause syntactic problems. If the macro returns a value, use commas to separate expressions; otherwise, use do { <macro body> } while (FALSE) construct.",
+ "A macro is defined in a way that may cause syntactic problems. "
+ "If the macro returns a value, use commas to separate expressions; "
+ "otherwise, use do { <macro body> } while (FALSE) construct.",
0, 0
},
{
"denote parameters that must be side-effect free.",
0, 0
},
+ {
+ FK_MACROS, FK_CONTROL, modeFlag,
+ "macroret",
+ FLG_MACRORETURN,
+ "return statement in macro body",
+ "The body of a macro declared as a function uses a return statement. "
+ "This exhibits behavior that "
+ "could not be implemented by a function.",
+ 0, 0
+ },
{
FK_MACROS, FK_PARAMS, modeFlag,
"macroassign",
"could not be implemented by a function.",
0, 0
},
- {
- FK_MACROS, FK_PARAMS, modeFlag,
- "sefparams",
- FLG_SEFPARAMS,
- "a parameter with side-effects is passed as a sef parameter",
- "An actual parameter corresponding to a sef parameter may have a side-effect.",
- 0, 0
- },
- {
- FK_MACROS, FK_PARAMS, modeFlag,
- "sefuncon",
- FLG_SEFUNSPEC,
- "a parameter with unconstrained side-effects is passed as a sef parameter",
- "An actual parameter corresponding to a sef parameter involves a call "
- "to a procedure with no modifies clause that may have a side-effect.",
- 0, 0
- },
{
FK_MACROS, FK_NONE, modeFlag,
"macroparens",
"Function macro has no declaration.",
0, 0
},
+
+ /* 11.2.1 Side Effect Free Parameters */
+
{
- FK_MACROS, FK_PROTOS, modeFlag,
- "macroconstdecl",
- FLG_MACROCONSTDECL,
- "non-parameterized macro without prototype or specification",
- "Macro constant has no declaration. Use /*@constant ...@*/ to "
- "declare the macro.",
+ FK_MACROS, FK_PARAMS, modeFlag,
+ "sefparams",
+ FLG_SEFPARAMS,
+ "a parameter with side-effects is passed as a sef parameter",
+ "An actual parameter corresponding to a sef parameter may have a side-effect.",
0, 0
},
{
- FK_MACROS, FK_NONE, modeFlag,
- "macroredef",
- FLG_MACROREDEF,
- "macro redefined",
- "A macro is defined in more than one place.", 0, 0
+ FK_MACROS, FK_PARAMS, modeFlag,
+ "sefuncon",
+ FLG_SEFUNSPEC,
+ "a parameter with unconstrained side-effects is passed as a sef parameter",
+ "An actual parameter corresponding to a sef parameter involves a call "
+ "to a procedure with no modifies clause that may have a side-effect.",
+ 0, 0
},
+
+ /* 11.3 Controlling Macro Checking */
+
{
- FK_MACROS, FK_UNRECOG, modeFlag,
- "macrounrecog",
- FLG_MACROUNDEF,
- "unrecognized identifier in macro",
- "An unrecognized identifier appears in a macro. If the identifier "
- "is defined before the macro is used, then this is okay.",
- 0, 0
+ FK_MACROS, FK_NONE, plainFlag,
+ "constmacros",
+ FLG_CONSTMACROS,
+ "check all macros without parameter lists as constants",
+ "Every non-parameterized macro (not preceded by /*@notfunction@*/) "
+ "is checked as a constant.",
+ 0, 0
},
{
- FK_MEMORY, FK_DEAD, modeFlag,
- "stackref",
- FLG_RETSTACK,
- "external reference to stack-allocated storage is created",
- "A stack reference is pointed to by an external reference when the "
- "function returns. The stack-allocated storage is destroyed "
- "after the call, leaving a dangling reference.",
- 0, 0
+ FK_MACROS, FK_NONE, plainFlag,
+ "fcnmacros",
+ FLG_FCNMACROS,
+ "check all macros with parameter lists as functions",
+ "Every parameterized macro (not preceded by /*@notfunction@*/) "
+ "is checked as a function.",
+ 0, 0
},
{
- FK_MEMORY, FK_DEAD, modeFlag,
- "usereleased",
- FLG_USERELEASED,
- "storage used after release",
- "Memory is used after it has been released (either by passing "
- "as an only param or assigning to and only global.",
- 0, 0
+ FK_MACROS, FK_NONE, plainSpecialFlag,
+ "allmacros",
+ FLG_ALLMACROS,
+ "sets fcnmacros and constmacros",
+ "All macros (not preceded by /*@notfunction@*/) are checked as functions or "
+ "constants depending on whether or not they have parameter lists.",
+ 0, 0
},
{
- FK_MEMORY, FK_DEAD, modeFlag,
- "strictusereleased",
- FLG_STRICTUSERELEASED,
- "element used after it may have been released",
- "Memory (through fetch) is used after it may have been released "
- "(either by passing as an only param or assigning to and only global.",
- 0, 0
+ FK_MACROS, FK_NONE, plainFlag,
+ "libmacros",
+ FLG_LIBMACROS,
+ "check all macros with declarations in library as functions",
+ "Every macro declared in the load library is checked.",
+ 0, 0
},
{
- FK_MEMORY, FK_DEF, modeFlag,
- "compdef",
- FLG_COMPDEF,
- "parameter, return value or global completely defined",
- "Storage derivable from a parameter, return value or global is "
- "not defined. Use /*@out@*/ to denote passed or returned "
- "storage which need not be defined.",
- 0, 0
+ FK_MACROS, FK_NONE, plainFlag,
+ "specmacros",
+ FLG_SPECMACROS,
+ "check all macros corresponding to specified functions or constants",
+ "Every macro declared a specification file is checked.",
+ 0, 0
},
{
- FK_MEMORY, FK_DEF, modeFlag,
- "compmempass",
- FLG_COMPMEMPASS,
- "actual parameter matches alias kind of formal parameter completely ",
- "Storage derivable from a parameter does not match the alias kind "
- "expected for the formal parameter.",
+ FK_MACROS, FK_NONE, modeFlag,
+ "macromatchname",
+ FLG_MACROMATCHNAME,
+ "macro definition does not match iter or constant declaration",
+ "A iter or constant macro is defined using a different name from the "
+ "one used in the previous syntactic comment",
0, 0
},
{
- FK_MEMORY, FK_DEF, modeFlag,
- "mustdefine",
- FLG_MUSTDEFINE,
- "out storage not defined before return or scope exit",
- "An out parameter or global is not defined before control is transferred.",
+ FK_MACROS, FK_NONE, plainFlag,
+ "nextlinemacros",
+ FLG_MACRONEXTLINE,
+ "the line after a constant or iter declaration must be a macro definition",
+ "A constant or iter declaration is not immediately followed by a macro definition.",
0, 0
},
+
+ /* 11.4 Iterators */
+
{
- FK_MEMORY, FK_DEF, modeFlag,
- "uniondef",
- FLG_UNIONDEF,
- "at least one field of a union must be defined",
- "No field of a union is defined. Generally, one field of a union is "
- "expected to be defined.",
+ FK_ITER, FK_NONE, plainFlag,
+ "iterbalance",
+ FLG_ITERBALANCE,
+ "iter is not balanced with end_<iter>",
+ NULL,
0, 0
},
{
- FK_MEMORY, FK_IMPLICIT, modeFlag,
- "memimp",
- FLG_MEMIMPLICIT,
- "memory errors for unqualified storage",
- NULL, 0, 0
+ FK_ITER, FK_NONE, plainFlag,
+ "iteryield",
+ FLG_ITERYIELD,
+ "iter yield parameter is inappropriate",
+ NULL,
+ 0, 0
},
{
- FK_MEMORY, FK_IMPLICIT, plainFlag,
- "paramimptemp",
- FLG_PARAMIMPTEMP,
- "assume unannotated parameter is temp",
- NULL, 0, 0
+ FK_ITER, FK_NONE, plainFlag,
+ "hasyield",
+ FLG_HASYIELD,
+ "iter declaration has no yield parameters",
+ "An iterator has been declared with no parameters annotated with "
+ "yield. This may be what you want, if the iterator is meant "
+ "to do something a fixed number of times, but returns no "
+ "information to the calling context. Probably, a parameter "
+ "is missing the yield annotation to indicate that it is "
+ "assigned a value in the calling context.",
+ 0, 0
},
+
+ /*
+ ** 12. Naming Conventions
+ */
+
{
- FK_MEMORY, FK_IMPLICIT, specialFlag,
- "allimponly",
- FLG_ALLIMPONLY,
- "sets globimponly, retimponly, structimponly, specglobimponly, "
- "specretimponly and specstructimponly",
- NULL, 0, 0
- },
- {
- FK_MEMORY, FK_IMPLICIT, specialFlag,
- "codeimponly",
- FLG_CODEIMPONLY,
- "sets globimponly, retimponly and structimponly",
- NULL, 0, 0
- },
- {
- FK_MEMORY, FK_IMPLICIT, specialFlag,
- "specimponly",
- FLG_SPECALLIMPONLY,
- "sets specglobimponly, specretimponly and specstructimponly",
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "namechecks",
+ FLG_NAMECHECKS,
+ "controls name checking without changing other settings",
NULL, 0, 0
- },
+ },
+
+ /* 12.1.1 Czech Names */
+
{
- FK_MEMORY, FK_IMPLICIT, plainFlag,
- "globimponly",
- FLG_GLOBIMPONLY,
- "assume unannotated global storage is only",
- NULL, 0, 0
+ FK_NAMES, FK_ABSTRACT, specialFlag,
+ "czech",
+ FLG_CZECH,
+ "czech naming convention (sets accessczech, czechfunctions, czechvars, "
+ "czechconstants, czechenums, and czechmacros)",
+ "Name is not consistent with Czech naming convention.", 0, 0
},
{
- FK_MEMORY, FK_IMPLICIT, plainFlag,
- "retimponly",
- FLG_RETIMPONLY,
- "assume unannotated returned storage is only",
- NULL, 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechfcns",
+ FLG_CZECHFUNCTIONS,
+ "czech naming convention violated in a function or iterator declaration",
+ "Function or iterator name is not consistent with Czech naming convention.", 0, 0
},
{
- FK_MEMORY, FK_IMPLICIT, plainFlag,
- "structimponly",
- FLG_STRUCTIMPONLY,
- "assume unannotated structure field is only",
- NULL, 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechvars",
+ FLG_CZECHVARS,
+ "czech naming convention violated in a variable declaration",
+ "Variable name is not consistent with Czech naming convention.", 0, 0
},
{
- FK_MEMORY, FK_IMPLICIT, plainFlag,
- "specglobimponly",
- FLG_SPECGLOBIMPONLY,
- "assume unannotated global storage is only",
- NULL, 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechmacros",
+ FLG_CZECHMACROS,
+ "czech naming convention violated in an expanded macro name",
+ "Expanded macro name is not consistent with Czech naming convention.", 0, 0
},
{
- FK_MEMORY, FK_IMPLICIT, plainFlag,
- "specretimponly",
- FLG_SPECRETIMPONLY,
- "assume unannotated returned storage is only",
- NULL, 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechconsts",
+ FLG_CZECHCONSTANTS,
+ "czech naming convention violated in a constant declaration",
+ "Constant name is not consistent with Czech naming convention.", 0, 0
},
{
- FK_MEMORY, FK_IMPLICIT, plainFlag,
- "specstructimponly",
- FLG_SPECSTRUCTIMPONLY,
- "assume unannotated structure field is only",
- NULL, 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechtypes",
+ FLG_CZECHTYPES,
+ "czech naming convention violated in a user-defined type definition",
+ "Type name is not consistent with Czech naming convention. Czech type "
+ "names must not use the underscore character.", 0, 0
},
+
+ /* 12.1.2 Slovak Names */
+
{
- FK_MEMORY, FK_ARRAY, modeFlag,
- "deparrays",
- FLG_DEPARRAYS,
- "array elements are dependent storage",
- "When an element is fetched from an array, lclint analysis is "
- "not able to determine if the same element is reused. "
- "If +deparrays, lclint will mark local storage assigned from "
- "array fetches as dependent.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, specialFlag,
+ "slovak",
+ FLG_SLOVAK,
+ "slovak naming convention violated",
+ "Name is not consistent with Slovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_LEAK, modeFlag,
- "compdestroy",
- FLG_COMPDESTROY,
- "all only references derivable from void pointer out only parameter are released",
- "A storage leak due to incomplete deallocation of a structure or deep "
- "pointer is suspected. Unshared storage that is reachable from "
- "a reference that is being deallocated has not yet been deallocated. "
- "LCLint assumes when an object is passed "
- "as an out only void pointer that the outer object will be "
- "deallocated, but the inner objects will not.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "slovakfcns",
+ FLG_SLOVAKFUNCTIONS,
+ "slovak naming convention violated in a function or iterator declaration",
+ "Function or iterator name is not consistent with Slovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_LEAK, modeFlag,
- "strictdestroy",
- FLG_STRICTDESTROY,
- "report complete destruction errors for array elements that "
- "may have been released",
- NULL,
- 0, 0
- },
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "slovakmacros",
+ FLG_SLOVAKMACROS,
+ "slovak naming convention violated in an expanded macro name",
+ "Expanded macro name is not consistent with Slovak naming convention.", 0, 0
+ },
{
- FK_MEMORY, FK_LEAK, modeFlag,
- "mustfree",
- FLG_MUSTFREE,
- "only storage not released before return or scope exit",
- "A memory leak has been detected. Newly-allocated or "
- "only-qualified storage is not released before the last "
- "reference to it is lost.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "slovakvars",
+ FLG_SLOVAKVARS,
+ "slovak naming convention violated in a variable declaration",
+ "Variable name is not consistent with Slovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_NONE, modeFlag,
- "branchstate",
- FLG_BRANCHSTATE,
- "storage has inconsistent states of alternate paths through a branch",
- "The state of a variable is different depending on which branch "
- "is taken. This means no annotation can sensibly be applied "
- "to the storage.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "slovakconsts",
+ FLG_SLOVAKCONSTANTS,
+ "slovak naming convention violated in a constant declaration",
+ "Constant name is not consistent with Slovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_NONE, modeFlag,
- "strictbranchstate",
- FLG_STRICTBRANCHSTATE,
- "storage through array fetch has inconsistent states of alternate "
- "paths through a branch",
- "The state of a variable through an array fetch is different depending "
- "on which branch is taken. This means no annotation can sensibly be applied "
- "to the storage.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "slovaktypes",
+ FLG_SLOVAKTYPES,
+ "slovak naming convention violated in a use-defined type definition",
+ "Type name is not consistent with Slovak naming convention. Slovak type "
+ "names may not include uppercase letters.", 0, 0
},
+
+ /* 12.1.3 Czechoslovak Names */
{
- FK_MEMORY, FK_NONE, specialFlag,
- "memchecks",
- FLG_MEMCHECKS,
- "sets all dynamic memory checking flags (memimplicit, mustfree, mustdefine, "
- "mustnotalias, null, memtrans)",
- NULL, 0, 0
+ FK_NAMES, FK_ABSTRACT, specialFlag,
+ "czechoslovak",
+ FLG_CZECHOSLOVAK,
+ "czech or slovak naming convention violated",
+ "Name is not consistent with either Czech or Slovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_NONE, specialFlag,
- "memtrans",
- FLG_MEMTRANS,
- "memory transfer errors (sets all *trans flags)",
- "Memory is transferred in a way that violates annotations.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechoslovakfcns",
+ FLG_CZECHOSLOVAKFUNCTIONS,
+ "czechoslovak naming convention violated in a function or iterator declaration",
+ "Function name is not consistent with Czechoslovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_NONE, modeFlag,
- "exposetrans",
- FLG_EXPOSETRANS,
- "exposure transfer errors",
- "Exposed storage is transferred to a non-exposed, non-observer reference.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechoslovakmacros",
+ FLG_CZECHOSLOVAKMACROS,
+ "czechoslovak naming convention violated in an expanded macro name",
+ "Expanded macro name is not consistent with Czechoslovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_NONE, modeFlag,
- "observertrans",
- FLG_OBSERVERTRANS,
- "observer transfer errors",
- "Observer storage is transferred to a non-observer reference.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechoslovakvars",
+ FLG_CZECHOSLOVAKVARS,
+ "czechoslovak naming convention violated in a variable declaration",
+ "Variable name is not consistent with Czechoslovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_NONE, modeFlag,
- "dependenttrans",
- FLG_DEPENDENTTRANS,
- "dependent transfer errors",
- "Dependent storage is transferred to a non-dependent reference.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechoslovakconsts",
+ FLG_CZECHOSLOVAKCONSTANTS,
+ "czechoslovak naming convention violated in a constant declaration",
+ "Constant name is not consistent with Czechoslovak naming convention.", 0, 0
},
{
- FK_MEMORY, FK_NONE, modeFlag,
- "newreftrans",
- FLG_NEWREFTRANS,
- "new reference transfer to reference counted reference",
- "A new reference is transferred to a reference counted reference.",
- 0, 0
+ FK_NAMES, FK_ABSTRACT, plainFlag,
+ "czechoslovaktypes",
+ FLG_CZECHOSLOVAKTYPES,
+ "czechoslovak naming convention violated in a user-defined type definition",
+ "Type name is not consistent with Czechoslovak naming convention. Czechoslovak "
+ "type names may not include uppercase letters or the underscore character.", 0, 0
},
+
+ /* 12.2 Namespace Prefixes */
+
{
- FK_MEMORY, FK_NONE, modeFlag,
- "onlytrans",
- FLG_ONLYTRANS,
- "only storage transferred to non-only reference (memory leak)",
- "The only reference to this storage is transferred to another "
- "reference (e.g., by returning it) that does not have the "
- "only annotation. This may lead to a memory leak, since the "
- "new reference is not necessarily released.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "macrovarprefix",
+ FLG_MACROVARPREFIX,
+ "set namespace prefix for variables declared in a macro body",
+ "A variable declared in a macro body does not start with the macrovarprefix.",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "onlyunqglobaltrans",
- FLG_ONLYUNQGLOBALTRANS,
- "only storage transferred to an unqualified global or "
- "static reference (memory leak)",
- "The only reference to this storage is transferred to another "
- "reference that does not have an aliasing annotation. "
- "This may lead to a memory leak, since the "
- "new reference is not necessarily released.",
- 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "macrovarprefixexclude",
+ FLG_MACROVARPREFIXEXCLUDE,
+ "the macrovarprefix may not be used for non-macro variables",
+ "A variable declared outside a macro body starts with the macrovarprefix.",
+ 0, 0
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "ownedtrans",
- FLG_OWNEDTRANS,
- "owned storage transferred to non-owned reference (memory leak)",
- "The owned reference to this storage is transferred to another "
- "reference (e.g., by returning it) that does not have the "
- "owned annotation. This may lead to a memory leak, since the "
- "new reference is not necessarily released.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "tagprefix",
+ FLG_TAGPREFIX,
+ "set namespace prefix for struct, union and enum tags",
+ "A tag identifier does not start with the tagprefix.",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "freshtrans",
- FLG_FRESHTRANS,
- "fresh storage transferred to non-only reference (memory leak)",
- "Fresh storage (newly allocated in this function) is transferred "
- "in a way that the obligation to release storage is not "
- "propagated. Use the /*@only@*/ annotation to indicate "
- "the a return value is the only reference to the returned "
- "storage.",
- 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "tagprefixexclude",
+ FLG_TAGPREFIXEXCLUDE,
+ "the tagprefix may not be used for non-tag identifiers",
+ "An identifier that is not a tag starts with the tagprefix.",
+ 0, 0
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "sharedtrans",
- FLG_SHAREDTRANS,
- "shared storage transferred to non-shared reference",
- "Shared storage is transferred to a non-shared reference. The other "
- "reference may release storage needed by this reference.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "enumprefix",
+ FLG_ENUMPREFIX,
+ "set namespace prefix for enum members",
+ "An enum member does not start with the enumprefix.",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "temptrans",
- FLG_TEMPTRANS,
- "temp storage transferred to non-temporary reference",
- "Temp storage (associated with a formal parameter) is transferred "
- "to a non-temporary reference. The storage may be released "
- "or new aliases created.",
- 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "enumprefixexclude",
+ FLG_ENUMPREFIXEXCLUDE,
+ "the enumprefix may not be used for non-enum member identifiers",
+ "An identifier that is not an enum member starts with the enumprefix.",
+ 0, 0
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "kepttrans",
- FLG_KEPTTRANS,
- "kept storage transferred to non-temporary reference",
- "Kept storage is transferred "
- "to a non-temporary reference. The storage may be released "
- "or new aliases created.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "filestaticprefix",
+ FLG_FILESTATICPREFIX,
+ "set namespace prefix for file static declarations",
+ "A file-static identifier does not start with the filestaticprefix.",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "keeptrans",
- FLG_KEEPTRANS,
- "keep storage transferred inconsistently",
- "Keep storage is transferred inconsistently --- either in a way "
- "that may add a new alias to it, or release it.",
- 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "filestaticprefixexclude",
+ FLG_FILESTATICPREFIXEXCLUDE,
+ "the filestaticprefix may not be used for identifiers that are not file static",
+ "An identifier that is not file static starts with the filestaticprefix.",
+ 0, 0
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "immediatetrans",
- FLG_IMMEDIATETRANS,
- "an immediate address (result of &) is transferred inconsistently",
- "An immediate address (result of & operator) is transferred "
- "inconsistently.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "globalprefix",
+ FLG_GLOBPREFIX,
+ "set namespace prefix for global variables",
+ "A global variable does not start with the globalprefix",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "refcounttrans",
- FLG_REFCOUNTTRANS,
- "reference counted storage is transferred in an inconsistent way",
- "Reference counted storage is transferred in a way that may not "
- "be consistent with the reference count.",
- 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "globalprefixexclude",
+ FLG_GLOBPREFIXEXCLUDE,
+ "the globalprefix may not be used for non-global identifiers",
+ "An identifier that is not a global variable starts with the globalprefix.",
+ 0, 0
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "statictrans",
- FLG_STATICTRANS,
- "static storage is transferred in an inconsistent way",
- "Static storage is transferred in an inconsistent way.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "typeprefix",
+ FLG_TYPEPREFIX,
+ "set namespace prefix for user-defined types",
+ "A user-defined type does not start with the typeprefix",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "unqualifiedtrans",
- FLG_UNKNOWNTRANS,
- "unqualified storage is transferred in an inconsistent way",
- "Unqualified storage is transferred in an inconsistent way.",
- 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "typeprefixexclude",
+ FLG_TYPEPREFIXEXCLUDE,
+ "the typeprefix may not be used for identifiers that are not type names",
+ "An identifier that is not a type name starts with the typeprefix.",
+ 0, 0
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "staticinittrans",
- FLG_STATICINITTRANS,
- "static storage is used as an initial value in an inconsistent way",
- "Static storage is used as an initial value in an inconsistent way.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "externalprefix",
+ FLG_EXTERNALPREFIX,
+ "set namespace prefix for external identifiers",
+ "An external identifier does not start with the externalprefix",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "unqualifiedinittrans",
- FLG_UNKNOWNINITTRANS,
- "unqualified storage is used as an initial value in an inconsistent way",
- "Unqualified storage is used as an initial value in an inconsistent way.",
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "externalprefixexclude",
+ FLG_EXTERNALPREFIXEXCLUDE,
+ "the externalprefix may not be used for non-external identifiers",
+ "An identifier that is not external starts with the externalprefix.",
+ 0, 0
+ } ,
+ {
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "localprefix",
+ FLG_LOCALPREFIX,
+ "set namespace prefix for local variables",
+ "A local variable does not start with the localprefix",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "readonlystrings",
- FLG_READONLYSTRINGS,
- "string literals are read-only (error if one is modified or released)",
- "String literals are read-only (ANSI semantics). An error is reported "
- "if a string literal may be modified or released.",
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "localprefixexclude",
+ FLG_LOCALPREFIXEXCLUDE,
+ "the localprefix may not be used for non-local identifiers",
+ "An identifier that is not a local variable starts with the localprefix.",
+ 0, 0
+ } ,
+ {
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "uncheckedmacroprefix",
+ FLG_UNCHECKEDMACROPREFIX,
+ "set namespace prefix for unchecked macros",
+ "An unchecked macro name does not start with the uncheckedmacroprefix",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_NONE, modeFlag,
- "readonlytrans",
- FLG_READONLYTRANS,
- "report memory transfer errors for initializations to read-only string literals",
- "A read-only string literal is assigned to a non-observer reference.",
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "uncheckedmacroprefixexclude",
+ FLG_UNCHECKEDMACROPREFIXEXCLUDE,
+ "the uncheckmacroprefix may not be used for identifiers that are not "
+ "unchecked macros",
+ "An identifier that is not the name of an unchecked macro "
+ "starts with the uncheckedmacroprefix.",
+ 0, 0
+ } ,
+ {
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "constprefix",
+ FLG_CONSTPREFIX,
+ "set namespace prefix for constants",
+ "A constant does not start with the constantprefix",
0, 0
- },
+ } ,
{
- FK_MEMORY, FK_PARAMS, modeFlag,
- "passunknown",
- FLG_PASSUNKNOWN,
- "passing a value as an un-annotated parameter clears its annotation",
- NULL, 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "constprefixexclude",
+ FLG_CONSTPREFIXEXCLUDE,
+ "the constprefix may not be used for non-constant identifiers",
+ "An identifier that is not a constant starts with the constantprefix.",
+ 0, 0
+ } ,
{
- FK_MODIFIES, FK_SPEC, plainFlag,
- "mods",
- FLG_MODIFIES,
- "unspecified modification of caller-visible state",
- "An externally-visible object is modified by a function, but not "
- "listed in its modifies clause.",
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "iterprefix",
+ FLG_ITERPREFIX,
+ "set namespace prefix for iterators",
+ "An iter does not start with the iterator prefix",
0, 0
- },
+ } ,
{
- FK_MODIFIES, FK_SPEC, modeFlag,
- "mustmod",
- FLG_MUSTMOD,
- "specified modification is not detected",
- "An object listed in the modifies clause is not modified by the "
- "implementation of the function. The modification may not "
- "be detected if it is done through a call to an unspecified "
- "function.",
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "iterprefixexclude",
+ FLG_ITERPREFIXEXCLUDE,
+ "the iterprefix may not be used for non-iter identifiers",
+ "An identifier that is not a iter starts with the iterprefix.",
+ 0, 0
+ } ,
+ {
+ FK_NAMES, FK_PREFIX, idemStringFlag, ARG_STRING,
+ "protoparamprefix",
+ FLG_DECLPARAMPREFIX,
+ "set namespace prefix for parameters in function prototype declarations",
+ "A parameter name in a function prototype declaration does not start with the "
+ "declaration parameter prefix",
0, 0
- },
+ } ,
+
+ /* 12.3 Naming Restrictions */
{
- FK_MODIFIES, FK_MEMORY, plainFlag,
- "modobserver",
- FLG_MODOBSERVER,
- "possible modification of observer storage",
- "Storage declared with observer is possibly modified. Observer "
- "storage may not be modified.",
+ FK_NAMES, FK_ANSI, modeFlag,
+ "isoreserved",
+ FLG_ISORESERVED,
+ "external name conflicts with name reserved for system or standard library",
+ "External name is reserved for system use by ISO C99 standard.",
0, 0
},
{
- FK_MODIFIES, FK_MEMORY, modeFlag,
- "modobserveruncon",
- FLG_MODOBSERVERUNCON,
- "possible modification of observer storage through unconstrained call",
- "Storage declared with observer may be modified through a call to an "
- "unconstrained function.",
+ FK_NAMES, FK_ANSI, modeFlag,
+ "cppnames",
+ FLG_CPPNAMES,
+ "external or internal name is a C++ keyword or reserved word",
+ "External name is a C++ keyword or reserved word. "
+ "This could lead to problems if the "
+ "code is compiled with a C++ compiler.",
0, 0
},
{
- FK_MODIFIES, FK_MEMORY, modeFlag,
- "modinternalstrict",
- FLG_MODINTERNALSTRICT,
- "possible modification of internal storage through function call",
- "A function that modifies internalState is called from a function that "
- "does not list internalState in its modifies clause",
+ FK_NAMES, FK_ANSI, modeFlag,
+ "isoreservedinternal",
+ FLG_ISORESERVEDLOCAL,
+ "internal name conflicts with name reserved for system or standard library",
+ "Internal name is reserved for system in ISO C99 standard (this should not be necessary unless you are worried about C library implementations that violate the standard and use macros).",
0, 0
},
{
- FK_MODIFIES, FK_UNSPEC, modeFlag,
- "modfilesys",
- FLG_MODFILESYSTEM,
- "report undocumented file system modifications (applies to unspecified "
- "functions if modnomods is set)",
- NULL, 0, 0
+ FK_NAMES, FK_ANSI, plainFlag,
+ "distinctexternalnames",
+ FLG_DISTINCTEXTERNALNAMES,
+ "external name is not distinguishable from another external name using "
+ "the number of significant characters",
+ "An external name is not distinguishable from another external name "
+ "using the number of significant characters. According to "
+ "ANSI Standard (3.1), an implementation may only consider the first 6 "
+ "characters significant, and ignore alphabetical case "
+ "distinctions (ISO C99 requires 31). The "
+ "+externalnamelen <n> flag may be used to change the number "
+ "of significant characters, and -externalnamecaseinsensitive to make "
+ "alphabetical case significant in external names.",
+ 0, 0
},
{
- FK_MODIFIES, FK_UNSPEC, specialFlag,
- "modunspec",
- FLG_MODUNSPEC,
- "modification in unspecified functions (sets modnomods, "
- "modglobunspec and modstrictglobsunspec)",
- NULL, 0, 0
+ FK_NAMES, FK_ANSI, specialValueFlag,
+ "externalnamelen",
+ FLG_EXTERNALNAMELEN,
+ "set the number of significant characters in an external name",
+ "Sets the number of significant characters in an external name (default is 6 for old "
+ "ANSI89 limit, C99 requires 31). "
+ "Sets +distinctexternalnames.",
+ 0, 0
},
{
- FK_MODIFIES, FK_UNSPEC, modeFlag,
- "modnomods",
- FLG_MODNOMODS,
- "modification in a function with no modifies clause",
- "An externally-visible object is modified by a function with no "
- "/*@modifies@*/ comment. The /*@modifies ... @*/ control "
- "comment can be used to give a modifies list for an "
- "unspecified function.",
+ FK_NAMES, FK_ANSI, plainSpecialFlag,
+ "externalnamecaseinsensitive",
+ FLG_EXTERNALNAMECASEINSENSITIVE,
+ "alphabetic comparisons for external names are case-insensitive",
+ "Make alphabetic case insignificant in external names. By ANSI89 "
+ "standard, case need not be significant in an external name. "
+ "If +distinctexternalnames is not set, sets "
+ "+distinctexternalnames with unlimited external name length.",
0, 0
},
{
- FK_MODIFIES, FK_UNSPEC, modeFlag,
- "moduncon",
- FLG_MODUNCON,
- "possible modification through a call to an unconstrained function",
- "An unconstrained function is called in a function body where "
- "modifications are checked. Since the unconstrained function "
- "may modify anything, there may be undetected modifications in "
- "the checked function.",
+ FK_NAMES, FK_ANSI, plainFlag,
+ "distinctinternalnames",
+ FLG_DISTINCTINTERNALNAMES,
+ "internal name is not distinguishable from another internal name using "
+ "the number of significant characters",
+ "An internal name is not distinguishable from another internal name "
+ "using the number of significant characters. According to "
+ "ANSI89 Standard (3.1), an implementation may only consider the first 31 "
+ "characters significant (ISO C99 specified 63). The "
+ "+internalnamelen <n> flag changes the number "
+ "of significant characters, -internalnamecaseinsensitive to makes "
+ "alphabetical case significant, and "
+ "+internalnamelookalike to make similar-looking characters "
+ "non-distinct.",
0, 0
},
{
- FK_MODIFIES, FK_UNSPEC, modeFlag,
- "modunconnomods",
- FLG_MODUNCONNOMODS,
- "possible modification through a call to an unconstrained function in "
- "a function with no modifies clause",
- "An unconstrained function is called in a function body where "
- "modifications are checked. Since the unconstrained function "
- "may modify anything, there may be undetected modifications in "
- "the checked function.",
+ FK_NAMES, FK_ANSI, specialValueFlag,
+ "internalnamelen",
+ FLG_INTERNALNAMELEN,
+ "set the number of significant characters in an internal name",
+ "Sets the number of significant characters in an internal name (ANSI89 "
+ "default is 31.) Sets +distinctinternalnames.",
0, 0
},
{
- FK_MODIFIES, FK_GLOBALS, modeFlag,
- "globsimpmodsnothing",
- FLG_GLOBALSIMPMODIFIESNOTHING,
- "functions declared with a globals list but no modifies clause are "
- "assumed to modify nothing",
- "An implicit modifies nothing clause is assumed for a function "
- "declared with a globals list but not modifies clause.",
+ FK_NAMES, FK_ANSI, plainSpecialFlag,
+ "internalnamecaseinsensitive",
+ FLG_INTERNALNAMECASEINSENSITIVE,
+ "set whether case is significant an internal names "
+ "(-internalnamecaseinsensitive means case is significant)" ,
+ "Set whether case is significant an internal names "
+ "(-internalnamecaseinsensitive "
+ "means case is significant). By ANSI89 default, case is not "
+ "significant. If +distinctinternalnames is not set, sets "
+ "+distinctinternalnames with unlimited internal name length.",
0, 0
},
{
- FK_MODIFIES, FK_GLOBALS, modeFlag,
- "modsimpnoglobs",
- FLG_MODIFIESIMPNOGLOBALS,
- "functions declared with a modifies clause but no globals list "
- "are assumed to use no globals",
- "An implicit empty globals list is assumed for a function "
- "declared with a modifies clause but no globals list.",
+ FK_NAMES, FK_ANSI, plainSpecialFlag,
+ "internalnamelookalike",
+ FLG_INTERNALNAMELOOKALIKE,
+ "lookalike characters match in internal names",
+ "Set whether similar looking characters (e.g., \"1\" and \"l\") "
+ "match in internal names.",
0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "namechecks",
- FLG_NAMECHECKS,
- "controls name checking without changing other settings",
- NULL, 0, 0
- },
- {
- FK_NAMES, FK_ABSTRACT, specialFlag,
- "czech",
- FLG_CZECH,
- "czech naming convention (sets accessczech, czechfunctions, czechvars, "
- "czechconstants, czechenums, and czechmacros)",
- "Name is not consistent with Czech naming convention.", 0, 0
- },
- {
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechfcns",
- FLG_CZECHFUNCTIONS,
- "czech naming convention violated in a function or iterator declaration",
- "Function or iterator name is not consistent with Czech naming convention.", 0, 0
- },
- {
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechvars",
- FLG_CZECHVARS,
- "czech naming convention violated in a variable declaration",
- "Variable name is not consistent with Czech naming convention.", 0, 0
- },
+ FK_NAMES, FK_PREFIX, modeFlag,
+ "protoparamname",
+ FLG_DECLPARAMNAME,
+ "a parameter in a function prototype has a name",
+ "A parameter in a function prototype has a name. This is dangerous, "
+ "since a macro definition could be visible here.",
+ 0, 0
+ } ,
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechmacros",
- FLG_CZECHMACROS,
- "czech naming convention violated in an expanded macro name",
- "Expanded macro name is not consistent with Czech naming convention.", 0, 0
- },
+ FK_NAMES, FK_PREFIX, modeFlag,
+ "protoparammatch",
+ FLG_DECLPARAMMATCH,
+ "the name of a parameter in a function prototype and corresponding "
+ "declaration must match (after removing the protoparamprefix",
+ "A parameter in a function definition does not have the same name as "
+ "the corresponding in the declaration of the function after "
+ "removing the protoparamprefix",
+ 0, 0
+ } ,
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechconsts",
- FLG_CZECHCONSTANTS,
- "czech naming convention violated in a constant declaration",
- "Constant name is not consistent with Czech naming convention.", 0, 0
- },
+ FK_NAMES, FK_PREFIX, plainFlag,
+ "protoparamprefixexclude",
+ FLG_DECLPARAMPREFIXEXCLUDE,
+ "the protoparamprefix may not be used for non-declaraction parameter identifiers",
+ "An identifier that is not a parameter name in a function prototype "
+ "starts with the protoparamprefix.",
+ 0, 0
+ } ,
+
+ /*
+ ** 13. Completeness
+ */
+
+ /* 13.1 Unused Declarations */
+
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechtypes",
- FLG_CZECHTYPES,
- "czech naming convention violated in a user-defined type definition",
- "Type name is not consistent with Czech naming convention. Czech type "
- "names must not use the underscore character.", 0, 0
+ FK_USE, FK_COMPLETE, modeFlag,
+ "topuse",
+ FLG_TOPUNUSED,
+ "declaration at top level not used",
+ "An external declaration not used in any source file.", 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, specialFlag,
- "slovak",
- FLG_SLOVAK,
- "slovak naming convention violated",
- "Name is not consistent with Slovak naming convention.", 0, 0
+ FK_USE, FK_EXPORT, modeFlag,
+ "exportlocal",
+ FLG_EXPORTLOCAL,
+ "a declaration is exported but not used outside this module",
+ "A declaration is exported, but not used outside this module. "
+ "Declaration can use static qualifier.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "slovakfcns",
- FLG_SLOVAKFUNCTIONS,
- "slovak naming convention violated in a function or iterator declaration",
- "Function or iterator name is not consistent with Slovak naming convention.", 0, 0
+ FK_USE, FK_EXPORT, modeFlag,
+ "exportheader",
+ FLG_EXPORTHEADER,
+ "a declaration is exported but does not appear in a header file",
+ "A declaration is exported, but does not appear in a header file.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "slovakmacros",
- FLG_SLOVAKMACROS,
- "slovak naming convention violated in an expanded macro name",
- "Expanded macro name is not consistent with Slovak naming convention.", 0, 0
+ FK_USE, FK_EXPORT, modeFlag,
+ "exportheadervar",
+ FLG_EXPORTHEADERVAR,
+ "a variable declaration is exported but does not appear in a header file",
+ "A variable declaration is exported, but does not appear in a header "
+ "file. (Used with exportheader.)",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "slovakvars",
- FLG_SLOVAKVARS,
- "slovak naming convention violated in a variable declaration",
- "Variable name is not consistent with Slovak naming convention.", 0, 0
+ FK_USE, FK_NONE, modeFlag,
+ "fielduse",
+ FLG_FIELDUNUSED,
+ "field of structure type not used",
+ "A field is present in a structure type but never used. Use /*@unused@*/ in front of field declaration to suppress message.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "slovakconsts",
- FLG_SLOVAKCONSTANTS,
- "slovak naming convention violated in a constant declaration",
- "Constant name is not consistent with Slovak naming convention.", 0, 0
+ FK_USE, FK_NONE, modeFlag,
+ "enummemuse",
+ FLG_ENUMMEMUNUSED,
+ "member of an enum type not used",
+ "A member of an enum type is never used.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "slovaktypes",
- FLG_SLOVAKTYPES,
- "slovak naming convention violated in a use-defined type definition",
- "Type name is not consistent with Slovak naming convention. Slovak type "
- "names may not include uppercase letters.", 0, 0
+ FK_USE, FK_NONE, modeFlag,
+ "constuse",
+ FLG_CONSTUNUSED,
+ "constant declared but not used",
+ "A constant is declared but not used. Use unused in the constant declaration to suppress message.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, specialFlag,
- "czechoslovak",
- FLG_CZECHOSLOVAK,
- "czech or slovak naming convention violated",
- "Name is not consistent with either Czech or Slovak naming convention.", 0, 0
+ FK_USE, FK_NONE, modeFlag,
+ "fcnuse",
+ FLG_FUNCUNUSED,
+ "function declared but not used",
+ "A function is declared but not used. Use /*@unused@*/ in front of function header to suppress message.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechoslovakfcns",
- FLG_CZECHOSLOVAKFUNCTIONS,
- "czechoslovak naming convention violated in a function or iterator declaration",
- "Function name is not consistent with Czechoslovak naming convention.", 0, 0
+ FK_USE, FK_PARAMS, modeFlag,
+ "paramuse",
+ FLG_PARAMUNUSED,
+ "function parameter not used ",
+ "A function parameter is not used in the body of the function. If the argument is needed for type compatibility or future plans, use /*@unused@*/ in the argument declaration.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechoslovakmacros",
- FLG_CZECHOSLOVAKMACROS,
- "czechoslovak naming convention violated in an expanded macro name",
- "Expanded macro name is not consistent with Czechoslovak naming convention.", 0, 0
+ FK_USE, FK_TYPE, modeFlag,
+ "typeuse",
+ FLG_TYPEUNUSED,
+ "type declared but not used",
+ "A type is declared but not used. Use /*@unused@*/ in front of typedef to suppress messages.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechoslovakvars",
- FLG_CZECHOSLOVAKVARS,
- "czechoslovak naming convention violated in a variable declaration",
- "Variable name is not consistent with Czechoslovak naming convention.", 0, 0
+ FK_USE, FK_NONE, modeFlag,
+ "varuse",
+ FLG_VARUNUSED,
+ "variable declared but not used",
+ "A variable is declared but never used. Use /*@unused@*/ in front "
+ "of declaration to suppress message.",
+ 0, 0
},
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechoslovakconsts",
- FLG_CZECHOSLOVAKCONSTANTS,
- "czechoslovak naming convention violated in a constant declaration",
- "Constant name is not consistent with Czechoslovak naming convention.", 0, 0
- },
+ FK_USE, FK_COMPLETE, modeFlag,
+ "unusedspecial",
+ FLG_UNUSEDSPECIAL,
+ "unused declaration in special file (corresponding to .l or .y file)",
+ NULL, 0, 0
+ } ,
+
+ /* 13.2 Complete Programs */
+
{
- FK_NAMES, FK_ABSTRACT, plainFlag,
- "czechoslovaktypes",
- FLG_CZECHOSLOVAKTYPES,
- "czechoslovak naming convention violated in a user-defined type definition",
- "Type name is not consistent with Czechoslovak naming convention. Czechoslovak "
- "type names may not include uppercase letters or the underscore character.", 0, 0
+ FK_COMPLETE, FK_NONE, modeFlag,
+ "declundef",
+ FLG_DECLUNDEF,
+ "function or variable declared but never defined",
+ "A function or variable is declared, but not defined in any source code file.",
+ 0, 0
},
{
- FK_NAMES, FK_ANSI, modeFlag,
- "ansireserved",
- FLG_ANSIRESERVED,
- "external name conflicts with name reserved for system or standard library",
- "External name is reserved for system in ANSI standard.",
+ FK_COMPLETE, FK_SPEC, modeFlag,
+ "specundef",
+ FLG_SPECUNDEF,
+ "function or variable specified but never defined",
+ "A function or variable is declared in an .lcl file, but not defined in any source code file.",
0, 0
},
{
- FK_NAMES, FK_ANSI, modeFlag,
- "cppnames",
- FLG_CPPNAMES,
- "external or internal name is a C++ keyword or reserved word",
- "External name is a C++ keyword or reserved word. "
- "This could lead to problems if the "
- "code is compiled with a C++ compiler.",
+ FK_COMPLETE, FK_SPEC, plainFlag,
+ "specundecl",
+ FLG_SPECUNDECL,
+ "function or variable specified but never declared in a source file",
+ "A function or variable is declared in an .lcl file, but not declared "
+ "in any source code file.",
0, 0
},
{
- FK_NAMES, FK_ANSI, modeFlag,
- "ansireservedinternal",
- FLG_ANSIRESERVEDLOCAL,
- "internal name conflicts with name reserved for system or standard library",
- "Internal name is reserved for system in ANSI standard (this should not be necessary unless you are worried about C library implementations that violate the standard and use macros).",
+ FK_DECL, FK_LIBS, plainFlag,
+ "newdecl",
+ FLG_NEWDECL,
+ "report new global declarations in source files",
+ "There is a new declaration that is not declared in a loaded library "
+ "or earlier file. (Use this flag to check for consistency "
+ "against a library.)",
0, 0
},
{
- FK_NAMES, FK_ANSI, plainFlag,
- "distinctexternalnames",
- FLG_DISTINCTEXTERNALNAMES,
- "external name is not distinguishable from another external name using "
- "the number of significant characters",
- "An external name is not distinguishable from another external name "
- "using the number of significant characters. According to "
- "ANSI Standard (3.1), an implementation may only consider the first 6 "
- "characters significant, and ignore alphabetical case "
- "distinctions. The "
- "+externalnamelen <n> flag may be used to change the number "
- "of significant characters, and -externalnamecaseinsensitive to make "
- "alphabetical case significant in external names.",
+ FK_INIT, FK_SPEC, plainFlag,
+ "needspec",
+ FLG_NEEDSPEC,
+ "information in specifications is not also included in syntactic comments",
+ "There is information in the specification that is not duplicated "
+ "in syntactic comments. Normally, this is not an "
+ "error, but it may be useful to detect it to make "
+ "sure checking incomplete systems without the specifications will "
+ "still use this information.",
0, 0
},
+
+ /*
+ ** 14. Libraries and Header File Inclusion
+ */
+
+ /* 14.1 Standard Libraries */
+
{
- FK_NAMES, FK_ANSI, specialValueFlag,
- "externalnamelen",
- FLG_EXTERNALNAMELEN,
- "set the number of significant characters in an external name",
- "Sets the number of significant characters in an external name (ANSI "
- "default is 6.) Sets +distinctexternalnames.",
- 0, 0
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "nolib",
+ FLG_NOLIB,
+ "do not load standard library",
+ NULL, 0, 0
},
{
- FK_NAMES, FK_ANSI, plainSpecialFlag,
- "externalnamecaseinsensitive",
- FLG_EXTERNALNAMECASEINSENSITIVE,
- "alphabetic comparisons for external names are case-insensitive",
- "Make alphabetic case insignificant in external names. By ANSI "
- "standard, case need not be significant in an external name. "
- "If +distinctexternalnames is not set, sets "
- "+distinctexternalnames with unlimited external name length.",
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "isolib",
+ FLG_ANSILIB,
+ "use normal standard library",
+ "Library based on the ISO standard library specification is used.",
0, 0
},
{
- FK_NAMES, FK_ANSI, plainFlag,
- "distinctinternalnames",
- FLG_DISTINCTINTERNALNAMES,
- "internal name is not distinguishable from another internal name using "
- "the number of significant characters",
- "An internal name is not distinguishable from another internal name "
- "using the number of significant characters. According to "
- "ANSI Standard (3.1), an implementation may only consider the first 31 "
- "characters significant. The "
- "+internalnamelen <n> flag changes the number "
- "of significant characters, -internalnamecaseinsensitive to makes "
- "alphabetical case significant, and "
- "+internalnamelookalike to make similar-looking characters "
- "non-distinct.",
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "strictlib",
+ FLG_STRICTLIB,
+ "interpret standard library strictly",
+ "Stricter version of the standard library is used. (The default "
+ "library is standard.lcd; strict library is strict.lcd.)",
0, 0
},
{
- FK_NAMES, FK_ANSI, specialValueFlag,
- "internalnamelen",
- FLG_INTERNALNAMELEN,
- "set the number of significant characters in an internal name",
- "Sets the number of significant characters in an internal name (ANSI "
- "default is 31.) Sets +distinctinternalnames.",
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "unixlib",
+ FLG_UNIXLIB,
+ "use UNIX (sort-of) standard library",
+ "UNIX version of the standard library is used.",
0, 0
},
{
- FK_NAMES, FK_ANSI, plainSpecialFlag,
- "internalnamecaseinsensitive",
- FLG_INTERNALNAMECASEINSENSITIVE,
- "set whether case is significant an internal names "
- "(-internalnamecaseinsensitive means case is significant)" ,
- "Set whether case is significant an internal names "
- "(-internalnamecaseinsensitive "
- "means case is significant). By ANSI default, case is not "
- "significant. If +distinctinternalnames is not set, sets "
- "+distinctinternalnames with unlimited internal name length.",
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "unixstrictlib",
+ FLG_UNIXSTRICTLIB,
+ "use strict version of UNIX (sort-of) library",
+ "strict version of the UNIX library is used.",
0, 0
},
{
- FK_NAMES, FK_ANSI, plainSpecialFlag,
- "internalnamelookalike",
- FLG_INTERNALNAMELOOKALIKE,
- "lookalike characters match in internal names",
- "Set whether similar looking characters (e.g., \"1\" and \"l\") "
- "match in internal names.",
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "posixlib",
+ FLG_POSIXLIB,
+ "use POSIX standard library",
+ "POSIX version of the standard library is used.",
0, 0
},
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "macrovarprefix",
- FLG_MACROVARPREFIX,
- "set namespace prefix for variables declared in a macro body",
- "A variable declared in a macro body does not start with the macrovarprefix.",
- 0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "macrovarprefixexclude",
- FLG_MACROVARPREFIXEXCLUDE,
- "the macrovarprefix may not be used for non-macro variables",
- "A variable declared outside a macro body starts with the macrovarprefix.",
- 0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "tagprefix",
- FLG_TAGPREFIX,
- "set namespace prefix for struct, union and enum tags",
- "A tag identifier does not start with the tagprefix.",
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "posixstrictlib",
+ FLG_POSIXSTRICTLIB,
+ "use strict POSIX standard library",
+ "POSIX version of the strict standard library is used.",
0, 0
- } ,
+ },
{
- FK_NAMES, FK_PREFIX, plainFlag,
- "tagprefixexclude",
- FLG_TAGPREFIXEXCLUDE,
- "the tagprefix may not be used for non-tag identifiers",
- "An identifier that is not a tag starts with the tagprefix.",
- 0, 0
- } ,
+ FK_LIBS, FK_INIT, idemGlobalFlag,
+ "whichlib",
+ FLG_WHICHLIB,
+ "show standard library filename",
+ NULL, 0, 0
+ },
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "enumprefix",
- FLG_ENUMPREFIX,
- "set namespace prefix for enum members",
- "An enum member does not start with the enumprefix.",
+ FK_LIBS, FK_ANSI, plainFlag,
+ "warnposixheaders",
+ FLG_WARNPOSIX,
+ "a POSIX header is included, but the POSIX library is not used",
+ "Header name matches a POSIX header, but the POSIX library is not selected.",
0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "enumprefixexclude",
- FLG_ENUMPREFIXEXCLUDE,
- "the enumprefix may not be used for non-enum member identifiers",
- "An identifier that is not an enum member starts with the enumprefix.",
- 0, 0
- } ,
+ },
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "filestaticprefix",
- FLG_FILESTATICPREFIX,
- "set namespace prefix for file static declarations",
- "A file-static identifier does not start with the filestaticprefix.",
+ FK_LIBS, FK_ANSI, plainFlag,
+ "warnunixlib",
+ FLG_WARNUNIXLIB,
+ "warn when the unix library is used",
+ "Unix library may not be compatible with all platforms",
0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "filestaticprefixexclude",
- FLG_FILESTATICPREFIXEXCLUDE,
- "the filestaticprefix may not be used for identifiers that are not file static",
- "An identifier that is not file static starts with the filestaticprefix.",
- 0, 0
- } ,
+ },
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "globalprefix",
- FLG_GLOBPREFIX,
- "set namespace prefix for global variables",
- "A global variable does not start with the globalprefix",
+ FK_LIBS, FK_ANSI, plainFlag,
+ "usevarargs",
+ FLG_USEVARARGS,
+ "non-standard <varargs.h> included",
+ "Header <varargs.h> is not part of ANSI Standard. "
+ "Should use <stdarg.h> instead.",
0, 0
- } ,
+ },
{
- FK_NAMES, FK_PREFIX, plainFlag,
- "globalprefixexclude",
- FLG_GLOBPREFIXEXCLUDE,
- "the globalprefix may not be used for non-global identifiers",
- "An identifier that is not a global variable starts with the globalprefix.",
- 0, 0
- } ,
+ FK_HEADERS, FK_FILES, plainFlag,
+ "caseinsensitivefilenames",
+ FLG_CASEINSENSITIVEFILENAMES,
+ "file names are case insensitive (file.h and FILE.H are the same file)",
+ NULL, 0, 0
+ },
+
+ /* 14.2 Generating Libraries */
+
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "typeprefix",
- FLG_TYPEPREFIX,
- "set namespace prefix for user-defined types",
- "A user-defined type does not start with the typeprefix",
- 0, 0
- } ,
+ FK_LIBS, FK_FILES, globalStringFlag, ARG_FILE,
+ "dump",
+ FLG_DUMP,
+ "save state for merging (default suffix .lcd)",
+ NULL, 0, 0
+ },
{
- FK_NAMES, FK_PREFIX, plainFlag,
- "typeprefixexclude",
- FLG_TYPEPREFIXEXCLUDE,
- "the typeprefix may not be used for identifiers that are not type names",
- "An identifier that is not a type name starts with the typeprefix.",
- 0, 0
- } ,
+ FK_LIBS, FK_FILES, globalStringFlag, ARG_FILE,
+ "load",
+ FLG_MERGE,
+ "load state from dump file (default suffix .lcd)",
+ NULL, 0, 0
+ },
+
+ /* 14.3 Header File Inclusion */
+
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "externalprefix",
- FLG_EXTERNALPREFIX,
- "set namespace prefix for external identifiers",
- "An external identifier does not start with the externalprefix",
+ FK_HEADERS, FK_SPEED, globalFlag,
+ "singleinclude",
+ FLG_SINGLEINCLUDE,
+ "optimize header inclusion to eliminate redundant includes",
+ "When checking multiple files, each header file is processed only "
+ "once. This may change the meaning of the code, if the "
+ "same header file is included in different contexts (e.g., the "
+ "header file includes #if directives and the values are "
+ "different when it is included in different places.)",
0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "externalprefixexclude",
- FLG_EXTERNALPREFIXEXCLUDE,
- "the externalprefix may not be used for non-external identifiers",
- "An identifier that is not external starts with the externalprefix.",
- 0, 0
- } ,
+ },
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "localprefix",
- FLG_LOCALPREFIX,
- "set namespace prefix for local variables",
- "A local variable does not start with the localprefix",
+ FK_HEADERS, FK_SPEED, globalFlag,
+ "neverinclude",
+ FLG_NEVERINCLUDE,
+ "optimize header inclusion to not include any header files",
+ "Ignore header includes. Only works if relevant information is "
+ "loaded from a library.",
0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "localprefixexclude",
- FLG_LOCALPREFIXEXCLUDE,
- "the localprefix may not be used for non-local identifiers",
- "An identifier that is not a local variable starts with the localprefix.",
- 0, 0
- } ,
+ },
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "uncheckedmacroprefix",
- FLG_UNCHECKEDMACROPREFIX,
- "set namespace prefix for unchecked macros",
- "An unchecked macro name does not start with the uncheckedmacroprefix",
+ FK_HEADERS, FK_SPEED, globalFlag,
+ "skipsysheaders",
+ FLG_SKIPSYSHEADERS,
+ "do not include header files in system directories (set by -sysdirs)",
+ "Do not include header files in system directories (set by -sysdirs)",
0, 0
- } ,
+ },
+
+ /*
+ ** A. Operation?
+ */
+
+
+ /*
+ ** Syntax
+ */
+
{
- FK_NAMES, FK_PREFIX, plainFlag,
- "uncheckedmacroprefixexclude",
- FLG_UNCHECKEDMACROPREFIXEXCLUDE,
- "the uncheckmacroprefix may not be used for identifiers that are not "
- "unchecked macros",
- "An identifier that is not the name of an unchecked macro "
- "starts with the uncheckedmacroprefix.",
- 0, 0
- } ,
+ FK_SYNTAX, FK_ANSI, plainFlag,
+ "gnuextensions",
+ FLG_GNUEXTENSIONS,
+ "support some gnu (gcc) language extensions",
+ "ANSI C does not allow some language features supported by gcc and other compilers. "
+ "Use +gnuextensions to allow some of these extensions.", 0, 0
+ },
+
+ /* Prototypes */
+
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "constprefix",
- FLG_CONSTPREFIX,
- "set namespace prefix for constants",
- "A constant does not start with the constantprefix",
+ FK_PROTOS, FK_ANSI, modeFlag,
+ "noparams",
+ FLG_NOPARAMS,
+ "function declaration has no parameter list",
+ "A function declaration does not have a parameter list.",
0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "constprefixexclude",
- FLG_CONSTPREFIXEXCLUDE,
- "the constprefix may not be used for non-constant identifiers",
- "An identifier that is not a constant starts with the constantprefix.",
- 0, 0
- } ,
+ },
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "iterprefix",
- FLG_ITERPREFIX,
- "set namespace prefix for iterators",
- "An iter does not start with the iterator prefix",
+ FK_PROTOS, FK_ANSI, modeFlag,
+ "oldstyle",
+ FLG_OLDSTYLE,
+ "old style function definition",
+ "Function definition is in old style syntax. Standard prototype "
+ "syntax is preferred.",
0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "iterprefixexclude",
- FLG_ITERPREFIXEXCLUDE,
- "the iterprefix may not be used for non-iter identifiers",
- "An identifier that is not a iter starts with the iterprefix.",
- 0, 0
- } ,
+ },
+
+
+ /*
+ ** System functions
+ */
+
{
- FK_NAMES, FK_PREFIX, idemStringFlag,
- "protoparamprefix",
- FLG_DECLPARAMPREFIX,
- "set namespace prefix for parameters in function prototype declarations",
- "A parameter name in a function prototype declaration does not start with the "
- "declaration parameter prefix",
+ FK_SYSTEMFUNCTIONS, FK_TYPE, plainFlag,
+ "maintype",
+ FLG_MAINTYPE,
+ "type of main does not match expected type",
+ "The function main does not match the expected type.",
0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, modeFlag,
- "protoparamname",
- FLG_DECLPARAMNAME,
- "a parameter in a function prototype has a name",
- "A parameter in a function prototype has a name. This is dangerous, "
- "since a macro definition could be visible here.",
- 0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, modeFlag,
- "protoparammatch",
- FLG_DECLPARAMMATCH,
- "the name of a parameter in a function prototype and corresponding "
- "declaration must match (after removing the protoparamprefix",
- "A parameter in a function definition does not have the same name as "
- "the corresponding in the declaration of the function after "
- "removing the protoparamprefix",
- 0, 0
- } ,
- {
- FK_NAMES, FK_PREFIX, plainFlag,
- "protoparamprefixexclude",
- FLG_DECLPARAMPREFIXEXCLUDE,
- "the protoparamprefix may not be used for non-declaraction parameter identifiers",
- "An identifier that is not a parameter name in a function prototype "
- "starts with the protoparamprefix.",
- 0, 0
- } ,
+ },
{
- FK_LIMITS, FK_ANSI, modeValueFlag,
- "controlnestdepth",
- FLG_CONTROLNESTDEPTH,
- "set maximum nesting depth of compound statements, iteration control "
- "structures, and selection control structures (ANSI minimum is 15)",
- "Maximum number of control levels exceeded.",
+ FK_SYSTEMFUNCTIONS, FK_BEHAVIOR, modeFlag,
+ "exitarg",
+ FLG_EXITARG,
+ "argument to exit has implementation defined behavior",
+ "The argument to exit should be 0, EXIT_SUCCESS or EXIT_FAILURE",
0, 0
},
+
{
- FK_LIMITS, FK_ANSI, modeValueFlag,
- "stringliterallen",
- FLG_STRINGLITERALLEN,
- "set maximum length of string literals (ANSI minimum is 509)",
- "Maximum length of string literal exceeded.",
+ FK_DECL, FK_NONE, modeFlag,
+ "shadow",
+ FLG_SHADOW,
+ "declaration reuses name visible in outer scope",
+ "An outer declaration is shadowed by the local declaration.",
0, 0
},
{
- FK_LIMITS, FK_ANSI, modeValueFlag,
- "numstructfields",
- FLG_NUMSTRUCTFIELDS,
- "set maximum number of fields in a struct or union (ANSI minimum is 127)",
- "Maximum number of fields in a struct or union exceeded.",
+ FK_DECL, FK_LIBS, modeFlag,
+ "incondefslib",
+ FLG_INCONDEFSLIB,
+ "function, variable or constant defined in a library is redefined with inconsistent type",
+ "A function, variable or constant previously defined in a library is "
+ "redefined with a different type.",
0, 0
},
{
- FK_LIMITS, FK_ANSI, modeValueFlag,
- "numenummembers",
- FLG_NUMENUMMEMBERS,
- "set maximum number of members of an enum (ANSI minimum is 127)",
- "Limit on maximum number of members of an enum is exceeded.",
+ FK_DECL, FK_LIBS, modeFlag,
+ "overload",
+ FLG_WARNOVERLOAD,
+ "library function overloaded",
+ "A function, variable or constant defined in the library is redefined "
+ "with a different type.",
0, 0
},
{
- FK_LIMITS, FK_ANSI, modeValueFlag,
- "includenest",
- FLG_INCLUDENEST,
- "set maximum number of nested #include files",
- "Maximum number of nested #include files exceeded.",
+ FK_DECL, FK_NONE, modeFlag,
+ "nestedextern",
+ FLG_NESTEDEXTERN,
+ "an extern declaration is inside a function scope",
+ "An extern declaration is used inside a function scope.",
0, 0
- },
+ },
{
- FK_LIMITS, FK_ANSI, specialFlag,
- "ansilimits",
- FLG_ANSILIMITS,
- "check for violations of standard limits (controlnestdepth, "
- "stringliterallen, includenest, numstructfields, numenummembers)",
- NULL,
+ FK_DECL, FK_NONE, modeFlag,
+ "redecl",
+ FLG_REDECL,
+ "function or variable redeclared",
+ "A function or variable is declared in more than one place. This is "
+ "not necessarily a problem, since the declarations are consistent.",
0, 0
- },
+ },
{
- FK_NAMES, FK_NONE, plainFlag,
- "name",
- FLG_NAME,
- NULL,
- "Naming convention is violated.",
+ FK_DECL, FK_NONE, plainFlag,
+ "redef",
+ FLG_REDEF,
+ "function or variable redefined",
+ "A function or variable is redefined. One of the declarations should use extern.",
0, 0
},
{
- FK_NONE, FK_NONE, plainFlag,
- "special",
- FLG_SPECIAL,
- NULL, NULL,
+ FK_DECL, FK_TYPE, modeFlag,
+ "imptype",
+ FLG_IMPTYPE,
+ "variable declaration has unknown (implicitly int) type",
+ "A variable declaration has no explicit type. The type is implicitly int.",
0, 0
},
+
{
- FK_NULL, FK_MEMORY, specialFlag,
- "null",
- FLG_NULL,
- "misuses of null pointer",
- "A possibly null pointer is misused (sets nullderef, nullpass, "
- "nullref, nullassign, and nullstate).",
- 0, 0
+ FK_DIRECT, FK_FILES, globalStringFlag, ARG_DIRECTORY,
+ "tmpdir",
+ FLG_TMPDIR,
+ "set directory for writing temp files",
+ NULL, 0, 0
},
{
- FK_NT, FK_MEMORY, modeFlag,
- "nullterminated",
- FLG_NULLTERMINATED,
- "misuse of nullterminated allocation",
- "A possibly non-nullterminated string/memory is used/referenced as a nullterminated one, ",
- 0, 0
+ FK_DIRECT, FK_FILES, globalStringFlag, ARG_PATH,
+ "larchpath",
+ FLG_LARCHPATH,
+ "set path for searching for library files (overrides LARCH_PATH environment variable)",
+ NULL, 0, 0
},
{
- FK_NT, FK_MEMORY, modeFlag,
- "arrayread",
- FLG_ARRAYREAD,
- "possible out of bounds read",
- "An array or pointer access references memory beyond the array or buffer, ",
- 0, 0
+ FK_DIRECT, FK_FILES, globalStringFlag, ARG_DIRECTORY,
+ "lclimportdir",
+ FLG_LCLIMPORTDIR,
+ "set directory to search for LCL import files (overrides LCLIMPORTDIR)",
+ NULL, 0, 0
},
{
- FK_NT, FK_MEMORY, modeFlag,
- "arraywrite",
- FLG_ARRAYWRITE,
- "possible buffer overflow from an out of bounds write",
- "Memory is set past the end of an array or or after the allocated buffer, ",
- 0, 0
+ FK_DIRECT, FK_FILES, globalStringFlag, ARG_PATH,
+ "sysdirs",
+ FLG_SYSTEMDIRS,
+ "set directories for system files (default /usr/include). Separate "
+ "directories with path separator (colons in Unix, semi-colons in Windows). "
+ "Flag settings propagate to files in a system directory. If "
+ "-sysdirerrors is set, no errors are reported for files in "
+ "system directories.",
+ NULL, 0, 0
},
-
{
- FK_NT, FK_MEMORY, modeFlag,
- "fcnpost",
- FLG_FUNCTIONPOST,
- "Function has the post condition",
- "LCLint has determined that the following statement is true after the function, ",
- 0, 0
+ FK_DIRECT, FK_FILES, plainFlag,
+ "skipisoheaders",
+ FLG_SKIPISOHEADERS,
+ "prevent inclusion of header files in a system directory with "
+ "names that match standard ANSI headers. The symbolic information "
+ "in the standard library is used instead. Flag in effect only "
+ "if a library including the ANSI library is loaded. The ANSI "
+ "headers are: assert, ctype, errno, float, limits, locale, math, "
+ "setjmp, signal, stdarg, stddef, stdio, stdlib, strings, string, "
+ "time, and wchar.",
+ NULL, 0, 0
},
{
- FK_NT, FK_MEMORY, modeFlag,
- "debugfcnconstraint",
- FLG_DEBUGFUNCTIONCONSTRAINT,
- "debuging constraint flas",
- "This flag is for debuging. It causes lclint to perform buffer overflow checking even if the error will be surpressed. Normal buffer overflow checking will only be performed if errors would be printed. ",
- 0, 0
+ FK_DIRECT, FK_FILES, plainFlag,
+ "skipposixheaders",
+ FLG_SKIPPOSIXHEADERS,
+ "prevent inclusion of header files in a system directory with "
+ "names that match standard POSIX headers. The symbolic information "
+ "in the posix library is used instead. The POSIX headers are: "
+ "dirent, fcntl, grp, pwd, termios, sys/stat, sys/times, "
+ "sys/types, sys/utsname, sys/wait, unistd, and utime.",
+ NULL, 0, 0
},
{
- FK_NT, FK_MEMORY, modeFlag,
- "fcnconstraint",
- FLG_FUNCTIONCONSTRAINT,
- "unresolved constraint",
- "LCLint was unable to resolve a constraint at the top of the function. If code is correct consider using explict annotation assertions, ",
- 0, 0
+ FK_DIRECT, FK_SUPPRESS, modeFlag,
+ "sysdirerrors",
+ FLG_SYSTEMDIRERRORS,
+ "report errors in files in system directories (set by -sysdirs)",
+ NULL, 0, 0
},
- /*drl7x added 6/18/01 */
{
- FK_NT, FK_MEMORY, modeFlag,
- "checkpost",
- FLG_CHECKPOST,
- "unable to verify ensures annotation",
- "LCLint was unable to determine that the function satisfies a post condition given in an ensures annotation, ",
- 0, 0
+ FK_DIRECT, FK_MACROS, plainFlag,
+ "sysdirexpandmacros",
+ FLG_SYSTEMDIREXPAND,
+ "expand macros in system directories regardless of other settings, "
+ "except for macros corresponding to names defined in a load library",
+ NULL, 0, 0
},
{
- FK_NT, FK_MEMORY, modeFlag,
- "constraintlocation",
- FLG_CONSTRAINTLOCATION,
- "display full c expression for every constraint generated",
- ", ",
- 0, 0
- },/*drl added flag 4/26/01*/
+ FK_DIRECT, FK_HEADERS, globalExtraArgFlag,
+ "I<directory>",
+ FLG_INCLUDEPATH,
+ "add to C include path",
+ NULL, 0, 0
+ },
{
- FK_NT, FK_MEMORY, modeFlag,
- "implictconstraint",
- FLG_IMPLICTCONSTRAINT,
- "Try to generate implicit constraints for functions",
- ", ",
- 0, 0
+ FK_DIRECT, FK_SPEC, globalExtraArgFlag,
+ "S<directory>",
+ FLG_SPECPATH,
+ "add to spec path",
+ NULL, 0, 0
},
- /*drl7x added 4/29/01 */
{
- FK_NT, FK_MEMORY, modeFlag,
- "orconstraint",
- FLG_ORCONSTRAINT,
- "Use limited OR expressions to resolve constraints",
- ", ",
+ FK_EXPORT, FK_SPEC, specialFlag,
+ "exportany",
+ FLG_EXPORTANY,
+ "variable, function or type exported but not specified",
+ "A variable, function or type is exported, but not specified.",
0, 0
},
-
{
- FK_NT, FK_MEMORY, modeFlag,
- "nullterminated",
- FLG_NULLTERMINATEDWARNING,
- "misuse of nullterminated allocation",
- "WARNING:A user annotated non-nullterminated buffer is used/referenced as a nullterminated one, ",
- 0, 0
+ FK_EXPORT, FK_SPEC, modeFlag,
+ "exportfcn",
+ FLG_EXPORTFCN,
+ "function exported but not specified",
+ "A function is exported, but not specified.", 0, 0
},
{
- FK_NULL, FK_MEMORY, modeFlag,
- "nullderef",
- FLG_NULLDEREF,
- "possible dereferencce of null pointer",
- "A possibly null pointer is dereferenced. Value is "
- "either the result of a function which may return null "
- "(in which case, code should check it is not null), or a "
- "global, parameter or structure field declared with the "
- "null qualifier.",
- 0, 0
+ FK_EXPORT, FK_SPEC, modeFlag,
+ "exportmacro",
+ FLG_EXPORTMACRO,
+ "expanded macro exported but not specified",
+ "A macro is exported, but not specified.", 0, 0
},
{
- FK_TYPE, FK_NONE, modeFlag,
- "fcnderef",
- FLG_FCNDEREF,
- "dereferencce of a function type",
- "A function type is dereferenced. The ANSI standard allows this "
- "because of implicit conversion of function designators, however the "
- "dereference is unnecessary.",
- 0, 0
+ FK_EXPORT, FK_SPEC, modeFlag,
+ "exporttype",
+ FLG_EXPORTTYPE,
+ "type definition exported but not specified",
+ "A type is exported, but not specified.", 0, 0
+ },
+ {
+ FK_EXPORT, FK_SPEC, modeFlag,
+ "exportvar",
+ FLG_EXPORTVAR,
+ "variable exported but not specified",
+ "A variable is exported, but not specified.", 0, 0
+ },
+ {
+ FK_EXPORT, FK_SPEC, modeFlag,
+ "exportconst",
+ FLG_EXPORTCONST,
+ "constant exported but not specified",
+ "A constant is exported, but not specified.", 0, 0
+ },
+ {
+ FK_EXPORT, FK_SPEC, modeFlag,
+ "exportiter",
+ FLG_EXPORTITER,
+ "constant exported but not specified",
+ "A constant is exported, but not specified.", 0, 0
+ },
+
+ {
+ FK_FORMAT, FK_DISPLAY, valueFlag,
+ "linelen",
+ FLG_LINELEN,
+ "set length of messages (number of chars)",
+ NULL, 0, 0
+ },
+ {
+ FK_FORMAT, FK_DISPLAY, valueFlag,
+ "indentspaces",
+ FLG_INDENTSPACES,
+ "set number of spaces to indent sub-messages",
+ NULL, 0, 0
+ },
+ {
+ FK_FORMAT, FK_DISPLAY, valueFlag,
+ "locindentspaces",
+ FLG_LOCINDENTSPACES,
+ "set number of spaces to indent sub-messages that start with file locations",
+ NULL, 0, 0
+ },
+ {
+ FK_FORMAT, FK_DISPLAY, plainFlag,
+ "showdeephistory",
+ FLG_SHOWDEEPHISTORY,
+ "show all available information about storage mentioned in warnings",
+ NULL, 0, 0
+ },
+ {
+ FK_FORMAT, FK_DISPLAY, plainFlag,
+ "showcolumn",
+ FLG_SHOWCOL,
+ "show column number where error is found",
+ NULL, 0, 0
+ },
+ {
+ FK_FORMAT, FK_DISPLAY, plainFlag,
+ "showloadloc",
+ FLG_SHOWLOADLOC,
+ "show location information for load files",
+ NULL, 0, 0
+ },
+ {
+ FK_FORMAT, FK_DISPLAY, globalFileFlag,
+ "csv",
+ FLG_CSV,
+ "produce comma-separated values (CSV) warnings output file",
+ NULL, 0, 0
},
{
- FK_NULL, FK_MEMORY, modeFlag,
- "nullpass",
- FLG_NULLPASS,
- "possibly null pointer passed as formal with no null annotation",
- "A possibly null pointer is passed as a parameter corresponding to "
- "a formal parameter with no /*@null@*/ annotation. If NULL "
- "may be used for this parameter, add a /*@null@*/ annotation "
- "to the function parameter declaration.",
- 0, 0
+ FK_FORMAT, FK_DISPLAY, plainFlag,
+ "csvoverwrite",
+ FLG_CSVOVERWRITE,
+ "overwrite exisiting CVS output file",
+ NULL, 0, 0
},
{
- FK_NULL, FK_MEMORY, modeFlag,
- "nullret",
- FLG_NULLRET,
- "possibly null pointer returned as result with no null annotation",
- "Function returns a possibly null pointer, but is not declared "
- "using /*@null@*/ annotation of result. If function may "
- "return NULL, add /*@null@*/ annotation to the return "
- "value declaration.",
- 0, 0
+ FK_FORMAT, FK_DISPLAY, plainFlag,
+ "parenfileformat",
+ FLG_PARENFILEFORMAT,
+ "show column number where error is found",
+ NULL, 0, 0
},
{
- FK_NULL, FK_MEMORY, modeFlag,
- "nullstate",
- FLG_NULLSTATE,
- "possibly null pointer reachable from a reference with no null annotation",
- "A possibly null pointer is reachable from a parameter or global "
- "variable that is not declared using a /*@null@*/ annotation.",
- 0, 0
+ FK_FORMAT, FK_DISPLAY, plainFlag,
+ "htmlfileformat",
+ FLG_HTMLFILEFORMAT,
+ "show file locations as links",
+ NULL, 0, 0
},
{
- FK_NULL, FK_MEMORY, modeFlag,
- "nullassign",
- FLG_NULLASSIGN,
- "inconsistent assignment or initialization involving null pointer",
- "A reference with no null annotation is assigned or initialized "
- "to NULL. Use /*@null@*/ to declare the reference as "
- "a possibly null pointer.",
- 0, 0
+ FK_FORMAT, FK_NONE, plainFlag,
+ "showfunc",
+ FLG_SHOWFUNC,
+ "show name of function containing error",
+ NULL, 0, 0
},
{
- FK_OPS, FK_BOOL, modeFlag,
- "boolcompare",
- FLG_BOOLCOMPARE,
- "comparison between bools (dangerous because of multiple TRUE values)",
- "Two bool values are compared directly using a C primitive. This "
- "may produce unexpected results since all non-zero values are "
- "considered TRUE, so different TRUE values may not be equal. "
- "The file bool.h (included in lclint/lib) provides bool_equal "
- "for safe bool comparisons.", 0, 0
+ FK_FORMAT, FK_NONE, plainFlag,
+ "showallconjs",
+ FLG_SHOWALLCONJS,
+ "show all possible types",
+ "When a library function is declared with multiple possible type, the "
+ "alternate types are shown only if +showallconjs.",
+ 0, 0
},
{
- FK_OPS, FK_NONE, modeFlag,
- "realcompare",
- FLG_REALCOMPARE,
- "dangerous comparison between reals (dangerous because of inexact "
- "floating point representations)",
- "Two real (float, double, or long double) values are compared "
- "directly using a C primitive. "
- "This may produce unexpected results since floating point "
- "representations are inexact. Instead, compare the difference to "
- "FLT_EPSILON or DBL_EPSILON.",
- 0, 0,
+ FK_LIBS, FK_NONE, plainFlag,
+ "impconj",
+ FLG_IMPCONJ,
+ "make all alternate types implicit (useful for making system libraries",
+ NULL, 0, 0
+ } ,
+ {
+ FK_GLOBAL, FK_ERRORS, globalValueFlag,
+ "expect",
+ FLG_EXPECT,
+ "expect <int> code errors",
+ NULL, 0, 0
},
{
- FK_OPS, FK_POINTER, modeFlag,
- "ptrarith",
- FLG_POINTERARITH,
- "arithmetic involving pointer and integer",
- "Pointer arithmetic using pointer and integer.", 0, 0
+ FK_GLOBAL, FK_ERRORS, globalValueFlag,
+ "lclexpect",
+ FLG_LCLEXPECT,
+ "expect <int> spec errors",
+ NULL, 0, 0
},
{
- FK_OPS, FK_POINTER, modeFlag,
- "nullptrarith",
- FLG_NULLPOINTERARITH,
- "arithmetic involving possibly null pointer and integer",
- "Pointer arithmetic using a possibly null pointer and integer.", 0, 0
+ FK_GLOBAL, FK_USE, idemSpecialFlag,
+ "partial",
+ FLG_PARTIAL,
+ "check as partial system (-specundef, -declundef, -exportlocal, "
+ "don't check macros in headers without corresponding .c files)",
+ NULL, 0, 0
},
+
+ /*
+ ** Appendix D. Specifications
+ */
+
{
- FK_OPS, FK_POINTER, modeFlag,
- "ptrcompare",
- FLG_PTRNUMCOMPARE,
- "comparison between pointer and number",
- "A pointer is compared to a number.", 0, 0
+ FK_HEADERS, FK_SPEC, globalFlag,
+ "lh",
+ FLG_DOLH,
+ "generate .lh files", NULL,
+ 0, 0
},
{
- FK_OPS, FK_TYPE, modeFlag,
- "strictops",
- FLG_STRICTOPS,
- "primitive operation does not type check strictly",
- "A primitive operation does not type check strictly.", 0, 0
+ FK_HEADERS, FK_SPEC, globalFlag,
+ "lcs",
+ FLG_DOLCS,
+ "generate .lcs files", NULL,
+ 0, 0
},
+
+ /*
+ ** 1. Operation
+ */
+
{
- FK_OPS, FK_TYPE, modeFlag,
- "bitwisesigned",
- FLG_BITWISEOPS,
- "a bitwise logical operator does not have unsigned operands",
- "An operand to a bitwise operator is not an unsigned values. This "
- "may have unexpected results depending on the signed "
- "representations.", 0, 0
+ FK_HELP, FK_NONE, plainFlag,
+ "warnflags",
+ FLG_WARNFLAGS,
+ "warn when command line sets flag in abnormal way",
+ "Command line sets flag in abnormal way",
+ 0, 0
},
{
- FK_OPS, FK_TYPE, modeFlag,
- "shiftsigned",
- FLG_SHIFTSIGNED,
- "a shift operator does not have unsigned operands",
- "An operand to a shift operator is not unsigned values. This "
- "may have unexpected results depending on the signed "
- "representations.", 0, 0
+ FK_HELP, FK_NONE, plainFlag,
+ "warnrc",
+ FLG_WARNRC,
+ "warn when there are problems with reading the initialization files",
+ "There was a problem reading an initialization file",
+ 0, 0
},
{
- FK_OPS, FK_BOOL, modeFlag,
- "boolops",
- FLG_BOOLOPS,
- "primitive operation (!, && or ||) does not has a boolean argument",
- "The operand of a boolean operator is not a boolean. Use +ptrnegate "
- "to allow ! to be used on pointers.",
+ FK_HELP, FK_NONE, plainFlag,
+ "badflag",
+ FLG_BADFLAG,
+ "warn about bad command line flags",
+ "A flag is not recognized or used in an incorrect way",
0, 0
},
{
- FK_OPS, FK_POINTER, modeFlag,
- "ptrnegate",
- FLG_PTRNEGATE,
- "allow ! to be used on pointer operand",
- "The operand of ! operator is a pointer.", 0, 0
+ FK_HELP, FK_NONE, plainFlag,
+ "fileextensions",
+ FLG_FILEEXTENSIONS,
+ "warn when command line file does not have a recognized extension",
+ NULL, 0, 0
},
{
- FK_OPS, FK_TYPE, modeFlag,
- "sizeoftype",
- FLG_SIZEOFTYPE,
- "sizeof operator has a type argument",
- "Operand of sizeof operator is a type. (Safer to use expression, "
- "int *x = sizeof (*x); instead of sizeof (int).)",
+ FK_HELP, FK_NONE, globalExtraArgFlag,
+ "help",
+ FLG_HELP,
+ "-help <flags> will describe flags",
+ "Display help",
0, 0
},
{
- FK_OPS, FK_TYPE, plainFlag,
- "sizeofformalarray",
- FLG_SIZEOFFORMALARRAY,
- "sizeof operator has an array formal parameter argument",
- "Operand of a sizeof operator is a function parameter declared as "
- "an array. The value of sizeof will be the size of a pointer to the "
- "element type, not the number of elements in the array.",
+ FK_INIT, FK_FILES, globalFileFlag,
+ "f",
+ FLG_OPTF,
+ "read an options file (default ~/.splintrc not loaded)",
+ "Read an options file (instead of loading default ~/.splintc)",
0, 0
},
{
- FK_DECL, FK_TYPE, plainFlag,
- "fixedformalarray",
- FLG_FIXEDFORMALARRAY,
- "formal parameter of type array is declared with size",
- "A formal parameter is declared as an array with size. The size of the array "
- "is ignored in this context, since the array formal parameter is treated "
- "as a pointer.",
+ FK_INIT, FK_FILES, globalFileFlag,
+ "i",
+ FLG_INIT,
+ "set LCL initilization file",
+ NULL, 0, 0
+ },
+ {
+ FK_INIT, FK_FILES, globalFlag,
+ "nof",
+ FLG_NOF,
+ "do not read options file",
+ "Do not read the default options file (~/.splintrc)",
0, 0
},
{
- FK_DECL, FK_TYPE, plainFlag,
- "incompletetype",
- FLG_INCOMPLETETYPE,
- "formal parameter has an incomplete type",
- "A formal parameter is declared with an incomplete type.",
+ FK_INIT, FK_COMMENTS, charFlag,
+ "commentchar",
+ FLG_COMMENTCHAR,
+ "set marker character for syntactic comments (default is '@')",
+ "Set the marker character for syntactic comments. Comments beginning "
+ "with /*<char> are interpreted by Splint, where <char> is the "
+ "comment marker character.",
0, 0
},
+
+ /*
+ ** Limits
+ */
+
{
- FK_DECL, FK_TYPE, plainFlag,
- "formalarray",
- FLG_FORMALARRAY,
- "formal parameter is an array",
- "A formal parameter is declared as an array. This can be confusing, since "
- "a formal array parameter is treated as a pointer.",
+ FK_LIMITS, FK_ANSI, modeValueFlag,
+ "controlnestdepth",
+ FLG_CONTROLNESTDEPTH,
+ "set maximum nesting depth of compound statements, iteration control "
+ "structures, and selection control structures (ANSI89 minimum is 15; ISO99 is 63)",
+ "Maximum number of control levels exceeded.",
0, 0
},
{
- FK_PRED, FK_BOOL, plainFlag,
- "predassign",
- FLG_PREDASSIGN,
- "condition test (if, while or for) is an assignment",
- "The condition test is an assignment expression. Probably, you mean "
- "to use == instead of =. If an assignment is intended, add an "
- "extra parentheses nesting (e.g., if ((a = b)) ...) to suppress "
- "this message.",
+ FK_LIMITS, FK_ANSI, modeValueFlag,
+ "stringliterallen",
+ FLG_STRINGLITERALLEN,
+ "set maximum length of string literals (ANSI89 minimum is 509; ISO99 is 4095)",
+ "Maximum length of string literal exceeded.",
0, 0
},
{
- FK_PRED, FK_BOOL, specialFlag,
- "predbool",
- FLG_PREDBOOL,
- "type of condition test (if, while or for) not bool (sets predboolint, "
- "predboolptr and predboolothers)",
- "Test expression type is not boolean.", 0, 0
+ FK_LIMITS, FK_ANSI, modeValueFlag,
+ "numstructfields",
+ FLG_NUMSTRUCTFIELDS,
+ "set maximum number of fields in a struct or union (ANSI89 minimum is 127; ISO99 is 1023)",
+ "Maximum number of fields in a struct or union exceeded.",
+ 0, 0
},
{
- FK_PRED, FK_BOOL, modeFlag,
- "predboolint",
- FLG_PREDBOOLINT,
- "type of condition test (if, while or for) is an integral type",
- "Test expression type is not boolean or int.", 0, 0
+ FK_LIMITS, FK_ANSI, modeValueFlag,
+ "numenummembers",
+ FLG_NUMENUMMEMBERS,
+ "set maximum number of members of an enum (ANSI89 minimum is 127; ISO99 is 1023)",
+ "Limit on maximum number of members of an enum is exceeded.",
+ 0, 0
},
{
- FK_PRED, FK_BOOL, modeFlag,
- "predboolothers",
- FLG_PREDBOOLOTHERS,
- "type of condition test (if, while or for) not bool, int or pointer",
- "Test expression type is not boolean.", 0, 0
+ FK_LIMITS, FK_ANSI, modeValueFlag,
+ "includenest",
+ FLG_INCLUDENEST,
+ "set maximum number of nested #include files (ANSI89 minimum is 8; ISO99 is 63)",
+ "Maximum number of nested #include files exceeded.",
+ 0, 0
},
{
- FK_PRED, FK_BOOL, modeFlag,
- "predboolptr",
- FLG_PREDBOOLPTR,
- "type of condition test (if, while or for) is a pointer",
- "Test expression type is not boolean.", 0, 0
+ FK_LIMITS, FK_ANSI, specialFlag,
+ "ansi89limits",
+ FLG_ANSI89LIMITS,
+ "check for violations of standard limits (controlnestdepth, "
+ "stringliterallen, includenest, numstructfields, numenummembers) based on ANSI89 standard",
+ NULL,
+ 0, 0
+ },
+ {
+ FK_LIMITS, FK_ANSI, specialFlag,
+ "iso99limits",
+ FLG_ISO99LIMITS,
+ "check for violations of standard limits (controlnestdepth, "
+ "stringliterallen, includenest, numstructfields, numenummembers) based on ISO99 standard",
+ NULL,
+ 0, 0
},
+
{
FK_PREPROC, FK_NONE, globalExtraArgFlag,
"D<initializer>",
NULL, 0, 0
},
{
- FK_GLOBALS, FK_NONE, modeFlag,
- "globstate",
- FLG_GLOBSTATE,
- "returns with global in inconsistent state (null or undefined)",
- "A global variable does not satisfy its annotations when control is transferred.",
+ FK_PREPROC, FK_SYNTAX, plainFlag,
+ "unrecogdirective",
+ FLG_UNRECOGDIRECTIVE,
+ "unrecognized pre-processor directive",
+ "Pre-processor directive is not recognized.",
0, 0
},
{
"syntax",
FLG_SYNTAX,
"syntax error in parsing",
- "Code cannot be parsed. For help on parse errors, see lclint -help parseerrors.",
+ "Code cannot be parsed. For help on parse errors, see splint -help parseerrors.",
0, 0
},
{
"trytorecover",
FLG_TRYTORECOVER,
"try to recover from parse error",
- "Try to recover from parse error. It really means try --- this doesn't usually work.", 0, 0
+ "Try to recover from parse error. It really means try - this doesn't usually work.", 0, 0
},
{
FK_SYNTAX, FK_PREPROC, plainFlag,
"preproc",
FLG_PREPROC,
- NULL, NULL,
+ "preprocessing error",
+ "Preprocessing error.",
0, 0
},
+
{
FK_TYPE, FK_NONE, plainFlag,
"type",
"Types are incompatible.",
0, 0
},
+
{
FK_TYPE, FK_NONE, plainFlag,
- "fullinitblock",
- FLG_FULLINITBLOCK,
- "initializer sets all fields",
- "Initializer does not set every field in the structure.",
+ "stringliteraltoolong",
+ FLG_STRINGLITTOOLONG,
+ "string literal too long for character array",
+ "A string literal is assigned to a char array too small to hold it.",
+ 0, 0
+ },
+ {
+ FK_TYPE, FK_NONE, modeFlag,
+ "stringliteralnoroomfinalnull",
+ FLG_STRINGLITNOROOMFINALNULL,
+ "string literal leaves no room for null terminator",
+ "A string literal is assigned to a char array that is not big enough to hold the final null terminator. This may not be a problem because a null character has been explictedly included in the string literal using an escape sequence",
+ 0, 0
+ },
+ {
+ FK_TYPE, FK_NONE, modeFlag,
+ "stringliteralnoroom",
+ FLG_STRINGLITNOROOM,
+ "string literal leaves no room for null terminator",
+ "A string literal is assigned to a char array that is not big enough to hold the null terminator.",
+ 0, 0
+ },
+ {
+ FK_TYPE, FK_NONE, modeFlag,
+ "stringliteralsmaller",
+ FLG_STRINGLITSMALLER,
+ "string literal is smaller than the char array it is assigned to",
+ "A string literal is assigned to a char array that smaller than the string literal needs.",
0, 0
},
{
0, 0
},
- {
- FK_TYPE, FK_NONE, plainFlag,
- "maintype",
- FLG_MAINTYPE,
- "type of main does not match expected type",
- "The function main does not match the expected type.",
- 0, 0
- },
{
FK_TYPE, FK_NONE, plainFlag,
"formattype",
"To make enum and int types equivalent, use +enumint.",
0, 0
},
+ {
+ FK_TYPEEQ, FK_NONE, modeFlag,
+ "longint",
+ FLG_LONGINT,
+ "long int and int are equivalent",
+ "To make long int and int types equivalent, use +longint.",
+ 0, 0
+ },
+ {
+ FK_TYPEEQ, FK_NONE, modeFlag,
+ "shortint",
+ FLG_SHORTINT,
+ "short int and int are equivalent",
+ "To make short int and int types equivalent, use +shortint.",
+ 0, 0
+ },
{
FK_TYPEEQ, FK_NONE, modeFlag,
"floatdouble",
FK_TYPEEQ, FK_SYNTAX, plainFlag,
"duplicatequals",
FLG_DUPLICATEQUALS,
- "report duplicate type qualifiers (e.g., long long)",
- "Duplicate type qualifiers not supported by ANSI. Some "
- "compilers (e.g., gcc) do support duplicate qualifiers.",
+ "report duplicate type qualifiers (e.g., unsigned unsigned)",
+ "Duplicate type qualifiers not supported by ISO standard.",
0, 0
},
{
FK_TYPEEQ, FK_POINTER, plainFlag,
"zeroptr",
FLG_ZEROPTR,
- "0 is treated as a pointer",
+ "treat 0 as a pointer",
NULL, 0, 0
},
{
FK_TYPEEQ, FK_BOOL, modeFlag,
"zerobool",
FLG_ZEROBOOL,
- "0 is treated as a boolean",
+ "treat 0 as a boolean",
NULL, 0, 0
},
{
FLG_SYSTEMUNRECOG,
"report unrecognized identifiers with system (__) prefix",
"Identifier used in code has not been declared. (Message repeated for "
- "future uses in this file.) Use +gnuextensions to make lclint "
+ "future uses in this file.) Use +gnuextensions to make Splint "
"recognize some keywords that are gnu extensions.",
0, 0
},
"unrecognized identifier",
"Identifier used in code has not been declared.", 0, 0
},
+
+
{
- FK_USE, FK_COMPLETE, modeFlag,
- "topuse",
- FLG_TOPUNUSED,
- "declaration at top level not used",
- "An external declaration not used in any source file.", 0, 0
+ FK_DECL, FK_TYPE, plainFlag,
+ "annotationerror",
+ FLG_ANNOTATIONERROR,
+ "annotation is used in inconsistent location",
+ "A declaration uses an invalid annotation.",
+ 0, 0
+ } ,
+ {
+ FK_DECL, FK_TYPE, plainFlag,
+ "commenterror",
+ FLG_COMMENTERROR,
+ "inconsistent syntactic comment",
+ "A syntactic comment is used inconsistently.",
+ 0, 0
+ } ,
+
+ /*
+ ** Use Warnings
+ */
+
+ {
+ FK_WARNUSE, FK_NONE, plainFlag,
+ "warnuse",
+ FLG_WARNUSE,
+ "warn when declaration marked with warn is used",
+ "Declaration marked with warn clause is used (can be suppresed by more specific flags).",
+ 0, 0
},
{
- FK_USE, FK_EXPORT, modeFlag,
- "exportlocal",
- FLG_EXPORTLOCAL,
- "a declaration is exported but not used outside this module",
- "A declaration is exported, but not used outside this module. "
- "Declaration can use static qualifier.",
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "bufferoverflow",
+ FLG_BUFFEROVERFLOW,
+ "possible buffer overflow vulnerability",
+ "Use of function that may lead to buffer overflow.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "bufferoverflowhigh",
+ FLG_BUFFEROVERFLOWHIGH,
+ "likely buffer overflow vulnerability",
+ "Use of function that may lead to buffer overflow.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "implementationoptional",
+ FLG_IMPLEMENTATIONOPTIONAL,
+ "declarator is implementation optional (ISO99 does not require an implementation to provide it)",
+ "Use of a declarator that is implementation optional, not required by ISO99.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_NONE, modeFlag,
+ "legacy",
+ FLG_LEGACY,
+ "legacy declaration in Unix Standard",
+ "Use of a declarator that is marked as a legacy entry in the Unix Standard.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "multithreaded",
+ FLG_MULTITHREADED,
+ "function is not reentrant",
+ "Non-reentrant function should not be used in multithreaded code.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "portability",
+ FLG_PORTABILITY,
+ "function may have undefined behavior",
+ "Use of function that may have implementation-dependent behavior.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "superuser",
+ FLG_SUPERUSER,
+ "function is restricted to superusers",
+ "Call to function restricted to superusers.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "toctou",
+ FLG_TOCTOU,
+ "possible time of check, time of use vulnerability",
+ "Possible time of check, time of use vulnerability.",
+ 0, 0
+ },
+ {
+ FK_WARNUSE, FK_SECURITY, modeFlag,
+ "unixstandard",
+ FLG_UNIXSTANDARD,
+ "function is not required in Standard UNIX Specification",
+ "Use of function that need not be provided by UNIX implementations",
+ 0, 0
+ },
+
+ /*
+ ** ITS4 Compability Flags
+ **
+ ** These flags flags must appear in order (most severe -> weakest).
+ */
+
+ {
+ FK_ITS4, FK_SECURITY, specialFlag,
+ "its4mostrisky",
+ FLG_ITS4MOSTRISKY,
+ "most risky security vulnerabilities (from its4 database)",
+ "Security vulnerability classified as most risky in its4 database.",
+ 0, 0
+ },
+ {
+ FK_ITS4, FK_SECURITY, specialFlag,
+ "its4veryrisky",
+ FLG_ITS4VERYRISKY,
+ "very risky security vulnerabilities (from its4 database)",
+ "Security vulnerability classified as very risky in its4 database.",
+ 0, 0
+ },
+ {
+ FK_ITS4, FK_SECURITY, specialFlag,
+ "its4risky",
+ FLG_ITS4RISKY,
+ "risky security vulnerabilities (from its4 database)",
+ "Security vulnerability classified as risky in its4 database.",
+ 0, 0
+ },
+ {
+ FK_ITS4, FK_SECURITY, specialFlag,
+ "its4moderate",
+ FLG_ITS4MODERATERISK,
+ "moderately risky security vulnerabilities (from its4 database)",
+ "Security vulnerability classified as moderate risk in its4 database.",
+ 0, 0
+ },
+ {
+ FK_ITS4, FK_SECURITY, specialFlag,
+ "its4low",
+ FLG_ITS4LOWRISK,
+ "risky security vulnerabilities (from its4 database)",
+ "Security vulnerability classified as risky in its4 database.",
+ 0, 0
+ },
+
+ /*
+ ** Syntactic comments
+ */
+
+ {
+ FK_SYNCOMMENTS, FK_SUPPRESS, plainFlag,
+ "nocomments",
+ FLG_NOCOMMENTS,
+ "ignore all stylized comments",
+ NULL, 0, 0
+ },
+ {
+ FK_SYNCOMMENTS, FK_ABSTRACT, plainFlag,
+ "noaccess",
+ FLG_NOACCESS,
+ "ignore access comments",
+ NULL, 0, 0
+ },
+
+ {
+ FK_SYNCOMMENTS, FK_SYNTAX, plainFlag,
+ "unrecogcomments",
+ FLG_UNRECOGCOMMENTS,
+ "stylized comment is unrecognized",
+ "Word after a stylized comment marker does not correspond to a "
+ "stylized comment.",
+ 0, 0
+ },
+ {
+ FK_SYNCOMMENTS, FK_SYNTAX, plainFlag,
+ "unrecogflagcomments",
+ FLG_UNRECOGFLAGCOMMENTS,
+ "stylized flag comment uses an unrecognized flag",
+ "Semantic comment attempts to set a flag that is not recognized.",
+ 0, 0
+ },
+ {
+ FK_SYNCOMMENTS, FK_SUPPRESS, modeFlag,
+ "tmpcomments",
+ FLG_TMPCOMMENTS,
+ "interpret t comments (ignore errors in lines marked with /*@t<n>@*/",
+ NULL, 0, 0
+ },
+ {
+ FK_SYNCOMMENTS, FK_SUPPRESS, plainFlag,
+ "lintcomments",
+ FLG_LINTCOMMENTS,
+ "interpret traditional lint comments (/*FALLTHROUGH*/, /*NOTREACHED*/)",
+ NULL, 0, 0
+ },
+ {
+ FK_SYNCOMMENTS, FK_SUPPRESS, modeFlag,
+ "warnlintcomments",
+ FLG_WARNLINTCOMMENTS,
+ "warn when a traditional lint comment is used",
+ "A traditional lint comment is used. Some traditional lint comments "
+ "are interpreted by Splint to enable easier checking of legacy "
+ "code. It is preferable to replace these comments with the "
+ "suggested Splint alternative.",
+ 0, 0
+ },
+
+ /*
+ ** Comments
+ */
+
+ {
+ FK_COMMENTS, FK_SYNTAX, plainFlag,
+ "continuecomment",
+ FLG_CONTINUECOMMENT,
+ "line continuation marker (\\) in comment before */ on same line",
+ "A line continuation marker (\\) appears inside a comment on the same "
+ "line as the comment close. Preprocessors should handle this "
+ "correctly, but it causes problems for some preprocessors.",
+ 0, 0
+ },
+ {
+ FK_COMMENTS, FK_SYNTAX, plainFlag,
+ "slashslashcomment",
+ FLG_SLASHSLASHCOMMENT,
+ "use of // comment",
+ "A // comment is used. ISO C99 allows // comments, but earlier standards did not.",
0, 0
},
{
- FK_USE, FK_EXPORT, modeFlag,
- "exportheader",
- FLG_EXPORTHEADER,
- "a declaration is exported but does not appear in a header file",
- "A declaration is exported, but does not appear in a header file.",
+ FK_COMMENTS, FK_SYNTAX, plainFlag,
+ "nestcomment",
+ FLG_NESTCOMMENT,
+ "comment begins inside comment",
+ "A comment open sequence (/*) appears within a comment. This usually "
+ "means an earlier comment was not closed.",
0, 0
},
+
+ /*
+ ** Flags for controlling warning message printing.
+ */
+
+ /* Display */
+
{
- FK_USE, FK_EXPORT, modeFlag,
- "exportheadervar",
- FLG_EXPORTHEADERVAR,
- "a variable declaration is exported but does not appear in a header file",
- "A variable declaration is exported, but does not appear in a header "
- "file. (Used with exportheader.)",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, plainFlag,
+ "quiet",
+ FLG_QUIET,
+ "suppress herald and error count",
+ NULL, 0, 0
},
+
+ /*
+ ** Default is to send messages, warnings and errors to stderr
+ */
+
{
- FK_USE, FK_NONE, modeFlag,
- "fielduse",
- FLG_FIELDUNUSED,
- "field of structure type not used",
- "A field is present in a structure type but never used. Use /*@unused@*/ in front of field declaration to suppress message.",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, idemGlobalFlag,
+ "messagestreamstdout",
+ FLG_MESSAGESTREAMSTDOUT,
+ "send status messages to standard output stream",
+ NULL, 0, 0
},
{
- FK_USE, FK_NONE, modeFlag,
- "enummemuse",
- FLG_ENUMMEMUNUSED,
- "member of an enum type not used",
- "A member of an enum type is never used.",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, idemGlobalFlag,
+ "messagestreamstderr",
+ FLG_MESSAGESTREAMSTDERR,
+ "send status messages to standard error stream",
+ NULL, 0, 0
},
{
- FK_USE, FK_NONE, modeFlag,
- "constuse",
- FLG_CONSTUNUSED,
- "constant declared but not used",
- "A constant is declared but not used. Use unused in the constant declaration to suppress message.",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, globalStringFlag, ARG_FILE,
+ "messagestream",
+ FLG_MESSAGESTREAM,
+ "send status messages to <file>",
+ NULL, 0, 0
},
+
{
- FK_USE, FK_NONE, modeFlag,
- "fcnuse",
- FLG_FUNCUNUSED,
- "function declared but not used",
- "A function is declared but not used. Use /*@unused@*/ in front of function header to suppress message.",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, idemGlobalFlag,
+ "warningstreamstdout",
+ FLG_WARNINGSTREAMSTDOUT,
+ "send warnings to standard output stream",
+ NULL, 0, 0
},
{
- FK_USE, FK_PARAMS, modeFlag,
- "paramuse",
- FLG_PARAMUNUSED,
- "function parameter not used ",
- "A function parameter is not used in the body of the function. If the argument is needed for type compatibility or future plans, use /*@unused@*/ in the argument declaration.",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, idemGlobalFlag,
+ "warningstreamstderr",
+ FLG_WARNINGSTREAMSTDERR,
+ "send warnings to standard error stream",
+ NULL, 0, 0
},
{
- FK_USE, FK_TYPE, modeFlag,
- "typeuse",
- FLG_TYPEUNUSED,
- "type declared but not used",
- "A type is declared but not used. Use /*@unused@*/ in front of typedef to suppress messages.",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, globalStringFlag, ARG_FILE,
+ "warningstream",
+ FLG_WARNINGSTREAM,
+ "send warnings to <file>",
+ NULL, 0, 0
},
+
{
- FK_USE, FK_NONE, modeFlag,
- "varuse",
- FLG_VARUNUSED,
- "variable declared but not used",
- "A variable is declared but never used. Use /*@unused@*/ in front "
- "of declaration to suppress message.",
- 0, 0
+ FK_DISPLAY, FK_ERRORS, idemGlobalFlag,
+ "errorstreamstdout",
+ FLG_ERRORSTREAMSTDOUT,
+ "send fatal errors to standard output stream",
+ NULL, 0, 0
},
{
- FK_USE, FK_COMPLETE, modeFlag,
- "unusedspecial",
- FLG_UNUSEDSPECIAL,
- "unused declaration in special file (corresponding to .l or .y file)",
+ FK_DISPLAY, FK_ERRORS, idemGlobalFlag,
+ "errorstreamstderr",
+ FLG_ERRORSTREAMSTDERR,
+ "send fatal errors to standard error stream",
NULL, 0, 0
- } ,
+ },
{
- FK_DECL, FK_TYPE, modeFlag,
- "redundantsharequal",
- FLG_REDUNDANTSHAREQUAL,
- "declaration uses observer qualifier that is always true",
- "A declaration of an immutable object uses a redundant observer qualifier.",
- 0, 0
- } ,
+ FK_DISPLAY, FK_ERRORS, globalStringFlag, ARG_FILE,
+ "errorstream",
+ FLG_ERRORSTREAM,
+ "send fatal errors to <file>",
+ NULL, 0, 0
+ },
+
{
- FK_DECL, FK_TYPE, modeFlag,
- "misplacedsharequal",
- FLG_MISPLACEDSHAREQUAL,
- "declaration of unsharable storage uses sharing annotation",
- "A declaration of an unsharable object uses a sharing annotation.",
- 0, 0
- } ,
+ FK_DISPLAY, FK_ERRORS, globalFlag,
+ "streamoverwrite",
+ FLG_STREAMOVERWRITE,
+ "warn and exit if a stream output file would overwrite an existing file",
+ NULL, 0, 0
+ },
+
+
{
- FK_DECL, FK_TYPE, plainFlag,
- "annotationerror",
- FLG_ANNOTATIONERROR,
- "annotation is used in inconsistent location",
- "A declaration uses an invalid annotation.",
- 0, 0
- } ,
+ FK_DISPLAY, FK_ERRORS, plainFlag,
+ "showsummary",
+ FLG_SHOWSUMMARY,
+ "show summary of all errors reported and suppressed",
+ NULL, 0, 0
+ },
{
- FK_DECL, FK_TYPE, plainFlag,
- "commenterror",
- FLG_COMMENTERROR,
- "inconsistent syntactic comment",
- "A syntactic comment is used inconsistently.",
- 0, 0
- } ,
+ FK_DISPLAY, FK_FILES, plainFlag,
+ "showscan",
+ FLG_SHOWSCAN,
+ "show file names are they are processed",
+ NULL, 0, 0
+ },
{
- FK_DEBUG, FK_NONE, debugFlag,
- "showsourceloc",
- FLG_SHOWSOURCELOC,
- NULL, NULL,
- 0, 0
+ FK_DISPLAY, FK_FILES, plainFlag,
+ "warnsysfiles",
+ FLG_WARNSYSFILES,
+ "Splint has been run on a system file, by default no errors are reported for system files. Use +systemdirerrors if you want splint to report errors in system files. A file is considered a system file if it is in a system directory or a subdirectory of a system directory. The sysdirs flag can be used to control the directories treated as system directories.",
+ NULL, 0, 0
},
{
- FK_DEBUG, FK_NONE, valueFlag,
- "bugslimit",
- FLG_BUGSLIMIT,
- "set maximum number of bugs detected before giving up",
+ FK_DISPLAY, FK_NONE, globalFlag,
+ "stats",
+ FLG_STATS,
+ "display lines processed and time",
NULL, 0, 0
},
-
{
- FK_HELP, FK_NONE, plainFlag,
- "fileextensions",
- FLG_FILEEXTENSIONS,
- "warn when command line file does not have a recognized extension",
+ FK_DISPLAY, FK_NONE, globalFlag,
+ "timedist",
+ FLG_TIMEDIST,
+ "display time distribution",
NULL, 0, 0
},
{
- FK_DECL, FK_NONE, plainFlag,
- "warnuse",
- FLG_WARNUSE,
- "warn when declaration marked with warn is used",
- "Declaration marked with warn clause is used (can be suppresed by more specific flags).",
- 0, 0
+ FK_DISPLAY, FK_USE, globalFlag,
+ "showalluses",
+ FLG_SHOWUSES,
+ "show sorted list of uses of all globals",
+ NULL, 0, 0
},
+
+ /* Hints */
+
{
- FK_MEMORY, FK_DEF, modeFlag,
- "statetransfer",
- FLG_STATETRANSFER,
- "storage has been transfered with invalid state",
- "Transfer violates user-defined state rules.",
+ FK_HINTS, FK_FORMAT, plainFlag,
+ "hints",
+ FLG_HINTS,
+ "provide a hint the first time a particular warning appears",
+ "Provide a hint the first time a particular warning appears",
0, 0
},
{
- FK_MEMORY, FK_DEF, modeFlag,
- "statemerge",
- FLG_STATEMERGE,
- "control paths merge with storage in incompatible states",
- "Control path merge violates user-defined state merge rules.",
+ FK_HINTS, FK_FORMAT, plainFlag,
+ "forcehints",
+ FLG_FORCEHINTS,
+ "provide a hint for every warnings",
+ "Provide a hint for every warning",
0, 0
},
/*
- ** The its4 level flags must appear in order.
+ ** Flags for debugging
*/
{
- FK_WARNUSE, FK_SECURITY, specialFlag,
- "its4mostrisky",
- FLG_ITS4MOSTRISKY,
- "most risky security vulnerabilities (from its4 database)",
- "Security vulnerability classified as most risky in its4 database.",
- 0, 0
+ FK_DEBUG, FK_NONE, valueFlag,
+ "bugslimit",
+ FLG_BUGSLIMIT,
+ "set maximum number of bugs detected before giving up",
+ NULL, 0, 0
},
{
- FK_WARNUSE, FK_SECURITY, specialFlag,
- "its4veryrisky",
- FLG_ITS4VERYRISKY,
- "very risky security vulnerabilities (from its4 database)",
- "Security vulnerability classified as very risky in its4 database.",
+ FK_DEBUG, FK_BOUNDS, plainFlag,
+ "debugfcnconstraint",
+ FLG_DEBUGFUNCTIONCONSTRAINT,
+ "debug function constraints",
+ "Perform buffer overflow checking even if the errors would be surpressed.",
0, 0
},
{
- FK_WARNUSE, FK_SECURITY, specialFlag,
- "its4risky",
- FLG_ITS4RISKY,
- "risky security vulnerabilities (from its4 database)",
- "Security vulnerability classified as risky in its4 database.",
+ FK_DEBUG, FK_NONE, specialDebugFlag,
+ "grammar",
+ FLG_GRAMMAR,
+ "debug parsing", NULL,
0, 0
},
{
- FK_WARNUSE, FK_SECURITY, specialFlag,
- "its4moderate",
- FLG_ITS4MODERATERISK,
- "moderately risky security vulnerabilities (from its4 database)",
- "Security vulnerability classified as moderate risk in its4 database.",
+ FK_DEBUG, FK_NONE, debugFlag,
+ "keep",
+ FLG_KEEP,
+ "do not delete temporary files", NULL,
0, 0
},
{
- FK_WARNUSE, FK_SECURITY, specialFlag,
- "its4low",
- FLG_ITS4LOWRISK,
- "risky security vulnerabilities (from its4 database)",
- "Security vulnerability classified as risky in its4 database.",
+ FK_DEBUG, FK_NONE, debugFlag,
+ "nopp",
+ FLG_NOPP,
+ "do not pre-process input files", NULL,
0, 0
},
-
- /*
- ** End of its4 level flags
- */
-
- {
- FK_WARNUSE, FK_SECURITY, modeFlag,
- "bufferoverflowhigh",
- FLG_BUFFEROVERFLOWHIGH,
- "likely buffer overflow vulnerability",
- "Use of function that may lead to buffer overflow.",
- 0, 0
- },
- {
- FK_WARNUSE, FK_SECURITY, modeFlag,
- "bufferoverflow",
- FLG_BUFFEROVERFLOW,
- "possible buffer overflow vulnerability",
- "Use of function that may lead to buffer overflow.",
- 0, 0
- },
{
- FK_WARNUSE, FK_SECURITY, modeFlag,
- "toctou",
- FLG_TOCTOU,
- "possible time of check, time of use vulnerability",
- "Possible time of check, time of use vulnerability.",
+ FK_DEBUG, FK_NONE, debugFlag,
+ "showsourceloc",
+ FLG_SHOWSOURCELOC,
+ "display the source code location where a warning is produced", NULL,
0, 0
},
-
} ;
+
+
andersk / splint.git / blobdiff