client9 / libinjection

SQL / SQLI tokenizer parser analyzer
Other
1k stars 274 forks source link

clang --analyze errors #85

Closed client9 closed 9 years ago

client9 commented 9 years ago
ip-10-0-1-9:src nickg$ clang --analyze *.c *.h
fptool.c:53:9: warning: Value stored to 'ok' is never read
        ok = libinjection_sqli_check_fingerprint(&sf);
        ^    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
reader.c:137:9: warning: Value stored to 'issqli' is never read
        issqli = 0;
        ^        ~
1 warning generated.
ip-10-0-1-9:src nickg$ clang --analyze -Weverything *.c *.h
fptool.c:53:9: warning: Value stored to 'ok' is never read
        ok = libinjection_sqli_check_fingerprint(&sf);
        ^    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
reader.c:137:9: warning: Value stored to 'issqli' is never read
        issqli = 0;
        ^        ~
1 warning generated.
libinjection_sqli.h:51:12: warning: padding struct 'struct
      libinjection_sqli_token' with 5 bytes to align 'pos' [-Wpadded]
    size_t pos;
           ^
libinjection_sqli.h:39:8: warning: padding size of 'struct
      libinjection_sqli_token' with 4 bytes to alignment boundary [-Wpadded]
struct libinjection_sqli_token {
       ^
2 warnings generated.
In file included from libinjection_sqli_data.h:6:
./libinjection_sqli.h:51:12: warning: padding struct 'struct
      libinjection_sqli_token' with 5 bytes to align 'pos' [-Wpadded]
    size_t pos;
           ^
./libinjection_sqli.h:39:8: warning: padding size of 'struct
      libinjection_sqli_token' with 4 bytes to alignment boundary [-Wpadded]
struct libinjection_sqli_token {
       ^
2 warnings generated.
client9 commented 9 years ago

padding errors are boring. Closing.