Question

这是以下代码的输出，但我希望它是正确的路径。奇数索引的输出有什么问题，例如pathsarray [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]);
    }


}

Answer 1

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

pathsarray中的字符串如何分配和分配？你没有显示那部分。他们是malloc编辑的吗？

如果您打算在创建字符串后将3个字符附加到字符串，则需要确保为字符串分配至少3个字符而不是其初始内容要求....或者您需要重新分配当你想要向它添加内容时，字符串的更大缓冲区。

Answer 2

这是一个SSCCE（Short, Self-Contained, Correct Example）。

#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
$

字符串连接与char * foo []

2 个答案: