summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Lange <DLange@git.local>2024-04-18 22:02:38 +0200
committerDaniel Lange <DLange@git.local>2024-04-18 22:02:38 +0200
commit52486dbc0f056c37b8cc81bf2f8c2830d77c7d52 (patch)
treed68b462cdbc6f6a536815bf6a1142cf868373d3f
parent7017b86bbfafe14859d0fa94ec89351ee167db9e (diff)
parent0a1db8a770128dec1bb78e5e5217c169a797d0bc (diff)
Merge branch 'gcc14-analyzer-fixes' of BenBE/htopmain
-rw-r--r--FunctionBar.c14
-rw-r--r--FunctionBar.h2
-rw-r--r--linux/LinuxProcessTable.c2
3 files changed, 10 insertions, 8 deletions
diff --git a/FunctionBar.c b/FunctionBar.c
index 08500373..af4142e9 100644
--- a/FunctionBar.c
+++ b/FunctionBar.c
@@ -30,25 +30,25 @@ static const int FunctionBar_EnterEscEvents[] = {13, 27};
static int currentLen = 0;
FunctionBar* FunctionBar_newEnterEsc(const char* enter, const char* esc) {
- const char* functions[] = {enter, esc, NULL};
+ const char* functions[FUNCTIONBAR_MAXEVENTS + 1] = {enter, esc, NULL};
return FunctionBar_new(functions, FunctionBar_EnterEscKeys, FunctionBar_EnterEscEvents);
}
FunctionBar* FunctionBar_new(const char* const* functions, const char* const* keys, const int* events) {
FunctionBar* this = xCalloc(1, sizeof(FunctionBar));
- this->functions = xCalloc(16, sizeof(char*));
+ this->functions = xCalloc(FUNCTIONBAR_MAXEVENTS + 1, sizeof(char*));
if (!functions) {
functions = FunctionBar_FLabels;
}
- for (int i = 0; i < 15 && functions[i]; i++) {
+ for (int i = 0; i < FUNCTIONBAR_MAXEVENTS && functions[i]; i++) {
this->functions[i] = xStrdup(functions[i]);
}
if (keys && events) {
this->staticData = false;
- this->keys.keys = xCalloc(15, sizeof(char*));
- this->events = xCalloc(15, sizeof(int));
+ this->keys.keys = xCalloc(FUNCTIONBAR_MAXEVENTS, sizeof(char*));
+ this->events = xCalloc(FUNCTIONBAR_MAXEVENTS, sizeof(int));
int i = 0;
- while (i < 15 && functions[i]) {
+ while (i < FUNCTIONBAR_MAXEVENTS && functions[i]) {
this->keys.keys[i] = xStrdup(keys[i]);
this->events[i] = events[i];
i++;
@@ -64,7 +64,7 @@ FunctionBar* FunctionBar_new(const char* const* functions, const char* const* ke
}
void FunctionBar_delete(FunctionBar* this) {
- for (int i = 0; i < 15 && this->functions[i]; i++) {
+ for (int i = 0; i < FUNCTIONBAR_MAXEVENTS && this->functions[i]; i++) {
free(this->functions[i]);
}
free(this->functions);
diff --git a/FunctionBar.h b/FunctionBar.h
index f01a5ef5..06d9c6c8 100644
--- a/FunctionBar.h
+++ b/FunctionBar.h
@@ -21,6 +21,8 @@ typedef struct FunctionBar_ {
bool staticData;
} FunctionBar;
+#define FUNCTIONBAR_MAXEVENTS 15
+
FunctionBar* FunctionBar_newEnterEsc(const char* enter, const char* esc);
FunctionBar* FunctionBar_new(const char* const* functions, const char* const* keys, const int* events);
diff --git a/linux/LinuxProcessTable.c b/linux/LinuxProcessTable.c
index f3271022..73f69106 100644
--- a/linux/LinuxProcessTable.c
+++ b/linux/LinuxProcessTable.c
@@ -417,7 +417,7 @@ static bool LinuxProcessTable_readStatusFile(Process* process, openat_arg_t proc
if (!statusfile)
return false;
- char buffer[PROC_LINE_LENGTH + 1];
+ char buffer[PROC_LINE_LENGTH + 1] = {0};
while (fgets(buffer, sizeof(buffer), statusfile)) {

© 2014-2024 Faster IT GmbH | imprint | privacy policy