-1

这是以下代码的输出，但我希望它是正确的路径。patharray[1] = ls/ls 等奇数索引的输出有什么问题？

---/usr/lib/lightdm/lightdm/ls
---ls/ls
---/usr/local/bin/ls
---ls/ls
---/usr/bin/ls
---ls/ls
---/bin/ls
---ls/ls


char *pathsarray[MAXARGS];//path
char *path=getenv("PATH");
tokenizeEnvPaths(path);

int i;

for(i=0;i<MAXARGS;i++)
{
    if(pathsarray[i] != NULL)
    {
        strcat(pathsarray[i],"/");
        strcat(pathsarray[i],"ls");
        fprintf(stderr,"---%s\n",pathsarray[i]);
    }


}
4

2 回答 2

1

看起来每个字符串的末尾都没有多余的空间 in pathsarray，这意味着没有可用空间将任何内容连接到它们的末尾。由于您不进行边界检查，因此您strcat会踩踏字符串末尾的内存并破坏其他内存（包括同一数组中的其他字符串）。

字符串是如何pathsarray分配和分配的？你没有展示那部分。他们被malloc教育了吗？

如果您打算在创建字符串后将 3 个字符附加到字符串，则需要确保为字符串分配的字节数至少比其初始内容所需的多 3 个字节......否则您需要重新分配更大的缓冲区稍后当您想向其附加内容时使用该字符串。

于 2012-12-08T18:51:04.720 回答
0

这是一个 SSCCE（简短、独立、正确的示例）。

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

enum { MAXARGS = 32 };
enum { MAXPATHLEN = 1024 };

char *pathsarray[MAXARGS];

static void add_path(char **dir, const char *begin, const char *end)
{
    if (end == begin)
    {
        begin = ".";
        end = begin + 1;
    }
    size_t len = end - begin;
    *dir = malloc(len + 1);
    memmove(*dir, begin, len);
    (*dir)[len] = '\0';
    printf("Found: %s\n", *dir); // Debug
}

/* Split path into components - ignore empty elements */
static size_t tokenizeEnvPaths(const char *path, char **dirs, size_t max_dirs)
{
    const char *begin = path;
    const char *end;
    size_t num_dirs = 0;

    while (num_dirs < max_dirs && (end = strchr(begin, ':')) != 0)
    {
        add_path(&dirs[num_dirs++], begin, end);
        begin = end + 1;
    }
    if (num_dirs < max_dirs && *begin != '\0')
        add_path(&dirs[num_dirs++], begin, begin + strlen(begin));
    return num_dirs;
}

int main(void)
{
    const char *path = getenv("PATH");
    size_t ndirs = tokenizeEnvPaths(path, pathsarray, MAXARGS);

    for (size_t i = 0; i < ndirs; i++)
    {
        if (pathsarray[i] != NULL)
        {
            char buffer[MAXPATHLEN];
            snprintf(buffer, sizeof(buffer), "%s/%s", pathsarray[i], "ls");
            fprintf(stderr, "---%s\n", pathsarray[i]);
        }
    }
    return(0);
}

示例输出：

$ PATH=:$PATH ./sp
Found: .
Found: /Users/jleffler/bin
Found: /usr/informix/11.70.FC6/bin
Found: /usr/gcc/v4.7.1/bin
Found: /usr/informix/11.70.FC6
Found: /Users/jleffler/perl/v5.16.0/bin
Found: /usr/local/bin
Found: /usr/bin
Found: /bin
Found: /usr/gnu/bin
Found: /usr/sbin
Found: /sbin
---./ls
---/Users/jleffler/bin/ls
---/usr/informix/11.70.FC6/bin/ls
---/usr/gcc/v4.7.1/bin/ls
---/usr/informix/11.70.FC6/ls
---/Users/jleffler/perl/v5.16.0/bin/ls
---/usr/local/bin/ls
---/usr/bin/ls
---/bin/ls
---/usr/gnu/bin/ls
---/usr/sbin/ls
---/sbin/ls
$
于 2012-12-08T19:34:20.517 回答