阅读/ proc程序

Question

我想写一个脚本，它具有获取过程详细信息的功能。

到目前为止，我有

#include <stdio.h> 
#include <string.h> 
#include <ctype.h> 

char* getField(FILE* file, char* prop, int len){ 
char line[100], *p; 

while(fgets(line, 100, file)) { 
    if(strncmp(line, prop, len) != 0) 
      continue; 

    p = line + len + 1; 
    while(isspace(*p)) ++p; 

    break; 
} 

return p; 
} 

int main(int argc, char *argv[]) { 

    char tgid[40], status[40], *s, *n; 
    FILE* statusf; 

    printf("Please Enter PID\n"); 

    if (fgets(tgid, sizeof tgid, stdin)) { 
    //Remove new line 
    strtok(tgid, "\n"); 
    snprintf(status, 40, "/proc/%s/status", tgid); 

    statusf = fopen(status, "r"); 
    if(!statusf){ 
     perror("Error"); 
     return 0; 
    } 

    s = getField(statusf, "State:", 6); 
    n = getField(statusf, "Name:", 5); 

    printf("State: %s\n", s); 
    printf("Name: %s\n", n); 

    }else{ 
    printf("Error on input"); 
    } 

    fclose(statusf); 
    return 1; 
} 

我还在寻找指针和记忆有点模糊。当我运行这个脚本没有

n = getField(statusf, "Name:", 5); 

我得到正确的输出（如S - 睡觉）;

但是，当我调用该函数来获取进程名称，我似乎得到了同样的输出，例如。

国家：ntary_ctx 名称：ntary_ctx

而这甚至不是正确的名称。我认为问题必须是变量保持价值的功能。但我认为，当一个函数返回它的内存然后弹出堆栈。

Answer 1

代码被重新调整指针的局部变量。
这是无效的 - 它是未定义的行为（UB）。 @stark
这解释了“我似乎得到同样的输出”，因为一个可能的UB是相同的缓冲区被重新使用。代码崩溃的另一个可能性是其他候选人。

// Bad code 
char* getField(FILE* file, char* prop, int len){ 
char line[100], *p; 
... 
p = line + len + 1; 
... 
return p; // `p` points to `line[]` 
} 

代码需要复制。可以通过分配或传递目标来完成此操作，如下所示。

char* getField(FILE* file, char *dest, const char* prop, int len){ 
    if (problem) return NULL; 
    ... 
    return strcpy(dest, p); 
} 

// Example call 
char prop_state[100]; 
if (getField(statusf, prop_state, "State:", 6)) Success(); 
else Handle_Problem(); 
... 
char prop_name[100]; 
if (getField(statusf, prop_name, "Name:", 6)) Success(); 
... 

更好的代码将通过在dest大小，以便getField()可以搞定

char* getField(FILE* file, char *dest, size_t size, const char* prop, int len){ 
    ... 
    if (strlen(p) >= size) return NULL; // Not enough room 
    return strcpy(dest, p); 
} 

// usage 
if (getField(statusf, prop_state, sizeof prop_state, "State:", 6)) Success(); 
... 

Answer 2

如果您使用该方法，则需要在每次读取该字段时关闭并重新打开该文件，或者在回退到开始时重新打开该文件。编写C输入文件解析器并不是一个好方法，但它可以用于短文件和快速程序。

Answer 3

一个问题是，在getField()你要返回p这是一个指向line加上一些偏移量。但line是该函数中的局部变量，因此当函数终止时它超出了范围。 The answer是一个很好的解释。

作为第一步，你可以让它static char line[100]被允许在函数返回后使用指针，但是第二个调用仍然会覆盖你第一次读到的内容。 所以最好的办法是通过一个额外的缓冲值：

char* getField(FILE* file, char* prop, int len, char *value){ 
    char line[100], *p; // now it's ok 
    ... // everything at it is now before return 
    strcpy(value, p); 
    return value; 
} 

，并在main（），你有两个不同的缓冲区

char name[100], state[100]; // at least same size as line - length of label 
.... 
s = getField(statusf, "State:", 6, state); 
n = getField(statusf, "Name:", 5, name);