Question

这个模拟数据库的程序有一个很大的问题，每当我调用Database_set函数时，它返回segfaul，但是当我使用gdb进行调试时，每个运行都运行良好。

通过更多测试，我发现在调试过程中一切运行良好如果我创建一个新数据库（用于存储数据的文件）而不重用已经创建的数据库，如果我以这种方式执行操作，则对Database_set的调用赢了返回segfaul。

从调试器出来，这个技巧不起作用，每次我尝试访问第122行addr->set时都会得到segfaul，即使addr不是NULL指针，因为之前对malloc的调用没有出错。

以下是代码：

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

struct Address {
    int id;
    int set;
    char *name;
    char *email;
};

struct Database {
    int MAX_DATA;
    int MAX_ROWS;
    struct Address *rows;
};

struct Connection {
    FILE *file;
    struct Database *db;
};

void die(const char *message)
{
    if(errno) {
        perror(message);
    } else {
        printf("ERROR: %s\n", message);
    }

    exit(1);
}

void Address_print(struct Address *addr)
{
    printf("%d %s %s\n",
            addr->id, addr->name, addr->email);
}


void Database_load(struct Connection *conn)
{
    int rc = fread(conn->db, sizeof(struct Database), 1, conn->file);
    if(rc != 1) die("Failed to load database.");
}


struct Connection *Database_open(const char *filename, char mode)
{
    struct Connection *conn = malloc(sizeof(struct Connection));
    if(!conn) die("Memory error");

    conn->db = malloc(sizeof(struct Database));
    if(!conn->db) die("Memory error");

    if(mode == 'c') {
        conn->file = fopen(filename, "w");
    } else {
        conn->file = fopen(filename, "r+");

        if(conn->file) {
            Database_load(conn);
        }
    }

    if(!conn->file) die("Failed to open the file");

    return conn;
}

void Database_close(struct Connection *conn)
{
    if(conn) {
        if(conn->file) fclose(conn->file);
        if(conn->db) free(conn->db);
        free(conn);
    }
}

void Database_write(struct Connection *conn)
{
    rewind(conn->file);

    int rc = fwrite(conn->db, sizeof(struct Database), 1, conn->file);
    if(rc != 1) die("Failed to write database.");

    rc =fflush(conn->file);
    if(rc == -1) die("Cannot flush database.");
}

void Database_create(struct Connection *conn)
{
    printf("Please, insert max database rows limit .....");
    scanf("%d",&conn->db->MAX_ROWS);

    int i = 0;

    conn->db->rows = malloc(sizeof(struct Address)*conn->db->MAX_ROWS);
    if (!conn->db->rows) die("Memory error !");

    for (i = 0; i < conn->db->MAX_ROWS; i++) {

        /* struct Address addr = {.id = i, .set = 0};
        conn->db->rows[i] = addr; */

        conn->db->rows[i].id = 1;
        conn->db->rows[i].set = 0;
    }
}

void Database_set(struct Connection *conn, int id, const char *name, const char *email)
{

    printf("Please, insert max data limit ....");
     scanf("%d", &conn->db->MAX_DATA);


    struct Address *addr = &conn->db->rows[id];

    if(addr->set) die("Already set, delete it first");

    addr->set = 1;

    addr->name = malloc(conn->db->MAX_DATA);
    if(!addr->name) die("Memory error !");

    char *res = strcpy(addr->name, name);
    if(!res) die("Name copy failed");


    addr->email =malloc(conn->db->MAX_DATA);
    if(!addr->email) die("Memory error !");

    res = strcpy(addr->email, email);
    if(!res) die("Email copy failed");


}

void Database_get(struct Connection *conn, int id)
{
    struct Address *addr = &conn->db->rows[id];

    if(addr->set) {
        Address_print(addr);
    } else {
        die("ID is not set");
    }
}

void Database_delete(struct Connection *conn, int id)
{
    conn->db->rows[id].id = id;
    conn->db->rows[id].set = 0;
}

void Database_list(struct Connection *conn)
{
    int i = 0;
    struct Database *db = conn->db;

    for(i = 0; i < conn->db->MAX_ROWS; i++) {
        struct Address *cur = &db->rows[i];

        if(cur->set) {
            Address_print(cur);
        }
    }
}

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

    if(argc < 3) die("USAGE: ex17 <dbfile> <action> [action params]");

    char *filename = argv[1];
    char action = argv[2][0];

    struct Connection *conn = Database_open(filename, action);

    int id = 0;

    if (action != 'c') 
    {
        if (argc > 3) id = atoi(argv[4]);
        if (id >= conn->db->MAX_ROWS) die("There's not that many records.");
    }

    switch(action) {
        case 'c':
            Database_create(conn);
            Database_write(conn);
            break;

        case 'g':
            if(argc != 4) die("Need an id to get");
            Database_get(conn, id);
            break;

        case 's':
            if(argc != 6) die("Need id, name, email to set");

            Database_set(conn, id, argv[4], argv[5]);
            Database_write(conn);
            break;

        case 'd':
            if(argc != 4) die("Need id to delete");

            Database_delete(conn, id);
            Database_write(conn);
            break;

        case 'l':
            Database_list(conn);
            break;
        default:
            die("Invalid action, only: c=create, g=get, s=set, d=del, l=list");
    }

    Database_close(conn);

    return 0;
}

Answer 1

我认为你的基本错误就是这样的事实，你假设你可以通过这样做来写/读你的“数据库”：

    int rc = fwrite(conn->db, sizeof(struct Database), 1, conn->file);

由于您的struct数据库还包含此字段：

struct Address *rows;

不能这样做。这会将指针（类似于0xc2b82200）写入您的文件，而不是您的数组本身。当您尝试加载文件时，将指针设置回相同的值，但由于这是新的sesion，现在这可能指向一些未分配的内存页面，它会为您提供段错误。

您需要逐行编写“表格”。

Segfaul从调试器出来

1 个答案: