指针未传输到分配的内存

Question

我有一个学校作业，除了一部分，一切都很好，我无法弄清楚。 以下是发生我的问题的代码片段：

    //something goes wrong here, wont copy over
    puts("bravo");
    //add to end of the list
    int size_list = sizeof(environ);
    //char** tmp_ptr = ( char** )calloc( size_list + 1, ( size_list + 1 ) * sizeof( char* ) );
    char** tmp_ptr = ( char** ) malloc ( ( size_list + 1 ) * sizeof( char* ) );
    int k;
    for ( k = 0; k < size_list; ++k )
    {
        tmp_ptr[k] = environ[k];
        //memcpy(tmp_ptr[k],environ[k],sizeof(environ[k]));
    }
    //char** tmp_ptr= (char**)realloc(*environ, size_list+2);
    environ = tmp_ptr;
    environ[size_list] = (char*)malloc(len_string+1);
    strcpy(environ[size_list],full_string);
    return 1;

您可以忽略“ bravo”，这是我定位问题所在的位置。 我试图让环境具有新的变量列表，但是当我将其设置为等于tmp_ptr时，它为空。 我很确定这些值会被复制过来，但是我不知道出了什么问题。 函数结束时，tmp_ptr中的数据是否被删除？ 这可能是解释吗？ 如何正确分配和复制内存。 我尝试使用realloc，但是这给了我无效的指针错误，所以我依赖calloc或malloc。 提前致谢。

Answer 1

environ的长度不是sizeof(environ) ，因为environ是一个char ** （因此sizeof(environ)取决于您的平台是4还是8）。 您正在执行的操作实际上是清除了大部分environ因为您只复制了前几个条目。

要找出environ有多少项，请执行

int cnt = 0;
char **envp = environ;
while(*envp++) cnt++;

我还要注意这种方法的问题：由于environ是一个以NULL终止的字符串数组（不要与“以null终止的字符串”相混淆），因此您必须在末尾用自己的条目替换 NULL ，然后添加一个新输入后为新NULL 。 当前，您的代码（如果它正确计算了大小）将在NULL后添加您的条目​​，并且对程序不可见。

旁注：绝对不建议以这种方式弄乱environ 。 使用getenv / setenv代替； 如果需要大规模设置环境，请尝试使用execve 。 （因此，在您的情况下，您可以简单地通过setenv("varname", "varvalue", 1)将varname=varvalue添加到环境中（如果已经设置，则替换现有的varname映射）。

Answer 2

environ是一个指针，表示sizeof(environ)不是该数组中的项目数。 而是指针的大小，在您的情况下可能是四个或八个。

如果environ与argv （字符指针数组，其中最后一个是空指针）的结构相同，则需要遍历该数组直到找到NULL来确定其大小。

有点像（未经测试，但想法很合理）：

char **envPtr = environ;
int size_list = 0;            // probably should be size_t
while (*envPtr != NULL) {
    envPtr++;
    size_list++;
}

您可以在此完整程序中看到效果（进行了一些更改以解决一些其他错误）：

#define _GNU_SOURCE
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

#define full_string "xyzzy=plugh"
#define len_string strlen(full_string)

int main (int argc, char **argv) {
    int k, size_list = sizeof(environ);
    char **tmp_ptr = malloc ((size_list + 1) * sizeof ( char*));

    for (k = 0; k < size_list; ++k) printf ("[%s]\n", environ[k]);
    for (k = 0; k < size_list; ++k) tmp_ptr[k] = environ[k];
    tmp_ptr[size_list] = NULL;

    environ = tmp_ptr;
    environ[size_list] = malloc (len_string + 1);
    strcpy(environ[size_list],full_string);

    printf ("=====\n");
    for (k = 0; k <= size_list; ++k) printf ("[%s]\n", environ[k]);

    return 0;
}

因为我有四个字节的指针，所以只输出我的四个环境变量：

[ORBIT_SOCKETDIR=/tmp/orbit-pax]
[SSH_AGENT_PID=1978]
[GPG_AGENT_INFO=/tmp/seahorse-tNDhG9/S.gpg-agent:2005:1]
[TERM=xterm]
=====
[ORBIT_SOCKETDIR=/tmp/orbit-pax]
[SSH_AGENT_PID=1978]
[GPG_AGENT_INFO=/tmp/seahorse-tNDhG9/S.gpg-agent:2005:1]
[TERM=xterm]
[xyzzy=plugh]

更改代码以正确确定环境的大小：

#define _GNU_SOURCE
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

#define full_string "xyzzy=plugh"
#define len_string strlen(full_string)

int main(int argc, char **argv) {
    int k, size_list = 0;
    char **envPtr = environ;
    while (*envPtr != NULL) {
        envPtr++;
        size_list++;
    }
    char **tmp_ptr = malloc ((size_list + 1) * sizeof (char*));

    for (k = 0; k < size_list; ++k) printf ("[%s]\n", environ[k]);
    for (k = 0; k < size_list; ++k) tmp_ptr[k] = environ[k];
    tmp_ptr[size_list] = NULL;

    environ = tmp_ptr;
    environ[size_list] = malloc (len_string + 1);
    strcpy(environ[size_list],full_string);

    printf ("=====\n");
    for (k = 0; k <= size_list; ++k) printf ("[%s]\n", environ[k]);

    return 0;
}

给了我很多：

[ORBIT_SOCKETDIR=/tmp/orbit-pax]
[SSH_AGENT_PID=1978]
[GPG_AGENT_INFO=/tmp/seahorse-tNDhG9/S.gpg-agent:2005:1]
[TERM=xterm]
[SHELL=/bin/bash]
[GTK_RC_FILES=/etc/gtk/gtkrc:/home/pax/.gtkrc-1.2-gnome2]
[WINDOWID=62914564]
[GNOME_KEYRING_CONTROL=/tmp/keyring-RADe9n]
[GTK_MODULES=canberra-gtk-module]
[USER=pax]
:
[XAUTHORITY=/var/run/gdm3/auth-for-pax-AO1dYc/database]
[_=./testprog]
=====
[ORBIT_SOCKETDIR=/tmp/orbit-pax]
[SSH_AGENT_PID=1978]
[GPG_AGENT_INFO=/tmp/seahorse-tNDhG9/S.gpg-agent:2005:1]
[TERM=xterm]
[SHELL=/bin/bash]
[GTK_RC_FILES=/etc/gtk/gtkrc:/home/pax/.gtkrc-1.2-gnome2]
[WINDOWID=62914564]
[GNOME_KEYRING_CONTROL=/tmp/keyring-RADe9n]
[GTK_MODULES=canberra-gtk-module]
[USER=pax]
:
[XAUTHORITY=/var/run/gdm3/auth-for-pax-AO1dYc/database]
[_=./testprog]
[xyzzy=plugh]

---

就其价值而言，您的代码存在以下问题：

• 显然，环境大小的计算不正确。
• 从malloc转换返回值-在C语言中这是不明智的选择，因为它可能导致代码问题对您隐藏。
• 不将新列表的最后一个元素设置为NULL。
• 不检查内存不足情况。

除最后一个外，所有这些都已在上面的最终代码示例中修复。