Bugfix: 10 identifizierte Fehler und Bugs (ISSUES.md) #1

Closed

ppfeiffer wants to merge 1 commit from bugfix/issues into master

ppfeiffer commented

2026-03-06 12:25:34 +01:00

Owner

Beschreibung

Diese Branch enthält die Ergebnisse einer statischen Code-Analyse. Es wurden 10 Bugs und Fehler identifiziert und in ISSUES.md dokumentiert.

Gefundene Issues

#	Schweregrad	Datei	Problem
01	HOCH	src/l7.c:419	`clrccp()`: `break` ausserhalb `if` – Schleife läuft nur 1x
02	HOCH	src/buffer.c:723	`stringReplace()`: Buffer-Overflow möglich
03	MITTEL	src/l7.c:239	`l2tol7()`: NULL-Pointer-Dereferenz auf `tcppoi`
04	MITTEL	src/callstr.c:418	`pnmtch()`: Out-of-Bounds `n[-1]` (bekannter Node Bug)
05	NIEDRIG	src/l2misc.c:774	`change_maxframe()`: `
06	NIEDRIG	src/callstr.c:261	`ydigipt()`: Fehlende Klammern
07	NIEDRIG	src/mh.c:755	`lese_rx/tx_bytes()`: `return` ausserhalb `else`
08	NIEDRIG	src/main.c:277	`main()`: Speicherleck bei `malloc()`
09	NIEDRIG	src/mh.c:288	`ccp_mh()`: Typmischung bei Initialisierung
10	NIEDRIG	src/callstr.c:323	`call2str()`: Space-Zeichen nicht behandelt

Details und Korrekturvorschläge siehe ISSUES.md.

## Beschreibung Diese Branch enthält die Ergebnisse einer statischen Code-Analyse. Es wurden 10 Bugs und Fehler identifiziert und in `ISSUES.md` dokumentiert. ## Gefundene Issues | # | Schweregrad | Datei | Problem | |---|-------------|-------|---------| | 01 | **HOCH** | src/l7.c:419 | `clrccp()`: `break` ausserhalb `if` – Schleife läuft nur 1x | | 02 | **HOCH** | src/buffer.c:723 | `stringReplace()`: Buffer-Overflow möglich | | 03 | **MITTEL** | src/l7.c:239 | `l2tol7()`: NULL-Pointer-Dereferenz auf `tcppoi` | | 04 | **MITTEL** | src/callstr.c:418 | `pnmtch()`: Out-of-Bounds `n[-1]` (bekannter Node Bug) | | 05 | **NIEDRIG** | src/l2misc.c:774 | `change_maxframe()`: `|` statt `||` | | 06 | **NIEDRIG** | src/callstr.c:261 | `ydigipt()`: Fehlende Klammern | | 07 | **NIEDRIG** | src/mh.c:755 | `lese_rx/tx_bytes()`: `return` ausserhalb `else` | | 08 | **NIEDRIG** | src/main.c:277 | `main()`: Speicherleck bei `malloc()` | | 09 | **NIEDRIG** | src/mh.c:288 | `ccp_mh()`: Typmischung bei Initialisierung | | 10 | **NIEDRIG** | src/callstr.c:323 | `call2str()`: Space-Zeichen nicht behandelt | Details und Korrekturvorschläge siehe `ISSUES.md`.

ppfeiffer added 1 commit

2026-03-06 12:25:34 +01:00

Buganalyse: ISSUES.md mit 10 gefundenen Fehlern und Bugs erstellt 12227afc54

Statische Code-Analyse der src/-Quelldateien ergab folgende Issues:
- #01 HOCH: clrccp() - break ausserhalb if-Block (Schleife laeuft nur 1x)
- #02 HOCH: stringReplace() - Buffer-Overflow moeglich
- #03 MITTEL: l2tol7() - NULL-Pointer-Dereferenz auf tcppoi
- #04 NIEDRIG: change_maxframe() - bitweises OR statt logisches OR
- #05 NIEDRIG: ydigipt() - fehlende Klammern
- #06 MITTEL: pnmtch() - Out-of-Bounds n[-1] (bekannter Node Bug)
- #07 NIEDRIG: lese_rx/tx_bytes() - return ausserhalb else-Block
- #08 NIEDRIG: main() - Speicherleck bei malloc
- #09 NIEDRIG: ccp_mh() - Typmischung bei Initialisierung
- #10 NIEDRIG: call2str() - Space-Zeichen nicht behandelt

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>