Opened 10 years ago

Closed 10 years ago

#317 closed Verbesserung (fixed)

Tool CPP Check

Reported by: Melanie Hermann Owned by: Melanie Hermann
Priority: kurzfristig Milestone:
Component: Gesamtsystem Version:
Severity: Aufgabe Keywords:
Cc:

Description

Tool CPP Check über GC-Sourcen laufen lassen.

Attachments (6)

CppCheck_Ergebnis.jpg​ (590.3 KB ) - added by Melanie Hermann 10 years ago.
CppCheck_Ergebnis_2.jpg​ (519.7 KB ) - added by Melanie Hermann 10 years ago.
Cppcheck_Ergebnis_20160212.jpg​ (445.1 KB ) - added by Melanie Hermann 10 years ago.
CppCheck_Ergebnis_20160212_2.jpg​ (303.1 KB ) - added by Melanie Hermann 10 years ago.
CppCheck_Ergebnis_20160215.jpg​ (102.4 KB ) - added by Melanie Hermann 10 years ago.
CppCheck_Ergebnis_20160215_2.jpg​ (41.6 KB ) - added by Melanie Hermann 10 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 by Melanie Hermann, 10 years ago

Änderungen aufgrund CPP Check:

  • InOut.cpp:
    • CheckLeitungsbruch(): Variable fkt zu Beginn definiert.
  • gcDoku.cpp:
    • createMain_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createRightFrame_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createLeftFrame_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createDSfG_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createFehlerliste_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createDSfG_REG_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createModbus_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createModbusTree_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createModbusCoils_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
    • createWebserverArchive_HTML(): Else-Zweig in dem die Variable fname auch definiert wird.
Last edited 10 years ago by Melanie Hermann (previous) (diff)

by Melanie Hermann, 10 years ago

Attachment: CppCheck_Ergebnis.jpg​ added

by Melanie Hermann, 10 years ago

Attachment: CppCheck_Ergebnis_2.jpg​ added

comment:2 by Melanie Hermann, 10 years ago

TODO:
Die weiteren angezeigten Stellen bei Gelegenheit prüfen und ggf. korrigieren.

comment:3 by Melanie Hermann, 10 years ago

Weitere Änderungen aufgrund CPP Check:

  • gcvars.cpp:
    • SetUserArchivModReg(): Aufruf von GetByMbAdr() via gcwert:: und nicht mehr per gcw->.
    • SetTrendBlock(): Aufruf von GetByMbAdr() via gcwert:: und nicht mehr per gcw->.

by Melanie Hermann, 10 years ago

Attachment: Cppcheck_Ergebnis_20160212.jpg​ added

comment:4 by Melanie Hermann, 10 years ago

Weitere Änderungen aufgrund CPP Check:

  • makxref1.cpp:
    • build_AGHs(): fclose(fp) hat gefehlt
  • makxref2.cpp:
    • SortFile(): delete [] sortInfo und delete [] buffer hat gefehlt
  • dsfgtime.cpp:
    • convertUtc(): Die Variable res war nicht definiert.
    • convert(): Die Variable res war nicht definiert.
  • dsfg_localwert.cpp:
    • DSfGGetValueAsISO646_DE(): Die Variable len war nicht definiert.

by Melanie Hermann, 10 years ago

Attachment: CppCheck_Ergebnis_20160212_2.jpg​ added

comment:5 by Melanie Hermann, 10 years ago

Priority: mittelfristig → kurzfristig

comment:6 by Melanie Hermann, 10 years ago

Weitere Änderungen:

  • DHCPServer.cpp:
    • DHCPDestroyItem(): realloc() durch realloc_or_free() ersetzt.
    • DHCPReallocItem(): realloc() durch realloc_or_free() ersetzt.
  • gctab.cpp:
    • buildCoilsTab(): 3x realloc() durch realloc_or_free() ersetzt.
    • readErrTab(): realloc() durch realloc_or_free() ersetzt.
    • realloc_or_free(): Neue Funktion hier in diesem Projekt.
    • FREE_AND_NULL: Neues Define hier in diesem Projekt.
  • mongoose.c:
    • handle_directory_request(): realloc() durch realloc_or_free() ersetzt.
Last edited 10 years ago by Melanie Hermann (previous) (diff)

by Melanie Hermann, 10 years ago

Attachment: CppCheck_Ergebnis_20160215.jpg​ added

comment:7 by Melanie Hermann, 10 years ago

Weitere Änderungen:

  • substitute_index_file(): st definiert.
  • set_acl_option(): fake definiert.

by Melanie Hermann, 10 years ago

Attachment: CppCheck_Ergebnis_20160215_2.jpg​ added

comment:8 by Melanie Hermann, 10 years ago

Rest bleibt so...

comment:9 by Melanie Hermann, 10 years ago

Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.