util/check-format-test-positives.c

   1 /*
   2  * Copyright 2007-2022 The OpenSSL Project Authors. All Rights Reserved.
   3  * Copyright Siemens AG 2015-2022
   4  *
   5  * Licensed under the Apache License 2.0 (the "License").  You may not use
   6  * this file except in compliance with the License.  You can obtain a copy
   7  * in the file LICENSE in the source distribution or at
   8  * https://www.openssl.org/source/license.html
   9  */
  10
  11 /*
  12  * This demonstrates/tests cases where check-format.pl should report issues.
  13  * Some of the reports are due to sanity checks for proper nesting of comment
  14  * delimiters and parenthesis-like symbols, e.g., on unexpected/unclosed braces.
  15  */
  16
  17 /*
  18  * The '@'s after '*' are used for self-tests: they mark lines containing
  19  * a single flaw that should be reported. Normally it should be reported
  20  * while handling the given line, but in case of delayed checks there is a
  21  * following digit indicating the number of reports expected for this line.
  22  */
  23
  24 /* For each of the following set of lines the tool should complain once */
  25 /*@ tab character:       */
  26 /*@ intra-line carriage return character: \r */
  27 /*@ non-printable ASCII character: \b */
  28 /*@ non-ASCII character: ä */
  29 /*@ whitespace at EOL: */
  30 // /*@ end-of-line comment style not allowed (for C90 compatibility) */
  31  /*@0 intra-line comment indent off by 1, reported unless sloppy-cmt */
  32 /*X */ /*@2 no space nor '*' after comment start, reported unless sloppy-spc */
  33 /* X*/ /*@ no space before comment end , reported unless sloppy-spc */
  34 /*@ comment starting delimiter: /* inside intra-line comment */
  35  /*@0
  36   *@ above multi-line comment start indent off by 1, reported unless sloppy-cmt; this comment line is too long
  37    *@ multi-line comment indent further off by 1 relative to comment start
  38   *@ multi-line comment ending with text on last line */
  39 /*@2 multi-line comment starting with text on first line
  40  *@ comment starting delimiter: /* inside multi-line comment
  41 *@ multi-line comment indent off by -1
  42  *X*@ no spc after leading '*' in multi-line comment, reported unless sloppy-spc
  43  *@0 more than two spaces after .   in comment, reported unless sloppy-spc
  44  *@0 more than two spaces after ?   in comment, reported unless sloppy-spc
  45  *@0 more than two spaces after !   in comment, reported unless sloppy-spc
  46 */ /*@ multi-line comment end indent off by -1 (relative to comment start) */
  47 */ /*@ unexpected comment ending delimiter outside comment */
  48 /*@ comment line is 4 columns tooooooooooooooooo wide, reported unless sloppy-len */
  49 /*@ comment line is 5 columns toooooooooooooooooooooooooooooooooooooooooooooo wide */
  50 #define X   1       /*@0 extra space false negative due to coincidence */
  51  #define Y  2       /*@ indent of preprocessor directive off by 1 (must be 0) */
  52 typedef struct  {   /*@0 extra space in code, reported unless sloppy-spc */
  53     enum {          /*@1 extra space  in comment, reported unless sloppy-spc */
  54            w = 0 /*@ hanging expr indent off by 1, or 3 for lines after '{' */
  55              && 1,  /*@ hanging expr indent off by 3, or -1 for leading '&&' */
  56          x = 1,     /*@ hanging expr indent off by -1 */
  57           y,z       /*@ no space after ',', reported unless sloppy-spc */
  58     } e_member ;    /*@ space before ';', reported unless sloppy-spc */
  59     int v[1;        /*@ unclosed bracket in type declaration */
  60    union {          /*@ statement/type declaration indent off by -1 */
  61         struct{} s; /*@ no space before '{', reported unless sloppy-spc */
  62     }u_member;      /*@ no space after '}', reported unless sloppy-spc */
  63     } s_type;       /*@ statement/type declaration indent off by 4 */
  64 int* somefunc();    /*@ no space before '*' in type decl, r unless sloppy-spc */
  65 void main(int n) {  /*@ opening brace at end of function definition header */
  66     for (;;n++) {   /*@ no space after ';', reported unless sloppy-spc */
  67         return;     /*@0 (1-line) single statement in braces */
  68     }}              /*@2 code after '}' outside expr */
  69 }                   /*@ unexpected closing brace (too many '}') outside expr */
  70 )                   /*@ unexpected closing paren outside expr */
  71 #endif              /*@ unexpected #endif */
  72 int f (int a,       /*@ space after fn before '(', reported unless sloppy-spc */
  73       int b,        /*@ hanging expr indent off by -1 */
  74        long I)      /*@ single-letter name 'I' */
  75 { int               /*@ code after '{' opening a block */
  76     xx = 1) +       /*@ unexpected closing parenthesis */
  77         0L <        /*@ constant on LHS of comparison operator */
  78         a] -        /*@ unexpected closing bracket */
  79         3: *        /*@ unexpected ':' (without preceding '?') within expr */
  80         4};         /*@ unexpected closing brace within expression */
  81     char y[] = {    /*@0 unclosed brace within initializer/enum expression */
  82         1* 1,       /*@ no space etc. before '*', reported unless sloppy-spc */
  83          2,         /*@ hanging expr indent (for lines after '{') off by 1 */
  84         (xx         /*@0 unclosed parenthesis in expression */
  85          ? y        /*@0 unclosed '? (conditional expression) */
  86          [0;        /*@4 unclosed bracket in expression */
  87     /*@ blank line within local decls */
  88    s_type s;        /*@2 local variable declaration indent off by -1 */
  89    t_type t;        /*@ local variable declaration indent again off by -1 */
  90     /* */           /*@0 missing blank line after local decls */
  91    somefunc(a,      /*@2 statement indent off by -1 */
  92           "aligned" /*@ expr indent off by -2 accepted if sloppy-hang */ "right"
  93            , b,     /*@ expr indent off by -1 */
  94            b,       /*@ expr indent as on line above, accepted if sloppy-hang */
  95     b, /*@ expr indent off -8 but @ extra indent accepted if sloppy-hang */
  96    "again aligned" /*@ expr indent off by -9 (left of stmt indent, */ "right",
  97             abc == /*@ .. so reported also with sloppy-hang; this line is too long */ 456
  98 # define MAC(A) (A) /*@ nesting indent of preprocessor directive off by 1 */
  99              ? 1    /*@ hanging expr indent off by 1 */
 100               : 2); /*@ hanging expr indent off by 2, or 1 for leading ':' */
 101     if(a            /*@ no space after 'if', reported unless sloppy-spc */
 102           /*@0 intra-line comment indent off by -1 (not: by 3 due to '&&') */
 103            && ! 0   /*@2 space after '!', reported unless sloppy-spc */
 104          || b ==    /*@ hanging expr indent off by 2, or -2 for leading '||' */
 105        (xx+= 2) +   /*@ no space before '+=', reported unless sloppy-spc */
 106        (a^ 1) +     /*@ no space before '^', reported unless sloppy-spc */
 107        a %2 /       /*@ no space after '%', reported unless sloppy-spc */
 108        1 +/* */     /*@ no space before comment, reported unless sloppy-spc */
 109        /* */+       /*@ no space after comment, reported unless sloppy-spc */
 110        s. e_member) /*@ space after '.', reported unless sloppy-spc */
 111          xx = a + b /*@ extra single-statement indent off by 1 */
 112                + 0; /*@ two times extra single-statement indent off by 3 */
 113     if (a ++)       /*@ space before postfix '++', reported unless sloppy-spc */
 114     {               /*@ {' not on same line as preceding 'if' */
 115         c;          /*@0 single stmt in braces, reported on 1-stmt */
 116     } else          /*@ no '{' on same line after '} else' */
 117       {             /*@ statement indent off by 2 */
 118         d;          /*@0 single stmt in braces, reported on 1-stmt */
 119           }         /*@ statement indent off by 6 */
 120     if (1) f(a,     /*@ (non-brace) code after end of 'if' condition */
 121              b); else /*@ (non-brace) code before 'else' */
 122         do f(c, c); /*@ (non-brace) code after 'do' */
 123         while ( 2); /*@ space after '(', reported unless sloppy-spc */
 124     b; c;           /*@ more than one statement per line */
 125   outer:            /*@ outer label special indent off by 1 */
 126     do{             /*@ no space before '{', reported unless sloppy-spc */
 127      inner:         /*@ inner label normal indent off by 1 */
 128         f (3,       /*@ space after fn before '(', reported unless sloppy-spc */
 129            4);      /*@0 false negative: should report single stmt in braces */
 130     }               /*@0 'while' not on same line as preceding '}' */
 131     while (a+ 0);   /*@2 no space before '+', reported unless sloppy-spc */
 132     switch (b ) {   /*@ space before ')', reported unless sloppy-spc */
 133    case 1:          /*@ 'case' special statement indent off by -1 */
 134     case(2):        /*@ no space after 'case', reported unless sloppy-spc */
 135     default: ;      /*@ code after 'default:' */
 136 }                   /*@ statement indent off by -4 */
 137     return(         /*@ no space after 'return', reported unless sloppy-spc */
 138            x); }    /*@ code before block-level '}' */
 139 /* Here the tool should stop complaining apart from the below issues at EOF */
 140
 141 void f_looong_body()
 142 {
 143     ;
 144     ;
 145     ;
 146     ;
 147     ;
 148     ;
 149     ;
 150     ;
 151     ;
 152     ;
 153     ;
 154     ;
 155     ;
 156     ;
 157     ;
 158     ;
 159     ;
 160     ;
 161     ;
 162     ;
 163     ;
 164     ;
 165     ;
 166     ;
 167     ;
 168     ;
 169     ;
 170     ;
 171     ;
 172     ;
 173     ;
 174     ;
 175     ;
 176     ;
 177     ;
 178     ;
 179     ;
 180     ;
 181     ;
 182     ;
 183     ;
 184     ;
 185     ;
 186     ;
 187     ;
 188     ;
 189     ;
 190     ;
 191     ;
 192     ;
 193     ;
 194     ;
 195     ;
 196     ;
 197     ;
 198     ;
 199     ;
 200     ;
 201     ;
 202     ;
 203     ;
 204     ;
 205     ;
 206     ;
 207     ;
 208     ;
 209     ;
 210     ;
 211     ;
 212     ;
 213     ;
 214     ;
 215     ;
 216     ;
 217     ;
 218     ;
 219     ;
 220     ;
 221     ;
 222     ;
 223     ;
 224     ;
 225     ;
 226     ;
 227     ;
 228     ;
 229     ;
 230     ;
 231     ;
 232     ;
 233     ;
 234     ;
 235     ;
 236     ;
 237     ;
 238     ;
 239     ;
 240     ;
 241     ;
 242     ;
 243     ;
 244     ;
 245     ;
 246     ;
 247     ;
 248     ;
 249     ;
 250     ;
 251     ;
 252     ;
 253     ;
 254     ;
 255     ;
 256     ;
 257     ;
 258     ;
 259     ;
 260     ;
 261     ;
 262     ;
 263     ;
 264     ;
 265     ;
 266     ;
 267     ;
 268     ;
 269     ;
 270     ;
 271     ;
 272     ;
 273     ;
 274     ;
 275     ;
 276     ;
 277     ;
 278     ;
 279     ;
 280     ;
 281     ;
 282     ;
 283     ;
 284     ;
 285     ;
 286     ;
 287     ;
 288     ;
 289     ;
 290     ;
 291     ;
 292     ;
 293     ;
 294     ;
 295     ;
 296     ;
 297     ;
 298     ;
 299     ;
 300     ;
 301     ;
 302     ;
 303     ;
 304     ;
 305     ;
 306     ;
 307     ;
 308     ;
 309     ;
 310     ;
 311     ;
 312     ;
 313     ;
 314     ;
 315     ;
 316     ;
 317     ;
 318     ;
 319     ;
 320     ;
 321     ;
 322     ;
 323     ;
 324     ;
 325     ;
 326     ;
 327     ;
 328     ;
 329     ;
 330     ;
 331     ;
 332     ;
 333     ;
 334     ;
 335     ;
 336     ;
 337     ;
 338     ;
 339     ;
 340     ;
 341
 342
 343     ;               /*@ 2 essentially blank lines before, if !sloppy-spc */
 344 }                   /*@ function body length > 200 lines */
 345 #if 0               /*@0 unclosed #if */
 346 struct t {          /*@0 unclosed brace at decl/block level */
 347     enum {          /*@0 unclosed brace at enum/expression level */
 348           v = (1    /*@0 unclosed parenthesis */
 349                etyp /*@0 blank line follows just before EOF, if !sloppy-spc: */
 350