Display: adds missing ffStrbufDestroy calls in ffOptionsDestroyDisplay (#2476)

13 FFstrbuf fields were initialized but never destroyed, potentially
leaking memory when configured via JSON or command-line:
- barBorderLeft, barBorderRight
- barBorderLeftElapsed, barBorderRightElapsed
- barColorElapsed, barColorTotal, barColorBorder
- tempColorGreen, tempColorYellow, tempColorRed
- percentColorGreen, percentColorYellow, percentColorRed

Co-authored-by: tru3 <tru3@tru3.com>
This commit is contained in:
h4sht
2026-07-24 02:43:31 +02:00
committed by GitHub
parent 97fa52fef0
commit f6c25667fe
+13
View File
@@ -897,6 +897,19 @@ void ffOptionsDestroyDisplay(FFOptionsDisplay* options) {
ffStrbufDestroy(&options->keyValueSeparator);
ffStrbufDestroy(&options->barCharElapsed);
ffStrbufDestroy(&options->barCharTotal);
ffStrbufDestroy(&options->barBorderLeft);
ffStrbufDestroy(&options->barBorderRight);
ffStrbufDestroy(&options->barBorderLeftElapsed);
ffStrbufDestroy(&options->barBorderRightElapsed);
ffStrbufDestroy(&options->barColorElapsed);
ffStrbufDestroy(&options->barColorTotal);
ffStrbufDestroy(&options->barColorBorder);
ffStrbufDestroy(&options->tempColorGreen);
ffStrbufDestroy(&options->tempColorYellow);
ffStrbufDestroy(&options->tempColorRed);
ffStrbufDestroy(&options->percentColorGreen);
ffStrbufDestroy(&options->percentColorYellow);
ffStrbufDestroy(&options->percentColorRed);
FF_LIST_FOR_EACH (FFstrbuf, item, options->constants) {
ffStrbufDestroy(item);
}