在 C 中从预分配的内存中直接读取结构失败

Question

我再次假设它是一个指针问题，但我在这里要做的是有一个函数搜索内存来查看是否存在 IP 地址，如果不存在，则保留一个随机内存空间用于IP 地址并返回零。该函数将 IP 地址作为字符串作为第一个参数，第二个参数指向结构。当 main() 函数执行时，result 应该等于 0，result2 应该等于 1，但是，我得到了崩溃，我认为这与我分配指针的方式有关？

在这个片段中...

ip=(iprec*)p;

我正在尝试设置传递给函数的 iprec 结构（之前由 malloc 分配），但我不确定在该语句中添加或删除星号的位置。

这是其余的代码。

char *shma;

typedef struct{
    unsigned char u;
    unsigned char a[4];
    unsigned int otherdata;
} iprec;

static int loadip(char* remoteip,iprec *ip){
    char *p=shma;
    int v=0;
    char *ep;
    unsigned char a[4];
    sscanf(remoteip,"%d.%d.%d.%d",a[0],a[1],a[2],a[3]);
    int i,n;int sz=5000;
    int szr=sizeof(iprec);
    for (i=3;i<sz;i+=szr){
        ip=(iprec*)p;
        if (ip->u=='Y'){
            for (n=0;n<=4;n++){
                if (a[n]!=ip->a[n]){break;}
            }
            if (n >= 3){v=1;break;}
        }else{
            ep=p;
        }
        p+=szr;
    }
    if (v==0){
        ip=(iprec*)ep;
        for (n=0;n<=3;n++){
            ip->a[n]=a[n];
        }
    }
    return v;
}

int main(){
    char *shma=(char*)malloc(5000); //alloc 5000 bytes for example.
    iprec *x=(iprec*)malloc(sizeof(iprec));
    int result=loadip("127.0.0.1",x);
    int result2=loadip("127.0.0.1",x);
}

Answer 1

1. 替换

   sscanf(remoteip,"%d.%d.%d.%d",a[0],a[1],a[2],a[3]);

与

sscanf(remoteip,"%d.%d.%d.%d",&a[0],&a[1],&a[2],&a[3]);

sccanf 需要指针类型参数。

1. shma 应该是全局的。

Answer 2

我尚未对您的代码进行全面分析，但这里是您崩溃的根源。底部是分辨率

分析

在你的 sn-p 的顶部，你有

char *shma;

这是一个名为 shma 的文件范围（全局）对象，具有静态持续时间存储。它在 main() 执行之前被初始化为 0。

然后，主要是：

int main(){
    char *shma=(char*)malloc(5000); //alloc 5000 bytes for example.

在这里，您声明了一个名为 shma 的新对象。它具有块范围（仅存在于 main 的上下文中），具有自动存储功能。你为它分配了一个来自 malloc() 的指针。然后你打电话给loadip。

loadip有

char *p=shma;

它声明了一个名为 p 的新对象，具有块范围（存在于 loadip 中）并具有自动存储功能。您将其设置为shma。 注意这是分配全局 shma，而不是 main 中的那个，因为 main 中的那个只存在于 main 中，而 loadip 位于 main 之外。全局 shma 为 0，所以现在 p 也是 0（至少对于您的第一个循环）。

然后，你这样做：

    ip=(iprec*)p;

所以，由于 p 为 0，现在 ip 为 0。在下一行：

    if (ip->u=='Y'){

您正在取消引用 ip 以获取成员 u。在大多数系统上，此取消引用将导致分段错误（或类似的内存访问冲突），因为您实际上已取消引用 NULL（0 地址）。

修复此问题

改变

int main(){
    char *shma=(char*)malloc(5000); //alloc 5000 bytes for example.

进入

int main(){
    shma=(char*)malloc(5000); //alloc 5000 bytes for example.

这将导致主分配给全局shma，而不是创建一个新的本地。

Answer 3

我学习 C 已经有一段时间了，但有一些想法：

char *p=shma;

但 shma 尚未初始化 - 因此 "ip=(iprec*)p;"表示 ip 将不是有效值。

unsigned char a[4];
sscanf(remoteip,"%d.%d.%d.%d",a[0],a[1],a[2],a[3]);

您需要接收变量的地址（即&a[0]、&a[1]、...）。 另外，由于 '%d' 将写入 4 字节整数（scanf 不知道它是一个 char 数组），所以最好有 "int a[4]"

ip=(iprec*)p;

但是 ip 是一个输入参数 - 所以在这里你将覆盖（即丢弃）用户在第二个参数中提供的任何值。

if (v==0){
    ip=(iprec*)ep;

即使 v=0，仍然有可能 ep 尚未初始化（例如，如果所有 ip->u == 'Y' 但 a-arrays 在第一个或第二个值上不同）