2

我正在尝试在我的 UNIX 系统中查找用户所属的所有组，以及每个用户的组。实现必须在 C 中。这是我的代码：

#include <stdio.h>
#include <unistd.h>
#include <sys/stat.h>
#include <sys/types.h>
#include <fcntl.h>
#include <assert.h>
#include <errno.h>
#include <stdlib.h>
#include <string.h>
#include <pwd.h>
#include <grp.h>


static void error_fatal(char* msg) 
{ perror(msg); exit(EXIT_FAILURE); }

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

    struct group* grp;
    struct passwd* pwd;
    char *name;
    int i = 0;

    setpwent();

    while((pwd = getpwent()) != NULL){

        if(  ( name = (char*) malloc( (strlen(pwd->pw_name)+1)*sizeof(char))) == NULL  ) error_fatal("malloc");
        strcpy(name, pwd->pw_name);
        printf("%s:\n", name); 

        setgrent();
        while( (grp = getgrent()) != NULL ) {
            for( i=0; i < (sizeof(grp->gr_mem)/sizeof(grp->gr_mem[0])); i++ ){
                if( /*strlen(&grp->gr_mem[i][0]) == strlen(name) && */ !strcmp(grp->gr_mem[i], name) )
                     printf("%s\n", name);
}                           }

        endgrent(); 
        free(name);

}
    endpwent();

    return 0;
}

但是在“root：”输出后出现分段错误。我很确定问题出在访问 /etc/group 文件的第四个字段中的成员列表（有关详细信息，请参见 man 5 group）。

所以，基本上我的问题是找出每个组有多少成员，所以我的计数器（我在程序中，最后一个 for 循环）会有很好的上限。

4

2 回答 2

1

你的问题在这里：

for( i=0; i < (sizeof(grp->gr_mem)/sizeof(grp->gr_mem[0])); i++ ){

struct group定义为：

       struct group {
           char   *gr_name;        /* group name */
           char   *gr_passwd;      /* group password */
           gid_t   gr_gid;         /* group ID */
           char  **gr_mem;         /* NULL-terminated array of pointers
                                      to names of group members */
       };

你假设gr_mem是一个数组，但它不是。它是一个指向数组第一个元素的指针。所以sizeof(grp->gr_mem)/sizeof(grp->gr_mem[0])给你一个指针的大小，在你的系统上可能是 8。因此，如果用户的组少于 8 个，您最终将读取数组的末尾gr_mem指向开头。

因为指向的数组gr_mem是 NULL 终止的，所以找到终止符会告诉您循环何时完成：

for( i=0; grp->gr_mem[i]; i++ ){
于 2018-12-07T19:48:31.263 回答
0

运行您的代码我检测到您的问题在调用grp->gr_mem[i] == NULL之前未验证：strcmp

if (grp->gr_mem[i] == NULL)
  continue;

在通话之前 添加这些行对我有用。strcmp

另外，不要忘记释放您正在使用的内存。我不知道这是否是您的完整代码，但在这里您应该考虑free(name)在您的 while 循环中使用。

于 2018-12-07T20:06:25.633 回答