diff options
author | Benny Baumann <BenBE@geshi.org> | 2021-04-10 11:46:57 +0200 |
---|---|---|
committer | BenBE <BenBE@geshi.org> | 2021-05-23 09:22:21 +0200 |
commit | 94a52cb5c9274fe021b3fc114180294cadb598b4 (patch) | |
tree | 9d00199d74bc600283ae007464de65cc27377edd /linux | |
parent | 666f70c58c9fcd6bc27ed93b7958efd7a064133a (diff) |
Rename cmdlineBasenameOffset to cmdlineBasenameEnd to properly indicate the fields purpose
Diffstat (limited to 'linux')
-rw-r--r-- | linux/LinuxProcess.c | 30 | ||||
-rw-r--r-- | linux/LinuxProcessList.c | 10 |
2 files changed, 20 insertions, 20 deletions
diff --git a/linux/LinuxProcess.c b/linux/LinuxProcess.c index 8fef153d..c112ff89 100644 --- a/linux/LinuxProcess.c +++ b/linux/LinuxProcess.c @@ -193,7 +193,7 @@ colorized for better readability, and it is implicit that only upto */ #define TASK_COMM_LEN 16 -static bool findCommInCmdline(const char *comm, const char *cmdline, int cmdlineBasenameOffset, int *pCommStart, int *pCommEnd) { +static bool findCommInCmdline(const char *comm, const char *cmdline, int cmdlineBasenameStart, int *pCommStart, int *pCommEnd) { /* Try to find procComm in tokenized cmdline - this might in rare cases * mis-identify a string or fail, if comm or cmdline had been unsuitably * modified by the process */ @@ -201,10 +201,10 @@ static bool findCommInCmdline(const char *comm, const char *cmdline, int cmdline size_t tokenLen; const size_t commLen = strlen(comm); - if (cmdlineBasenameOffset < 0) + if (cmdlineBasenameStart < 0) return false; - for (const char *token = cmdline + cmdlineBasenameOffset; *token; ) { + for (const char *token = cmdline + cmdlineBasenameStart; *token;) { for (tokenBase = token; *token && *token != '\n'; ++token) { if (*token == '/') { tokenBase = token + 1; @@ -249,7 +249,7 @@ static int matchCmdlinePrefixWithExeSuffix(const char *cmdline, int cmdlineBaseO * that make htop's identification of the basename in cmdline unreliable. * For e.g. /usr/libexec/gdm-session-worker modifies its cmdline to * "gdm-session-worker [pam/gdm-autologin]" and htop ends up with - * procCmdlineBasenameOffset at "gdm-autologin]". This issue could arise with + * proccmdlineBasenameEnd at "gdm-autologin]". This issue could arise with * chrome as well as it stores in cmdline its concatenated argument vector, * without NUL delimiter between the arguments (which may contain a '/') * @@ -391,21 +391,21 @@ void LinuxProcess_makeCommandStr(Process* this) { char *strStart = mc->str; char *str = strStart; - int cmdlineBasenameOffset = this->cmdlineBasenameStart; - int cmdlineBasenameEnd = this->cmdlineBasenameOffset; + int cmdlineBasenameStart = this->cmdlineBasenameStart; + int cmdlineBasenameEnd = this->cmdlineBasenameEnd; if (!cmdline) { - cmdlineBasenameOffset = 0; + cmdlineBasenameStart = 0; cmdlineBasenameEnd = 0; cmdline = "(zombie)"; } - assert(cmdlineBasenameOffset >= 0); - assert(cmdlineBasenameOffset <= (int)strlen(cmdline)); + assert(cmdlineBasenameStart >= 0); + assert(cmdlineBasenameStart <= (int)strlen(cmdline)); if (!showMergedCommand || !procExe || !procComm) { /* fall back to cmdline */ if (showMergedCommand && !procExe && procComm && strlen(procComm)) { /* Prefix column with comm */ - if (strncmp(cmdline + cmdlineBasenameOffset, procComm, MINIMUM(TASK_COMM_LEN - 1, strlen(procComm))) != 0) { + if (strncmp(cmdline + cmdlineBasenameStart, procComm, MINIMUM(TASK_COMM_LEN - 1, strlen(procComm))) != 0) { WRITE_HIGHLIGHT(0, strlen(procComm), commAttr, CMDLINE_HIGHLIGHT_FLAG_COMM); str = stpcpy(str, procComm); @@ -414,11 +414,11 @@ void LinuxProcess_makeCommandStr(Process* this) { } if (showProgramPath) { - WRITE_HIGHLIGHT(cmdlineBasenameOffset, cmdlineBasenameEnd - cmdlineBasenameOffset, baseAttr, CMDLINE_HIGHLIGHT_FLAG_BASENAME); + WRITE_HIGHLIGHT(cmdlineBasenameStart, cmdlineBasenameEnd - cmdlineBasenameStart, baseAttr, CMDLINE_HIGHLIGHT_FLAG_BASENAME); (void)stpcpyWithNewlineConversion(str, cmdline); } else { - WRITE_HIGHLIGHT(0, cmdlineBasenameEnd - cmdlineBasenameOffset, baseAttr, CMDLINE_HIGHLIGHT_FLAG_BASENAME); - (void)stpcpyWithNewlineConversion(str, cmdline + cmdlineBasenameOffset); + WRITE_HIGHLIGHT(0, cmdlineBasenameEnd - cmdlineBasenameStart, baseAttr, CMDLINE_HIGHLIGHT_FLAG_BASENAME); + (void)stpcpyWithNewlineConversion(str, cmdline + cmdlineBasenameStart); } return; @@ -460,10 +460,10 @@ void LinuxProcess_makeCommandStr(Process* this) { /* Try to match procComm with procExe's basename: This is reliable (predictable) */ if (searchCommInCmdline) { /* commStart/commEnd will be adjusted later along with cmdline */ - haveCommInCmdline = findCommInCmdline(procComm, cmdline, cmdlineBasenameOffset, &commStart, &commEnd); + haveCommInCmdline = findCommInCmdline(procComm, cmdline, cmdlineBasenameStart, &commStart, &commEnd); } - int matchLen = matchCmdlinePrefixWithExeSuffix(cmdline, cmdlineBasenameOffset, procExe, exeBasenameOffset, exeBasenameLen); + int matchLen = matchCmdlinePrefixWithExeSuffix(cmdline, cmdlineBasenameStart, procExe, exeBasenameOffset, exeBasenameLen); bool haveCommField = false; diff --git a/linux/LinuxProcessList.c b/linux/LinuxProcessList.c index 047097af..fc9e58eb 100644 --- a/linux/LinuxProcessList.c +++ b/linux/LinuxProcessList.c @@ -1013,7 +1013,7 @@ static bool LinuxProcessList_readCmdlineFile(Process* process, openat_arg_t proc if (amtRead == 0) { if (process->state == 'Z') { - process->cmdlineBasenameOffset = 0; + process->cmdlineBasenameEnd = 0; } else { ((LinuxProcess*)process)->isKernelThread = true; } @@ -1135,7 +1135,7 @@ static bool LinuxProcessList_readCmdlineFile(Process* process, openat_arg_t proc if (!process->cmdline || !String_eq(command, process->cmdline)) { free_and_xStrdup(&process->cmdline, command); process->cmdlineBasenameStart = tokenStart; - process->cmdlineBasenameOffset = tokenEnd; + process->cmdlineBasenameEnd = tokenEnd; lp->mergedCommand.cmdlineChanged = true; } @@ -1460,14 +1460,14 @@ static bool LinuxProcessList_recurseProcTree(LinuxProcessList* this, openat_arg_ LinuxProcessList_readCwd(lp, procFd); } - if (proc->state == 'Z' && (proc->cmdlineBasenameOffset == 0)) { - proc->cmdlineBasenameOffset = -1; + if (proc->state == 'Z' && (proc->cmdlineBasenameEnd == 0)) { + proc->cmdlineBasenameEnd = -1; free_and_xStrdup(&proc->cmdline, command); proc->cmdlineBasenameStart = 0; lp->mergedCommand.commChanged = true; } else if (Process_isThread(proc)) { if (settings->showThreadNames || Process_isKernelThread(proc)) { - proc->cmdlineBasenameOffset = -1; + proc->cmdlineBasenameEnd = -1; free_and_xStrdup(&proc->cmdline, command); proc->cmdlineBasenameStart = 0; lp->mergedCommand.commChanged = true; |